drm/i915/selftests: Remove flush_scheduled_work() from live_execlists
authorTvrtko Ursulin <tvrtko.ursulin@intel.com>
Thu, 30 Jun 2022 12:57:16 +0000 (13:57 +0100)
committerTvrtko Ursulin <tvrtko.ursulin@intel.com>
Fri, 23 Sep 2022 10:57:33 +0000 (11:57 +0100)
There are ongoing efforts to remove usages of flush_scheduled_work() from
drivers in order to avoid several cases of potentential problems when
flushing is done from certain contexts.

Remove the call from the live_execlists selftest. Its purpose was to be
thorough and sync with the execlists capture state handling, but that is
not strictly required for the test to function and can be removed.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Reviewed-by: Nirmoy Das <nirmoy.das@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220630125716.50835-1-tvrtko.ursulin@linux.intel.com
drivers/gpu/drm/i915/gt/selftest_execlists.c

index 1e08b24..56b7d5b 100644 (file)
@@ -85,8 +85,6 @@ static int wait_for_reset(struct intel_engine_cs *engine,
                        break;
        } while (time_before(jiffies, timeout));
 
-       flush_scheduled_work();
-
        if (rq->fence.error != -EIO) {
                pr_err("%s: hanging request %llx:%lld not reset\n",
                       engine->name,