[Bug 7771] Insanely slow startup on Radeon X300 SE

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jul 30 01:43:09 PDT 2007


http://bugs.freedesktop.org/show_bug.cgi?id=7771


michel at tungstengraphics.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|xorg-team at lists.x.org       |xorg-driver-ati at lists.x.org




------- Comment #20 from michel at tungstengraphics.com  2007-07-30 01:42 PST -------
Looks good, I'd like to push it but there are still some minor issues:

* The minimum actual sleep time probably depends on CONFIG_HZ and related
factors.
* The timeout value in RADEONWaitForVerticalSync() is missing a digit. Might be
worth using a define such as RADEON_VSYNC_TIMEOUT to avoid mistakes like this.
* In standard cases like this, I still think it would be better to use a global
struct timeval or one in RADEONInfoRec instead of adding a struct timeval
declaration for each caller.

Do you want to do another spin, or are you fine with me pushing with these
fixed?


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


More information about the xorg-driver-ati mailing list