Quoting Lionel Landwerlin (2019-07-09 13:53:38)
> On 09/07/2019 15:33, Lionel Landwerlin wrote:
> >
> > +static int eb_oa_config(struct i915_execbuffer *eb)
> > +{
> > + int ret;
> > +
> > + if (!eb->oa_config)
> > + return 0;
> > +
> > + ret = i915_mutex_lock_interruptible(&eb->i915->drm);
> > + if (ret)
> > + return ret;
>
> This is assuming the lock is dropped from the calling function.
>
> It's not at the moment, but I expect you'll be doing that soon?
Hence why I'm checking everything that has an implicit struct_mutex :)
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
