diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-10-29 11:29:00 +0100 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-10-29 11:34:32 +0100 |
| commit | 5d24a67c1b885084e0f0d51631ed0c5227bcb112 (patch) | |
| tree | 36a1fbf7a5d55a3726206f3b1d396aaa82ed36bf /lib/drmtest.c | |
| parent | 780807099a482fdcbaea40af7117127b6d92beae (diff) | |
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 <daniel.vetter@ffwll.ch>
Diffstat (limited to 'lib/drmtest.c')
| -rw-r--r-- | lib/drmtest.c | 4 |
1 files 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; } |