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 <chris at chris-wilson.co.uk> > > -Chris > > > And completely wrong, version I pushed to ubuntu's xorg-server for comparison: > > Nacked-by: Maarten Lankhorst <maarten.lankhorst at canonical.com> 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
