Quoting Chris Wilson (2019-03-18 16:30:58)
> Quoting Tvrtko Ursulin (2019-03-18 16:22:12)
> >
> > On 18/03/2019 09:51, Chris Wilson wrote:
> > > +static int gem_context_register(struct i915_gem_context *ctx,
> > > + struct drm_i915_file_private *fpriv)
> > > +{
> > > + int ret;
> > > +
> >
> > Assert struct mutex for now? Without it userspace can still see not
> > fully initialized ctx. It is kind of two arguments but good for
> > documentation nevertheless I think.
>
> The goal is that we need to fix that now. And we can't hold struct_mutex
> across the extensions, as we want to wrap ctx_setparam which expects to
> be able to take struct_mutex. So it has to be registered outside of
> struct_mutex in this or the next path.
Waitasec... It is the very next patch that drops the struct_mutex. (I
was thinking it was removed in this patch.)
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
