selftests: sigaltstack: do not use ksft_exit_skip after ksft_set_plan
authorPaolo Bonzini <pbonzini@redhat.com>
Tue, 23 Jun 2020 00:15:46 +0000 (20:15 -0400)
committerShuah Khan <skhan@linuxfoundation.org>
Mon, 6 Jul 2020 21:57:15 +0000 (15:57 -0600)
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>
tools/testing/selftests/sigaltstack/sas.c

index ad0f8df..8934a37 100644 (file)
@@ -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