diff options
| author |   Chris Wilson <chris@chris-wilson.co.uk> | 2019-07-03 10:17:12 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2019-07-03 11:07:57 +0100 |
| commit | 63251685c141ef8db611cd36609a8369d97b47da (patch) | |
| tree | b7c5992223347b412afc44ad727444b5a57e2af8 /drivers/gpu/drm/i915/gt/selftest_timeline.c | |
| parent | drm/i915/display: Handle lost primary_port across suspend (diff) | |
| download | linux-dev-63251685c141ef8db611cd36609a8369d97b47da.tar.xz linux-dev-63251685c141ef8db611cd36609a8369d97b47da.zip | |
drm/i915/selftests: Common live setup/teardown
We frequently, but not frequently enough!, remember to flush residual
operations and objects at the end of a live subtest. The purpose is to
cleanup after every subtest, leaving a clean slate for the next subtest,
and perform early detection of leaky state. As this should ideally be
common for all live subtests, pull the task into a common teardown
routine.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190703091726.11690-1-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/gt/selftest_timeline.c')
| -rw-r--r-- | drivers/gpu/drm/i915/gt/selftest_timeline.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/gt/selftest_timeline.c b/drivers/gpu/drm/i915/gt/selftest_timeline.c index 193cc564ade2..eae3b1963bf7 100644 --- a/drivers/gpu/drm/i915/gt/selftest_timeline.c +++ b/drivers/gpu/drm/i915/gt/selftest_timeline.c @@ -821,8 +821,6 @@ static int live_hwsp_recycle(void *arg) } out: - if (igt_flush_test(i915, I915_WAIT_LOCKED)) - err = -EIO; intel_runtime_pm_put(&i915->runtime_pm, wakeref); mutex_unlock(&i915->drm.struct_mutex); @@ -841,5 +839,5 @@ int intel_timeline_live_selftests(struct drm_i915_private *i915) if (i915_terminally_wedged(i915)) return 0; - return i915_subtests(tests, i915); + return i915_live_subtests(tests, i915); } |