drm/i915/selftests: Ignore self-preemption suppression under gvt
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 12 Jul 2019 08:25:49 +0000 (09:25 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 15 Jul 2019 09:24:15 +0000 (10:24 +0100)
GVT forces single port submission of individual requests. We do not
enjoy the context amalgamation that the test depends upon for setting up
the test (where port 0 has a large number of requests with a priority
change somewhere in the middle). Under single request submission of gvt
it is quite able for the preemption event to occur while another context
is active and so there be a real need to act upon that preemption.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=111108
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
Acked-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190712082549.25053-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/gt/selftest_lrc.c

index 678e9b2edf8dd3fc5142a6561500fcb052d57156..6b6cea76cc8b9814aca974051d6ea8d9871b8228 100644 (file)
@@ -854,6 +854,9 @@ static int live_suppress_self_preempt(void *arg)
        if (USES_GUC_SUBMISSION(i915))
                return 0; /* presume black blox */
 
+       if (intel_vgpu_active(i915))
+               return 0; /* GVT forces single port & request submission */
+
        mutex_lock(&i915->drm.struct_mutex);
        wakeref = intel_runtime_pm_get(&i915->runtime_pm);