drm/i915/selftests: Common live setup/teardown
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 3 Jul 2019 09:17:12 +0000 (10:17 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 3 Jul 2019 10:07:57 +0000 (11:07 +0100)
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
drivers/gpu/drm/i915/gem/selftests/i915_gem_client_blt.c
drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c
drivers/gpu/drm/i915/gem/selftests/i915_gem_object_blt.c
drivers/gpu/drm/i915/gt/selftest_hangcheck.c
drivers/gpu/drm/i915/gt/selftest_lrc.c
drivers/gpu/drm/i915/gt/selftest_timeline.c
drivers/gpu/drm/i915/i915_selftest.h
drivers/gpu/drm/i915/selftests/i915_gem.c
drivers/gpu/drm/i915/selftests/i915_selftest.c

index 8554812..fa79233 100644 (file)
@@ -11,8 +11,8 @@
 
 static int igt_client_fill(void *arg)
 {
-       struct intel_context *ce = arg;
-       struct drm_i915_private *i915 = ce->gem_context->i915;
+       struct drm_i915_private *i915 = arg;
+       struct intel_context *ce = i915->engine[BCS0]->kernel_context;
        struct drm_i915_gem_object *obj;
        struct rnd_state prng;
        IGT_TIMEOUT(end);
@@ -89,11 +89,6 @@ err_unpin:
 err_put:
        i915_gem_object_put(obj);
 err_flush:
-       mutex_lock(&i915->drm.struct_mutex);
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
-       mutex_unlock(&i915->drm.struct_mutex);
-
        if (err == -ENOMEM)
                err = 0;
 
@@ -112,5 +107,5 @@ int i915_gem_client_blt_live_selftests(struct drm_i915_private *i915)
        if (!HAS_ENGINE(i915, BCS0))
                return 0;
 
-       return i915_subtests(tests, i915->engine[BCS0]->kernel_context);
+       return i915_live_subtests(tests, i915);
 }
index 53c81b5..a23c6df 100644 (file)
@@ -1731,5 +1731,5 @@ int i915_gem_context_live_selftests(struct drm_i915_private *dev_priv)
        if (i915_terminally_wedged(dev_priv))
                return 0;
 
-       return i915_subtests(tests, dev_priv);
+       return i915_live_subtests(tests, dev_priv);
 }
index e23d8c9..11d3723 100644 (file)
@@ -11,8 +11,8 @@
 
 static int igt_fill_blt(void *arg)
 {
-       struct intel_context *ce = arg;
-       struct drm_i915_private *i915 = ce->gem_context->i915;
+       struct drm_i915_private *i915 = arg;
+       struct intel_context *ce = i915->engine[BCS0]->kernel_context;
        struct drm_i915_gem_object *obj;
        struct rnd_state prng;
        IGT_TIMEOUT(end);
@@ -83,11 +83,6 @@ err_unpin:
 err_put:
        i915_gem_object_put(obj);
 err_flush:
-       mutex_lock(&i915->drm.struct_mutex);
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
-       mutex_unlock(&i915->drm.struct_mutex);
-
        if (err == -ENOMEM)
                err = 0;
 
@@ -106,5 +101,5 @@ int i915_gem_object_blt_live_selftests(struct drm_i915_private *i915)
        if (!HAS_ENGINE(i915, BCS0))
                return 0;
 
-       return i915_subtests(tests, i915->engine[BCS0]->kernel_context);
+       return i915_live_subtests(tests, i915);
 }
index cf592a0..2d9cc3c 100644 (file)
@@ -1744,7 +1744,7 @@ int intel_hangcheck_live_selftests(struct drm_i915_private *i915)
        saved_hangcheck = fetch_and_zero(&i915_modparams.enable_hangcheck);
        drain_delayed_work(&i915->gpu_error.hangcheck_work); /* flush param */
 
-       err = i915_subtests(tests, i915);
+       err = i915_live_subtests(tests, i915);
 
        mutex_lock(&i915->drm.struct_mutex);
        igt_flush_test(i915, I915_WAIT_LOCKED);
index 0c97f95..11f4905 100644 (file)
@@ -73,7 +73,6 @@ err_ctx:
 err_spin:
        igt_spinner_fini(&spin);
 err_unlock:
-       igt_flush_test(i915, I915_WAIT_LOCKED);
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -290,8 +289,6 @@ err_map:
 err_obj:
        i915_gem_object_put(obj);
 err_unlock:
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
 
@@ -472,8 +469,6 @@ err_ctx_lo:
 err_ctx_hi:
        kernel_context_close(ctx_hi);
 err_unlock:
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -580,7 +575,6 @@ err_spin_lo:
 err_spin_hi:
        igt_spinner_fini(&spin_hi);
 err_unlock:
-       igt_flush_test(i915, I915_WAIT_LOCKED);
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -687,7 +681,6 @@ err_spin_lo:
 err_spin_hi:
        igt_spinner_fini(&spin_hi);
 err_unlock:
-       igt_flush_test(i915, I915_WAIT_LOCKED);
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -826,8 +819,6 @@ err_client_b:
 err_client_a:
        preempt_client_fini(&a);
 err_unlock:
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -995,8 +986,6 @@ err_client_1:
 err_client_0:
        preempt_client_fini(&client[0]);
 err_unlock:
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -1143,8 +1132,6 @@ err_client_lo:
 err_client_hi:
        preempt_client_fini(&hi);
 err_unlock:
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -1270,7 +1257,6 @@ err_spin_lo:
 err_spin_hi:
        igt_spinner_fini(&spin_hi);
 err_unlock:
-       igt_flush_test(i915, I915_WAIT_LOCKED);
        intel_runtime_pm_put(&i915->runtime_pm, wakeref);
        mutex_unlock(&i915->drm.struct_mutex);
        return err;
@@ -2054,5 +2040,5 @@ int intel_execlists_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);
 }
index 193cc56..eae3b19 100644 (file)
@@ -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);
 }
index 207e21b..d9b17b9 100644 (file)
@@ -66,12 +66,28 @@ struct i915_subtest {
        const char *name;
 };
 
+int __i915_nop_setup(void *data);
+int __i915_nop_teardown(int err, void *data);
+
+int __i915_live_setup(void *data);
+int __i915_live_teardown(int err, void *data);
+
 int __i915_subtests(const char *caller,
+                   int (*setup)(void *data),
+                   int (*teardown)(int err, void *data),
                    const struct i915_subtest *st,
                    unsigned int count,
                    void *data);
 #define i915_subtests(T, data) \
-       __i915_subtests(__func__, T, ARRAY_SIZE(T), data)
+       __i915_subtests(__func__, \
+                       __i915_nop_setup, __i915_nop_teardown, \
+                       T, ARRAY_SIZE(T), data)
+#define i915_live_subtests(T, data) ({ \
+       typecheck(struct drm_i915_private *, data); \
+       __i915_subtests(__func__, \
+                       __i915_live_setup, __i915_live_teardown, \
+                       T, ARRAY_SIZE(T), data); \
+})
 
 #define SUBTEST(x) { x, #x }
 
index ed0c17b..b8ffae4 100644 (file)
@@ -154,8 +154,6 @@ static int igt_gem_suspend(void *arg)
 
        mutex_lock(&i915->drm.struct_mutex);
        err = switch_to_context(i915, ctx);
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        mutex_unlock(&i915->drm.struct_mutex);
 out:
        mock_file_free(i915, file);
@@ -195,8 +193,6 @@ static int igt_gem_hibernate(void *arg)
 
        mutex_lock(&i915->drm.struct_mutex);
        err = switch_to_context(i915, ctx);
-       if (igt_flush_test(i915, I915_WAIT_LOCKED))
-               err = -EIO;
        mutex_unlock(&i915->drm.struct_mutex);
 out:
        mock_file_free(i915, file);
@@ -213,5 +209,5 @@ int i915_gem_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);
 }
index b18eaef..f46ccf8 100644 (file)
@@ -26,6 +26,8 @@
 #include "../i915_drv.h"
 #include "../i915_selftest.h"
 
+#include "igt_flush_test.h"
+
 struct i915_selftest i915_selftest __read_mostly = {
        .timeout_ms = 1000,
 };
@@ -240,7 +242,40 @@ static bool apply_subtest_filter(const char *caller, const char *name)
        return result;
 }
 
+int __i915_nop_setup(void *data)
+{
+       return 0;
+}
+
+int __i915_nop_teardown(int err, void *data)
+{
+       return err;
+}
+
+int __i915_live_setup(void *data)
+{
+       struct drm_i915_private *i915 = data;
+
+       return i915_terminally_wedged(i915);
+}
+
+int __i915_live_teardown(int err, void *data)
+{
+       struct drm_i915_private *i915 = data;
+
+       mutex_lock(&i915->drm.struct_mutex);
+       if (igt_flush_test(i915, I915_WAIT_LOCKED))
+               err = -EIO;
+       mutex_unlock(&i915->drm.struct_mutex);
+
+       i915_gem_drain_freed_objects(i915);
+
+       return err;
+}
+
 int __i915_subtests(const char *caller,
+                   int (*setup)(void *data),
+                   int (*teardown)(int err, void *data),
                    const struct i915_subtest *st,
                    unsigned int count,
                    void *data)
@@ -255,10 +290,17 @@ int __i915_subtests(const char *caller,
                if (!apply_subtest_filter(caller, st->name))
                        continue;
 
+               err = setup(data);
+               if (err) {
+                       pr_err(DRIVER_NAME "/%s: setup failed for %s\n",
+                              caller, st->name);
+                       return err;
+               }
+
                pr_info(DRIVER_NAME ": Running %s/%s\n", caller, st->name);
                GEM_TRACE("Running %s/%s\n", caller, st->name);
 
-               err = st->func(data);
+               err = teardown(st->func(data), data);
                if (err && err != -EINTR) {
                        pr_err(DRIVER_NAME "/%s: %s failed with error %d\n",
                               caller, st->name, err);