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