On Tue, Mar 19, 2013 at 11:02:14AM +0100, Maarten Lankhorst wrote: > Hey, > > Op 19-03-13 10:21, Chris Wilson schreef: > > On Mon, Mar 18, 2013 at 01:51:44PM -0700, Bryce Harrington wrote: > >> Update: Squashes a couple commits to avoid potential hang if > >> git bisecting. No other changes from v1. > > I'd probably drop the last EAGAIN patch as that is part of the libdrm > > API, but other than that it looks to be a reasonably self-contained w/a > > for this perplexing problem. > > > > Reviewed-by: Chris Wilson <[email protected]> > > -Chris > > > And completely wrong, version I pushed to ubuntu's xorg-server for comparison: > > Nacked-by: Maarten Lankhorst <[email protected]>
So you pushed the busy-spin into drmSetMaster(), which is just a tighter variant of the above. Anything which adds the minimal delay, warns about that delay, and works around the issue is fine by me. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
