On 28/08/2019 22:41, Chris Wilson wrote:
Quoting Lionel Landwerlin (2019-08-28 15:33:26)
+ rq = i915_request_create(i915->engine[RCS0]->kernel_context);
+ if (IS_ERR(rq))
+ return PTR_ERR(rq);
+
+ err = i915_active_request_set(&i915->engine[RCS0]->last_oa_config,
+ rq);
I am still not sold on putting the global oa_config timeline on the
engine, and still dislike the suggestion of using struct_mutex to
management it.
Where else could we put it?
You will notice that i915_active_request_set() checks that you have the
mutex held that your initialised with (in this case that is a *NULL).
-Chris
Will fix that, thanks.
-Lionel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
