On Wed, 2006-11-15 at 17:00 -0500, Alex Deucher wrote:
> On 11/15/06, Svante Signell <svante.signell@...> wrote:
> > On Wed, 2006-11-15 at 15:02 -0500, Alex Deucher wrote:
> > ...
> > > > See http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=395044 for further information.
> > > >
> > > > Thanks,
> > >
> > > Sounds like a locking problem in the DDX. the locking stuff was
> > > cleaned up when AIGLX was merged and a lot of drivers broke (savage,
> > > mga, others?). See bug 6357 (savage) for reference. I suspect the
> > > fix should be pretty simple.
> > I assume you mean the patch with id=7041, not 7035,7036. The savage
> > patch had changes in savage_dri.c and savage_driver.h. The DRILock
> > functions are called in both tdfx_accel.c and tdfx_driver.c as compared
> > to savage_dri.c. Additionally TDFXDoBlockHandler and TDFXDoWakeupHandler
> > are defined in tdfx_dri.c. Where to make the change?
> I thew together a quick and dirty untested patch:
> let me know if it works.
Thanks a lot. It works with the patch! glxgears spins at 480 fps and
other 3D applications do also work. Maybe you could release the updated
driver as version 1.2.3 and also increment the TDFX_PATCHLEVEL to 3 in
tdfx.h. (For 1.2.2 the patchlevel was not incremented, it is still
reported as 1.2.1, and never announced). When the fixed driver has been
released, I'll try to convince the Debian release managers to update the
unstable package to this driver.