Comment 49 for bug 347618

Revision history for this message
In , Udovdh (udovdh) wrote :

After undoing the patch at https://bugs.freedesktop.org/attachment.cgi?id=25571 amd applying this patch:

      --- a/src/r600_exa.c
      +++ b/src/r600_exa.c
      @@ -267,6 +267,7 @@ R600Solid(PixmapPtr pPix, int x1, int y1, int x2, int y2)

           if (((accel_state->vb_index + 3) * 8) > (accel_state->ib->total / 2)) {
              R600DoneSolid(pPix);
      + ErrorF("udo: bla\n");
              accel_state->vb_index = 0;
              accel_state->ib = RHDDRMCPBuffer(pScrn->scrnIndex);
           }

And staring Audacity and generating some DTMF we saw some `udo: bla` lines logged.
Conclusion:

<agd5f> udovdh: ok. well, if it's really dropping rendering due to missing state, the ddx really needs to be re-organized like mesa to set and emit state functions