Re: [igt-dev] [PATCH i-g-t 4/9] tests/gem_exec_reuse: Don't leak the hang detector

Chris Wilson
2019-02-11T18:06:46Z

Quoting Daniel Vetter (2019-02-11 18:02:03)
> My new "are there any child processes left?" check in igt_exit catched
> this one.
>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reuse.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/tests/i915/gem_exec_reuse.c b/tests/i915/gem_exec_reuse.c
> index df220be7bab8..559047188fbd 100644
> --- a/tests/i915/gem_exec_reuse.c
> +++ b/tests/i915/gem_exec_reuse.c
> @@ -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();

Better than my attempt which didn't remember it took void.
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris