Chris Wilson <[email protected]> writes: > Quoting Mika Kuoppala (2019-06-14 16:50:33) >> Chris Wilson <[email protected]> writes: >> > @@ -254,6 +254,7 @@ static bool i915_request_retire(struct i915_request >> > *rq) >> > intel_context_unpin(rq->hw_context); >> > >> > i915_request_remove_from_client(rq); >> > + list_del(&rq->link); >> >> This is now unguarded. and the timeline mutex is not here >> to help either. > > It is guarded by the caller to i915_request_retire, we can only retire > under the same lock as we construct requests. Currently that is > struct_mutex with the big switcheroo coming at the end.
Yup found it. Reviewed-by: Mika Kuoppala <[email protected]> > -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
