selftests/powerpc: Return errors from all tests

Fix some tests which weren't returning an error code from main.

Although these tests only ever return success, they can still fail if
they time out and the harness kills them. If that happens they still
return success to the shell, which is incorrect and confuses the higher
level error reporting.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://patch.msgid.link/20241106130453.1741013-4-mpe@ellerman.id.au

diff --git a/tools/testing/selftests/powerpc/signal/sigfuz.c b/tools/testing/selftests/powerpc/signal/sigfuz.c
index 08f9afe3b95c4c524596ce4e4a2deed21b2293e6..c101b1391696ecfbcc5e4aea25a889467bf0b428 100644 (file)
--- a/tools/testing/selftests/powerpc/signal/sigfuz.c
+++ b/tools/testing/selftests/powerpc/signal/sigfuz.c
@@ -321,5 +321,5 @@ int main(int argc, char **argv)
        if (!args)
                args = ARG_COMPLETE;
 
-       test_harness(signal_fuzzer, "signal_fuzzer");
+       return test_harness(signal_fuzzer, "signal_fuzzer");
 }

diff --git a/tools/testing/selftests/powerpc/tm/tm-signal-context-force-tm.c b/tools/testing/selftests/powerpc/tm/tm-signal-context-force-tm.c
index 421cb082f6befcb4b0401bbe2f4944c30b961c00..0a4bc479ae39b067e141f4a55b8c7299f9bda628 100644 (file)
--- a/tools/testing/selftests/powerpc/tm/tm-signal-context-force-tm.c
+++ b/tools/testing/selftests/powerpc/tm/tm-signal-context-force-tm.c
@@ -176,5 +176,5 @@ int tm_signal_context_force_tm(void)
 
 int main(int argc, char **argv)
 {
-       test_harness(tm_signal_context_force_tm, "tm_signal_context_force_tm");
+       return test_harness(tm_signal_context_force_tm, "tm_signal_context_force_tm");
 }

diff --git a/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c b/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
index 06b801906f275cab1afb1c72b74da8b78e913a3e..968864b052ece1daeafce99756377778a10ca6b5 100644 (file)
--- a/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
+++ b/tools/testing/selftests/powerpc/tm/tm-signal-sigreturn-nt.c
@@ -46,6 +46,5 @@ int tm_signal_sigreturn_nt(void)
 
 int main(int argc, char **argv)
 {
-       test_harness(tm_signal_sigreturn_nt, "tm_signal_sigreturn_nt");
+       return test_harness(tm_signal_sigreturn_nt, "tm_signal_sigreturn_nt");
 }
-