Commit 71a6688e authored by Chris Wilson's avatar Chris Wilson

drm/i915/selftests: Tidy up an error message for live_error_interrupt

Since we don't wait for the error interrupt to reset, restart and then
complete the guilty request, clean up the error messages.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarMika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200331091459.29179-1-chris@chris-wilson.co.uk
parent f53ae29c
...@@ -649,9 +649,9 @@ static int live_error_interrupt(void *arg) ...@@ -649,9 +649,9 @@ static int live_error_interrupt(void *arg)
error_repr(p->error[i])); error_repr(p->error[i]));
if (!i915_request_started(client[i])) { if (!i915_request_started(client[i])) {
pr_debug("%s: %s request not stated!\n", pr_err("%s: %s request not started!\n",
engine->name, engine->name,
error_repr(p->error[i])); error_repr(p->error[i]));
err = -ETIME; err = -ETIME;
goto out; goto out;
} }
...@@ -659,9 +659,10 @@ static int live_error_interrupt(void *arg) ...@@ -659,9 +659,10 @@ static int live_error_interrupt(void *arg)
/* Kick the tasklet to process the error */ /* Kick the tasklet to process the error */
intel_engine_flush_submission(engine); intel_engine_flush_submission(engine);
if (client[i]->fence.error != p->error[i]) { if (client[i]->fence.error != p->error[i]) {
pr_err("%s: %s request completed with wrong error code: %d\n", pr_err("%s: %s request (%s) with wrong error code: %d\n",
engine->name, engine->name,
error_repr(p->error[i]), error_repr(p->error[i]),
i915_request_completed(client[i]) ? "completed" : "running",
client[i]->fence.error); client[i]->fence.error);
err = -EINVAL; err = -EINVAL;
goto out; goto out;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment