[Intel-gfx] [PATCH 3/3] drm/i915: Enable rcu-only context lookups

Chris Wilson chris at chris-wilson.co.uk
Thu Mar 30 13:41:40 UTC 2017


On Thu, Mar 30, 2017 at 04:32:37PM +0300, Joonas Lahtinen wrote:
> On ke, 2017-03-29 at 19:28 +0100, Chris Wilson wrote:
> > Whilst the contents of the context is still protected by the big
> > struct_mutex, this is not much of an improvement. It is just one tiny
> > step towards reducing our BKL.
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> 
> <SNIP>
> 
> > +	/* We opt for unserialised reads here. This may result in tearing
> > +	 * in the extremely unlikely event of a GPU hang on this context
> > +	 * as we are querying them. If we need that extra layer of protection,
> > +	 * we should wrap the hangstats with a seqlock.
> > +	 */
> >  
> >  	if (capable(CAP_SYS_ADMIN))
> >  		args->reset_count = i915_reset_count(&dev_priv->gpu_error);
> >  	else
> >  		args->reset_count = 0;
> >  
> 
> Kill the extra newline.

Fortunately it was just a mirage introduced by diff.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list