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@xxxxxxxxxxxxxxxxxx> > > <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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
