diff mbox

[i-g-t] tests/gem_exec_reuse: Adds cleanup at the end of test.

Message ID 20170828231834.33242-1-antonio.argenziano@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Antonio Argenziano Aug. 28, 2017, 11:18 p.m. UTC
This patch introduces a fixture at the end of the test to destroy
objects that have been created and stop the hang detector.

Cc: Chris Wilson <chris@chris-wilson.co.uk>

Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
---
 tests/gem_exec_reuse.c | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Chris Wilson Aug. 29, 2017, 8:51 a.m. UTC | #1
Quoting Antonio Argenziano (2017-08-29 00:18:34)
> This patch introduces a fixture at the end of the test to destroy
> objects that have been created and stop the hang detector.

We shouldn't have to. Resources are cleaned up by the kernel, and hang
detector is using the fork helper which is meant to make sure all is
well upon exit.
-Chris
diff mbox

Patch

diff --git a/tests/gem_exec_reuse.c b/tests/gem_exec_reuse.c
index cdfa9783..db26e57c 100644
--- a/tests/gem_exec_reuse.c
+++ b/tests/gem_exec_reuse.c
@@ -234,4 +234,11 @@  igt_main
 		for (n = 0; n < ncontexts; n++)
 			gem_context_destroy(no.fd, contexts[n]);
 	}
+
+	igt_fixture {
+		igt_stop_hang_detector();
+
+		gem_close(no.fd, no.batch);
+		close(no.fd);
+	}
 }