Commit 41533940 authored by Chris Wilson's avatar Chris Wilson

drm/i915/selftests: Retarget igt_render_engine_reset_fallback()

The purpose of the test was to check per-engine resets would fallback to
the global reset when required, but first we actually need a test for a
basic i915_handle_error()!

Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Michel Thierry <michel.thierry@intel.com>
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarMichel Thierry <michel.thierry@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170728112110.6464-1-chris@chris-wilson.co.uk
parent e9d7486e
...@@ -770,22 +770,20 @@ static int igt_reset_queue(void *arg) ...@@ -770,22 +770,20 @@ static int igt_reset_queue(void *arg)
return err; return err;
} }
static int igt_render_engine_reset_fallback(void *arg) static int igt_handle_error(void *arg)
{ {
struct drm_i915_private *i915 = arg; struct drm_i915_private *i915 = arg;
struct intel_engine_cs *engine = i915->engine[RCS]; struct intel_engine_cs *engine = i915->engine[RCS];
struct hang h; struct hang h;
struct drm_i915_gem_request *rq; struct drm_i915_gem_request *rq;
unsigned int reset_count, reset_engine_count; struct i915_gpu_state *error;
int err = 0; int err;
/* Check that we can issue a global GPU and engine reset */ /* Check that we can issue a global GPU and engine reset */
if (!intel_has_reset_engine(i915)) if (!intel_has_reset_engine(i915))
return 0; return 0;
global_reset_lock(i915);
mutex_lock(&i915->drm.struct_mutex); mutex_lock(&i915->drm.struct_mutex);
err = hang_init(&h, i915); err = hang_init(&h, i915);
...@@ -801,69 +799,31 @@ static int igt_render_engine_reset_fallback(void *arg) ...@@ -801,69 +799,31 @@ static int igt_render_engine_reset_fallback(void *arg)
i915_gem_request_get(rq); i915_gem_request_get(rq);
__i915_add_request(rq, true); __i915_add_request(rq, true);
/* make reset engine fail */
rq->fence.error = -EIO;
if (!wait_for_hang(&h, rq)) { if (!wait_for_hang(&h, rq)) {
pr_err("Failed to start request %x\n", rq->fence.seqno); pr_err("Failed to start request %x\n", rq->fence.seqno);
err = -EIO; err = -EIO;
goto err_request; goto err_request;
} }
reset_engine_count = i915_reset_engine_count(&i915->gpu_error, engine);
reset_count = fake_hangcheck(rq);
/* unlock since we'll call handle_error */
mutex_unlock(&i915->drm.struct_mutex); mutex_unlock(&i915->drm.struct_mutex);
global_reset_unlock(i915);
i915_handle_error(i915, intel_engine_flag(engine), "live test"); /* Temporarily disable error capture */
error = xchg(&i915->gpu_error.first_error, (void *)-1);
if (i915_reset_engine_count(&i915->gpu_error, engine) != engine->hangcheck.stalled = true;
reset_engine_count) { engine->hangcheck.seqno = intel_engine_get_seqno(engine);
pr_err("render engine reset recorded! (full reset expected)\n");
err = -EINVAL;
goto out_rq;
}
if (i915_reset_count(&i915->gpu_error) == reset_count) { i915_handle_error(i915, intel_engine_flag(engine), "%s", __func__);
pr_err("No full GPU reset recorded!\n");
err = -EINVAL;
goto out_rq;
}
/* xchg(&i915->gpu_error.first_error, error);
* by using fence.error = -EIO, full reset sets the wedged flag, do one
* more full reset to re-enable the hw.
*/
if (i915_terminally_wedged(&i915->gpu_error)) {
global_reset_lock(i915);
rq->fence.error = 0;
mutex_lock(&i915->drm.struct_mutex);
set_bit(I915_RESET_HANDOFF, &i915->gpu_error.flags);
i915_reset(i915, I915_RESET_QUIET);
GEM_BUG_ON(test_bit(I915_RESET_HANDOFF,
&i915->gpu_error.flags));
mutex_unlock(&i915->drm.struct_mutex);
if (i915_reset_count(&i915->gpu_error) == reset_count) {
pr_err("No full GPU reset recorded!\n");
err = -EINVAL;
goto out_rq;
}
}
out_rq:
i915_gem_request_put(rq);
hang_fini(&h);
out_backoff:
global_reset_unlock(i915);
if (i915_terminally_wedged(&i915->gpu_error)) mutex_lock(&i915->drm.struct_mutex);
return -EIO;
return err; if (rq->fence.error != -EIO) {
pr_err("Guilty request not identified!\n");
err = -EINVAL;
goto err_request;
}
err_request: err_request:
i915_gem_request_put(rq); i915_gem_request_put(rq);
...@@ -871,7 +831,7 @@ static int igt_render_engine_reset_fallback(void *arg) ...@@ -871,7 +831,7 @@ static int igt_render_engine_reset_fallback(void *arg)
hang_fini(&h); hang_fini(&h);
err_unlock: err_unlock:
mutex_unlock(&i915->drm.struct_mutex); mutex_unlock(&i915->drm.struct_mutex);
goto out_backoff; return err;
} }
int intel_hangcheck_live_selftests(struct drm_i915_private *i915) int intel_hangcheck_live_selftests(struct drm_i915_private *i915)
...@@ -883,7 +843,7 @@ int intel_hangcheck_live_selftests(struct drm_i915_private *i915) ...@@ -883,7 +843,7 @@ int intel_hangcheck_live_selftests(struct drm_i915_private *i915)
SUBTEST(igt_reset_active_engines), SUBTEST(igt_reset_active_engines),
SUBTEST(igt_wait_reset), SUBTEST(igt_wait_reset),
SUBTEST(igt_reset_queue), SUBTEST(igt_reset_queue),
SUBTEST(igt_render_engine_reset_fallback), SUBTEST(igt_handle_error),
}; };
if (!intel_has_gpu_reset(i915)) if (!intel_has_gpu_reset(i915))
......
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