r300 mergedfb+mouse lockups back

2005-05-16 Thread Lorenzo Colitti
Hi, I recently updated my x.org, mesa and r300 trees to current code, but the machine would lock up if I moved the mouse when a 3d app was running. This used to happen a while ago too, but then Vladimir fixed it by calling RADEONWaitForIdleMMIO() in RADEONChooseCursorCRTC():

Re: r300 mergedfb+mouse lockups back

2005-05-16 Thread Dave Airlie
I reapplied Vladimir's fix and the machine doesn't lock up any more. Maybe the fix should be checked in to CVS again? http://cvs.freedesktop.org/xorg/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_mergedfb.c?rev=1.9view=log v 1.7 backs it out because it is buggy... Dave. -- David

Re: r300 mergedfb+mouse lockups back

2005-05-16 Thread Lorenzo Colitti
Dave Airlie wrote: I reapplied Vladimir's fix and the machine doesn't lock up any more. Maybe the fix should be checked in to CVS again? http://cvs.freedesktop.org/xorg/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_mergedfb.c?rev=1.9view=log v 1.7 backs it out because it is buggy... I know, I

Re: r300 mergedfb+mouse lockups back

2005-05-16 Thread Dave Airlie
I know, I read the checkin comment. However, is buggy better than locked up hard? usually no... people try to fix locked up hard.. buggy always comes back and bites you in the arse, 2 years down the road when you've forgotten about it all.. Dave. -- David Airlie, Software Engineer

Re: r300 mergedfb+mouse lockups back

2005-05-16 Thread Vladimir Dergachev
On Tue, 17 May 2005, Lorenzo Colitti wrote: Dave Airlie wrote: I reapplied Vladimir's fix and the machine doesn't lock up any more. Maybe the fix should be checked in to CVS again? http://cvs.freedesktop.org/xorg/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_mergedfb.c?rev=1.9view=log v 1.7

Re: r300 mergedfb+mouse lockups back

2005-05-16 Thread Aapo Tahkola
On Mon, 16 May 2005 23:23:49 -0400 (EDT) Vladimir Dergachev [EMAIL PROTECTED] wrote: On Tue, 17 May 2005, Lorenzo Colitti wrote: Dave Airlie wrote: I reapplied Vladimir's fix and the machine doesn't lock up any more. Maybe the fix should be checked in to CVS again?