selftests: timers: clocksource-switch: fix passing errors from child

The return value from system() is a waitpid-style integer. Do not return
it directly because with the implicit masking in exit() it will always
return 0. Access it with appropriate macros to really pass on errors.

Fixes: 7290ce1423c3 ("selftests/timers: Add clocksource-switch test from timetest suite")
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Acked-by: John Stultz <jstultz@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>

diff --git a/tools/testing/selftests/timers/clocksource-switch.c b/tools/testing/selftests/timers/clocksource-switch.c
index ef8eb3604595eb9a2b56e5e48c61673dc2c9d0fd..b57f0a9be49028d9c1e4baffd02ac2205cc03431 100644 (file)
--- a/tools/testing/selftests/timers/clocksource-switch.c
+++ b/tools/testing/selftests/timers/clocksource-switch.c
@@ -110,10 +110,10 @@ int run_tests(int secs)
 
        sprintf(buf, "./inconsistency-check -t %i", secs);
        ret = system(buf);
-       if (ret)
-               return ret;
+       if (WIFEXITED(ret) && WEXITSTATUS(ret))
+               return WEXITSTATUS(ret);
        ret = system("./nanosleep");
-       return ret;
+       return WIFEXITED(ret) ? WEXITSTATUS(ret) : 0;
 }