diff mbox series

kselftest/arm64: Fail the overall fp-stress test if any test fails

Message ID 20241017-arm64-fp-stress-exit-code-v1-1-f528e53a2321@kernel.org (mailing list archive)
State New
Headers show
Series kselftest/arm64: Fail the overall fp-stress test if any test fails | expand

Commit Message

Mark Brown Oct. 17, 2024, 5:43 p.m. UTC
Currently fp-stress does not report a top level test result if it runs to
completion, it always exits with a return code 0. Use the ksft_finished()
helper to ensure that the exit code for the top level program reports a
failure if any of the individual tests has failed.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 tools/testing/selftests/arm64/fp/fp-stress.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)


---
base-commit: 8e929cb546ee42c9a61d24fae60605e9e3192354
change-id: 20241017-arm64-fp-stress-exit-code-90fe21dc4bc3

Best regards,

Comments

Catalin Marinas Oct. 22, 2024, 11:37 a.m. UTC | #1
On Thu, 17 Oct 2024 18:43:31 +0100, Mark Brown wrote:
> Currently fp-stress does not report a top level test result if it runs to
> completion, it always exits with a return code 0. Use the ksft_finished()
> helper to ensure that the exit code for the top level program reports a
> failure if any of the individual tests has failed.
> 
> 

Applied to arm64 (for-next/kselftest), thanks!

[1/1] kselftest/arm64: Fail the overall fp-stress test if any test fails
      https://git.kernel.org/arm64/c/7a08cb9b4bb9
diff mbox series

Patch

diff --git a/tools/testing/selftests/arm64/fp/fp-stress.c b/tools/testing/selftests/arm64/fp/fp-stress.c
index faac24bdefeb9436e2daf20b7250d0ae25ca23a7..e62c9dbad5010234d70b477cf8c52ba0b312910e 100644
--- a/tools/testing/selftests/arm64/fp/fp-stress.c
+++ b/tools/testing/selftests/arm64/fp/fp-stress.c
@@ -651,7 +651,5 @@  int main(int argc, char **argv)
 
 	drain_output(true);
 
-	ksft_print_cnts();
-
-	return 0;
+	ksft_finished();
 }