[Intel-gfx] [PATCH v2 0/7] xfree86: Handle drm race condition

Chris Wilson chris at chris-wilson.co.uk
Tue Mar 19 11:27:07 CET 2013


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



More information about the Intel-gfx mailing list