diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2020-06-22 20:15:46 -0400 |
|---|---|---|
| committer | Shuah Khan <skhan@linuxfoundation.org> | 2020-07-06 15:57:15 -0600 |
| commit | 5b0b77ac41e8d97260b35bade6eea21b2d74fdca (patch) | |
| tree | e3fbd9ca033c54c294e5131b80c9847d8c19e79e | |
| parent | f000a39c27cc5792f7ac0e8bf2a6cae40de39807 (diff) | |
| download | lwn-5b0b77ac41e8d97260b35bade6eea21b2d74fdca.tar.gz lwn-5b0b77ac41e8d97260b35bade6eea21b2d74fdca.zip | |
selftests: sigaltstack: do not use ksft_exit_skip after ksft_set_plan
Calling ksft_exit_skip after ksft_set_plan results in executing fewer tests
than planned. Use ksft_test_result_skip when possible, or just bail out if
memory corruption is detected.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
| -rw-r--r-- | tools/testing/selftests/sigaltstack/sas.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/sigaltstack/sas.c b/tools/testing/selftests/sigaltstack/sas.c index ad0f8df2ca0a..8934a3766d20 100644 --- a/tools/testing/selftests/sigaltstack/sas.c +++ b/tools/testing/selftests/sigaltstack/sas.c @@ -71,7 +71,7 @@ void my_usr1(int sig, siginfo_t *si, void *u) swapcontext(&sc, &uc); ksft_print_msg("%s\n", p->msg); if (!p->flag) { - ksft_exit_skip("[RUN]\tAborting\n"); + ksft_exit_fail_msg("[RUN]\tAborting\n"); exit(EXIT_FAILURE); } } @@ -144,7 +144,7 @@ int main(void) err = sigaltstack(&stk, NULL); if (err) { if (errno == EINVAL) { - ksft_exit_skip( + ksft_test_result_skip( "[NOTE]\tThe running kernel doesn't support SS_AUTODISARM\n"); /* * If test cases for the !SS_AUTODISARM variant were |