[igt-dev] [PATCH i-g-t 2/3] i915/gem_eio: Drop redundant "health check"

Chris Wilson chris at chris-wilson.co.uk
Tue May 12 16:20:45 UTC 2020


After triggering the reset, we would ping the engines with a no-op. But
we already validate the reset, so this would seem superfluous.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
 tests/i915/gem_eio.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c
index e0d8390f6..6cec7cd4a 100644
--- a/tests/i915/gem_eio.c
+++ b/tests/i915/gem_eio.c
@@ -72,16 +72,6 @@ static void trigger_reset(int fd)
 
 	/* The forced reset should be immediate */
 	igt_assert_lte(igt_seconds_elapsed(&ts), 2);
-
-	/* And just check the gpu is indeed running again */
-	igt_kmsg(KMSG_DEBUG "Checking that the GPU recovered\n");
-	gem_test_engine(fd, ALL_ENGINES);
-
-	igt_debugfs_dump(fd, "i915_engine_info");
-	igt_drop_caches_set(fd, DROP_ACTIVE);
-
-	/* We expect the health check to be quick! */
-	igt_assert_lte(igt_seconds_elapsed(&ts), 10);
 }
 
 static void manual_hang(int drm_fd)
-- 
2.26.2



More information about the igt-dev mailing list