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

Chris Wilson chris at chris-wilson.co.uk
Mon Feb 11 18:06:35 UTC 2019


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 at 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 at chris-wilson.co.uk>
-Chris


More information about the igt-dev mailing list