diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-02-20 23:37:16 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2018-03-01 07:53:58 +0000 |
commit | 5aed726a723d0abd42e36a26dd6349739fefd568 (patch) | |
tree | a3c7c895f547f8c7fd574345c1641aac192e9b80 /tests/pm_rps.c | |
parent | 0d47ec161b4eca1b41c5348604aa05b105e5d1cf (diff) |
lib/dummyload: Avoid assertions in lowlevel spin constructor
__igt_spin_batch_new() may be used inside a background helper which is
competing against the GPU being reset. As such, we cannot even assert
that the spin->handle is busy immediately after submission as it may
have already been reset by another client writing to i915_wedged.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Abdiel Janulgue <abdiel.janulgue@linux.intel.com>
Diffstat (limited to 'tests/pm_rps.c')
-rw-r--r-- | tests/pm_rps.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/pm_rps.c b/tests/pm_rps.c index f71976ed..006d084b 100644 --- a/tests/pm_rps.c +++ b/tests/pm_rps.c @@ -221,6 +221,8 @@ static void load_helper_run(enum load load) return; } + igt_require_gem(drm_fd); + lh.exit = false; lh.load = load; @@ -233,7 +235,7 @@ static void load_helper_run(enum load load) igt_debug("Applying %s load...\n", lh.load ? "high" : "low"); - spin[0] = igt_spin_batch_new(drm_fd, 0, 0, 0); + spin[0] = __igt_spin_batch_new(drm_fd, 0, 0, 0); if (lh.load == HIGH) spin[1] = __igt_spin_batch_new(drm_fd, 0, 0, 0); while (!lh.exit) { |