diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-02-13 11:35:48 +0100 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-02-15 10:36:40 +0100 |
| commit | fee7466ade3e1605451b649ed1ea09bb152854f5 (patch) | |
| tree | 36b26a22dde6a441838cdfeffaca970b183b7644 /tests/i915 | |
| parent | 3e841ea25912de4fb6c7b91da33a6912adb0dfc1 (diff) | |
tests/i915_missed_irq: Don't leave the hang detector hanging
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.
v2: Rebase.
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/i915')
| -rw-r--r-- | tests/i915/missed_irq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/i915/missed_irq.c b/tests/i915/missed_irq.c index cade3f37..302da0e8 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++) { |