[PATCH 2/2] bug2?

[Tvrtko Ursulin]

From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
---
 drivers/gpu/drm/i915/gt/intel_ring_submission.c | 7 ++-----
 drivers/gpu/drm/i915/i915_request.c             | 3 ++-
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_ring_submission.c b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
index 2822b086cdf6..9388831b116a 100644
--- a/drivers/gpu/drm/i915/gt/intel_ring_submission.c
+++ b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
@@ -591,13 +591,10 @@ static void ring_context_revoke(struct intel_context *ce,
 				struct i915_request *rq,
 				unsigned int preempt_timeout_ms)
 {
-	struct intel_engine_cs *engine;
-
-	if (!rq || !i915_request_is_active(rq))
-		return;
+	struct intel_engine_cs *engine = ce->engine;
 
-	engine = rq->engine;
 	lockdep_assert_held(&engine->sched_engine->lock);
+
 	list_for_each_entry_continue(rq, &engine->sched_engine->requests,
 				     sched.link)
 		if (rq->context == ce) {
diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
index b1a9bec83339..cb37ac16cb95 100644
--- a/drivers/gpu/drm/i915/i915_request.c
+++ b/drivers/gpu/drm/i915/i915_request.c
@@ -2019,12 +2019,13 @@ void i915_request_show(struct drm_printer *m,
 
 	x = print_sched_attr(&rq->sched.attr, buf, x, sizeof(buf));
 
-	drm_printf(m, "%s%.*s%c %llx:%lld%s%s %s @ %dms: %s\n",
+	drm_printf(m, "%s%.*s%c %llx:%lld%s%s (%d) %s @ %dms: %s\n",
 		   prefix, indent, "                ",
 		   queue_status(rq),
 		   rq->fence.context, rq->fence.seqno,
 		   run_status(rq),
 		   fence_status(rq),
+		   READ_ONCE(rq->fence.error),
 		   buf,
 		   jiffies_to_msecs(jiffies - rq->emitted_jiffies),
 		   name);
-- 
2.30.2