Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > 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@xxxxxxxxxxxxxxxxxx> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> Reviewed-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/selftests/intel_hangcheck.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/selftests/intel_hangcheck.c b/drivers/gpu/drm/i915/selftests/intel_hangcheck.c > index 7e1bdd88eda3..71ce06680d66 100644 > --- a/drivers/gpu/drm/i915/selftests/intel_hangcheck.c > +++ b/drivers/gpu/drm/i915/selftests/intel_hangcheck.c > @@ -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; > } > -- > 2.14.2 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
