[Intel-gfx] [PATCH i-g-t] lib: Cancel all outstanding requests at the end of a test

Chris Wilson chris at chris-wilson.co.uk
Fri Sep 7 08:41:51 UTC 2018


Quite often on catastrophic failure the test leaves a long queue of
unterminated batches pending execution. Each runs until hangcheck fires
and skips onto the next, leaving us waiting for a very long time at test
exit.

On older kernels, this gracefully degrades into the existing mechanism.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
 lib/drmtest.c     | 25 +++++++++++++++++++------
 lib/igt_debugfs.h | 12 ++++++++++++
 2 files changed, 31 insertions(+), 6 deletions(-)

diff --git a/lib/drmtest.c b/lib/drmtest.c
index bfa2e0f0a..fee9d33ad 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -302,22 +302,35 @@ static int __drm_open_driver_render(int chipset)
 static int at_exit_drm_fd = -1;
 static int at_exit_drm_render_fd = -1;
 
-static void quiescent_gpu_at_exit(int sig)
+static void __cancel_work_at_exit(int fd)
+{
+	igt_terminate_spin_batches(); /* for older kernels */
+
+	igt_drop_caches_set(fd,
+			    /* cancel everything */
+			    DROP_RESET_ACTIVE | DROP_RESET_SEQNO |
+			    /* cleanup */
+			    DROP_ACTIVE | DROP_RETIRE | DROP_IDLE | DROP_FREED);
+}
+
+static void cancel_work_at_exit(int sig)
 {
 	if (at_exit_drm_fd < 0)
 		return;
 
-	gem_quiescent_gpu(at_exit_drm_fd);
+	__cancel_work_at_exit(at_exit_drm_fd);
+
 	close(at_exit_drm_fd);
 	at_exit_drm_fd = -1;
 }
 
-static void quiescent_gpu_at_exit_render(int sig)
+static void cancel_work_at_exit_render(int sig)
 {
 	if (at_exit_drm_render_fd < 0)
 		return;
 
-	gem_quiescent_gpu(at_exit_drm_render_fd);
+	__cancel_work_at_exit(at_exit_drm_render_fd);
+
 	close(at_exit_drm_render_fd);
 	at_exit_drm_render_fd = -1;
 }
@@ -369,7 +382,7 @@ int drm_open_driver(int chipset)
 			gem_quiescent_gpu(fd);
 
 			at_exit_drm_fd = __drm_open_driver(chipset);
-			igt_install_exit_handler(quiescent_gpu_at_exit);
+			igt_install_exit_handler(cancel_work_at_exit);
 		}
 	}
 
@@ -418,7 +431,7 @@ int drm_open_driver_render(int chipset)
 	at_exit_drm_render_fd = __drm_open_driver(chipset);
 	if(chipset & DRIVER_INTEL){
 		gem_quiescent_gpu(fd);
-		igt_install_exit_handler(quiescent_gpu_at_exit_render);
+		igt_install_exit_handler(cancel_work_at_exit_render);
 	}
 
 	return fd;
diff --git a/lib/igt_debugfs.h b/lib/igt_debugfs.h
index ff8612dc6..9f81be0a2 100644
--- a/lib/igt_debugfs.h
+++ b/lib/igt_debugfs.h
@@ -190,6 +190,18 @@ void igt_require_hpd_storm_ctl(int fd);
  * Flush the driver's idle_worker, releasing internal caches and wakerefs.
  */
 #define DROP_IDLE 0x40
+/**
+ * DROP_RESET_ACTIVE:
+ *
+ * Cancel all outstanding requests by forcing a gpu reset
+ */
+#define DROP_RESET_ACTIVE 0x80
+/**
+ * DROP_RESET_SEQNO:
+ *
+ * Reset the global request seqno counter back to 0
+ */
+#define DROP_RESET_SEQNO 0x100
 /**
  * DROP_ALL:
  *
-- 
2.19.0.rc2



More information about the Intel-gfx mailing list