diff options
| author | zhidao su <suzhidao@xiaomi.com> | 2026-03-19 08:30:26 +0300 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-03-21 21:35:51 +0300 |
| commit | 818dbedd043e94f270403c795fe7856bfa61e013 (patch) | |
| tree | 10331c8e5aa6d0b5d7afea2d86b5078af98c38e3 | |
| parent | 7e226f036a71c032cd1eb985dd02bc8314e69adf (diff) | |
| download | linux-818dbedd043e94f270403c795fe7856bfa61e013.tar.xz | |
selftests/sched_ext: Return non-zero exit code on test failure
runner.c always returned 0 regardless of test results. The kselftest
framework (tools/testing/selftests/kselftest/runner.sh) invokes the runner
binary and treats a non-zero exit code as a test failure; with the old
code, failed sched_ext tests were silently hidden from the parent harness
even though individual "not ok" TAP lines were emitted.
Return 1 when at least one test failed, 0 when all tests passed or were
skipped.
Signed-off-by: zhidao su <suzhidao@xiaomi.com>
Acked-by: Andrea Righi <arighi@nvidia.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | tools/testing/selftests/sched_ext/runner.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/sched_ext/runner.c b/tools/testing/selftests/sched_ext/runner.c index 37ad56c3eb29..4c68efa1512a 100644 --- a/tools/testing/selftests/sched_ext/runner.c +++ b/tools/testing/selftests/sched_ext/runner.c @@ -217,7 +217,7 @@ int main(int argc, char **argv) printf(" - %s\n", failed_tests[i]); } - return 0; + return failed > 0 ? 1 : 0; } void scx_test_register(struct scx_test *test) |