drm/i915/selftests: Hold the rpm wakeref for the reset tests
authorChris Wilson <chris@chris-wilson.co.uk>
Mon, 9 Oct 2017 11:03:00 +0000 (12:03 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 9 Oct 2017 16:07:28 +0000 (17:07 +0100)
The lowlevel reset functions expect the caller to be holding the rpm
wakeref for the device access across the reset. We were not explicitly
doing this in the sefltest, so for simplicity acquire the wakeref for
the duration of all subtests.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20171009110301.21705-4-chris@chris-wilson.co.uk
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
drivers/gpu/drm/i915/selftests/intel_hangcheck.c

index 7e1bdd88eda30e65db27f43b1d4ffec18baac835..71ce06680d66ad0db52c194d163315faaaa7ef7d 100644 (file)
@@ -878,9 +878,16 @@ int intel_hangcheck_live_selftests(struct drm_i915_private *i915)
                SUBTEST(igt_reset_queue),
                SUBTEST(igt_handle_error),
        };
+       int err;
 
        if (!intel_has_gpu_reset(i915))
                return 0;
 
-       return i915_subtests(tests, i915);
+       intel_runtime_pm_get(i915);
+
+       err = i915_subtests(tests, i915);
+
+       intel_runtime_pm_put(i915);
+
+       return err;
 }