[igt-dev] [PATCH i-g-t] i915/gem_exec_reuse: stop the hang detector afterwards

Chris Wilson chris at chris-wilson.co.uk
Mon Feb 11 17:52:42 UTC 2019


Take responsibility for the state we create, and in particular remember
to kill our child process (the hang detector) before exiting.

Reported-by: Daniel Vetter <daniel.vetter at ffwll.ch>
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 tests/i915/gem_exec_reuse.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_reuse.c b/tests/i915/gem_exec_reuse.c
index df220be7b..44946528f 100644
--- a/tests/i915/gem_exec_reuse.c
+++ b/tests/i915/gem_exec_reuse.c
@@ -116,7 +116,7 @@ static unsigned int max_nfd(void)
 
 igt_main
 {
-	struct noop no;
+	struct noop no = { .fd = -1 };
 	unsigned engines[16];
 	unsigned nengine;
 	unsigned n;
@@ -213,4 +213,7 @@ igt_main
 		for (n = 0; n < ncontexts; n++)
 			gem_context_destroy(no.fd, contexts[n]);
 	}
+
+	igt_fixture
+		igt_stop_hang_detector(no.fd);
 }
-- 
2.20.1



More information about the igt-dev mailing list