[Intel-gfx] [PATCH 1/2] drm/i915/execlists: Add a couple more validity checks to assert_pending()
Tvrtko Ursulin
tvrtko.ursulin at linux.intel.com
Tue Dec 3 15:28:03 UTC 2019
On 03/12/2019 11:53, Chris Wilson wrote:
> Check the pending request submission is valid: that it at least has a
> reference for the submission and that the request is on the active list.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> drivers/gpu/drm/i915/gt/intel_lrc.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c
> index fef4b7e823f5..37ab9742abe7 100644
> --- a/drivers/gpu/drm/i915/gt/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
> @@ -1303,6 +1303,9 @@ assert_pending_valid(const struct intel_engine_execlists *execlists,
> unsigned long flags;
> bool ok = true;
>
> + GEM_BUG_ON(!kref_read(&rq->fence.refcount));
> + GEM_BUG_ON(!i915_request_is_active(rq));
> +
> if (ce == rq->hw_context) {
> GEM_TRACE_ERR("Dup context:%llx in pending[%zd]\n",
> ce->timeline->fence_context,
>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Regards,
Tvrtko
More information about the Intel-gfx
mailing list