diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-02-19 08:57:13 +0100 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-02-20 13:58:04 +0100 |
| commit | 1c068c4f11a8f0b12a9b49b54c212315d8f1b347 (patch) | |
| tree | 69d1a4797540fccf70372b10aa435cc9877f3ff9 /lib/tests/igt_fork.c | |
| parent | 4494052aa3c8f95c2b21acc4a1273fc4431bb71e (diff) | |
lib/tests: add internal_assert_wexited/wsignaled
And convert everything over.
igt_segfault needed a bit of care to differentiate between a real
death-by-signal and igt_exit mapping a child process signal death to
an exit code.
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'lib/tests/igt_fork.c')
| -rw-r--r-- | lib/tests/igt_fork.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/tests/igt_fork.c b/lib/tests/igt_fork.c index 38c55d11..10003120 100644 --- a/lib/tests/igt_fork.c +++ b/lib/tests/igt_fork.c @@ -113,21 +113,21 @@ int main(int argc, char **argv) /* check that igt_assert is forwarded */ ret = do_fork(igt_fork_vs_assert); - internal_assert(WEXITSTATUS(ret) == IGT_EXIT_FAILURE); + internal_assert_wexited(ret, IGT_EXIT_FAILURE); /* check that igt_skip within a fork blows up */ ret = do_fork(igt_fork_vs_skip); - internal_assert(WEXITSTATUS(ret) == SIGABRT + 128); + internal_assert_wexited(ret, SIGABRT + 128); /* check that failure to clean up fails */ ret = do_fork(igt_fork_leak); - internal_assert(WTERMSIG(ret) == SIGABRT); + internal_assert_wsignaled(ret, SIGABRT); /* check that igt_waitchildren_timeout cleans up*/ ret = do_fork(igt_fork_timeout_leak); - internal_assert(WEXITSTATUS(ret) == SIGKILL + 128); + internal_assert_wexited(ret, SIGKILL + 128); /* check that any other process leaks are caught*/ ret = do_fork(plain_fork_leak); - internal_assert(WTERMSIG(ret) == SIGABRT); + internal_assert_wsignaled(ret, SIGABRT); } |