On Sun, Dec 13, 2009 at 9:06 PM, Patrice Mandin <[email protected]> wrote: > Le Sun, 13 Dec 2009 16:07:37 +0100 > Maarten Maathuis <[email protected]> a écrit: > >> - Added flush notify functions for NV30 and NV40. >> - NV30 and NV40 need testing (check for regressions). > > Just tested for NV30, no visible regression, just a minor performance > penalty. So it's OK for me. > > Timedemo with ioquake3: > > mesa git head: > 1260 frames 31.5 seconds 40.0 fps 9.0/25.0/50.0/8.0 ms > 1260 frames 31.4 seconds 40.1 fps 9.0/24.9/50.0/7.9 ms > > with your patch, outofreloc.diff: > 1260 frames 32.7 seconds 38.5 fps 9.0/26.0/52.0/8.2 ms > 1260 frames 32.6 seconds 38.7 fps 9.0/25.8/51.0/8.1 ms
That means (i guess) that you were flushing more than state emit was happening, since it's the same chunk of code just called after a flush and not a state emit (which is not correct anyway). So as long as rendering is still ok, i'm not worried. > > -- > Patrice Mandin > WWW: http://pmandin.atari.org/ > Programmeur Linux, Atari > Spécialité: Développement, jeux > > > > _______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
