I had Sven test this patch... he said it works. I think chipset number test was executing code that we thought was only either 0x50 or 0xc0, but was actually more specific with things like 0x92. On Sun, Nov 11, 2012 at 2:25 PM, Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxx> wrote: > Op 11-11-12 19:35, Marcin Slusarz schreef: >> On Sun, Nov 11, 2012 at 07:26:17PM +0100, Marcin Slusarz wrote: >>> On Tue, Nov 06, 2012 at 07:30:00PM +0100, Maarten Lankhorst wrote: >>>> >>>> Op 06-11-12 15:48, Kelly Doran schreef: >>>>> The vblank on nvc0 was not working correctly and would freeze X, I managed >>>>> to track it down and fix it with some help from m.b.lankhorst, see >>>>> https://bugs.freedesktop.org/show_bug.cgi?id=56692 for details. >>>>> >>>> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxx> >>>> >>>> I recommended the bar flush since nvd0 does as well, and there might be a small race >>>> otherwise. >>>> >>>> Can this still get in before final 3.7 release? >>> This patch breaks vblank here (nv92) completely. > Does removing the bar flush help? > > ~Maarte
Attachment:
vblank-fix1.patch
Description: Binary data
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
