[igt-dev] [PATCH i-g-t 5/9] tests/i915_missed_irq: Don't leave the hang detector hanging

Daniel Vetter
2019-02-11T18:02:23Z

Spotted by my new "are there any child processes left?" check in
igt_exit - we need to put all the igt_require before we start any real
test logic.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/missed_irq.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/i915/missed_irq.c b/tests/i915/missed_irq.c
index cade3f371401..302da0e8d071 100644
--- a/tests/i915/missed_irq.c
+++ b/tests/i915/missed_irq.c
@@ -108,13 +108,13 @@ igt_simple_main
igt_require_gem(device);
igt_skip_on(gem_has_guc_submission(device)); /* irq forced for guc */
gem_require_mmap_wc(device);
- igt_fork_hang_detector(device);
-
debugfs = igt_debugfs_dir(device);

expect_rings = engine_mask(debugfs);
igt_require(expect_rings);

+ igt_fork_hang_detector(device);
+
igt_debug("Clearing rings %x\n", expect_rings);
intel_detect_and_clear_missed_interrupts(device);
for (e = intel_execution_engines; e->name; e++) {
--
2.20.1