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 /lib/tests | |
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 'lib/tests')
-rw-r--r-- | lib/tests/igt_fork.c | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/lib/tests/igt_fork.c b/lib/tests/igt_fork.c index 6769e84a..1c10a801 100644 --- a/lib/tests/igt_fork.c +++ b/lib/tests/igt_fork.c @@ -61,6 +61,22 @@ static void igt_fork_vs_assert(void) igt_waitchildren(); } +static void igt_fork_leak(void) +{ + igt_fork(i, 1) { + sleep(10); + } +} + +static void igt_fork_timeout_leak(void) +{ + igt_fork(i, 1) { + sleep(10); + } + + igt_waitchildren_timeout(1, "library test"); +} + static int do_fork(void (*test_to_run)(void)) { int pid, status; @@ -95,4 +111,12 @@ int main(int argc, char **argv) /* check that igt_skip within a fork blows up */ ret = do_fork(igt_fork_vs_skip); internal_assert(WEXITSTATUS(ret) == SIGABRT + 128); + + /* check that failure to clean up fails */ + ret = do_fork(igt_fork_leak); + internal_assert(WTERMSIG(ret) == SIGABRT); + + /* check that igt_waitchildren_timeout cleans up*/ + ret = do_fork(igt_fork_timeout_leak); + internal_assert(WEXITSTATUS(ret) == SIGKILL + 128); } |