From 5d24a67c1b885084e0f0d51631ed0c5227bcb112 Mon Sep 17 00:00:00 2001 From: Daniel Vetter Date: Tue, 29 Oct 2013 11:29:00 +0100 Subject: [PATCH] lib/drmtest: Scream harder when igt_exit isn't called for subtest tests We really need this since otherwise the magic return value handling for running testcases with piglit (or on QA's validation infrastructure) doesn't work properly. We need to be careful though to only install this check on success. See also the previous commits to sprinkle igt_exit() calls over all the tests that missed it. Signed-off-by: Daniel Vetter --- lib/drmtest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 8164ef93..576b4ab1 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -812,8 +812,6 @@ int igt_subtest_init_parse_opts(int argc, char **argv, case 'l': if (!run_single_subtest) list_subtests = true; - else - igt_install_exit_handler(check_igt_exit); break; case 'r': if (!list_subtests) @@ -842,6 +840,8 @@ int igt_subtest_init_parse_opts(int argc, char **argv, } } + igt_install_exit_handler(check_igt_exit); + out: return ret; }