selftests/bpf: Make res_spin_lock test less verbose

Currently, the res_spin_lock test is too chatty as it constantly prints
the test_run results for each iteration in each thread, so in case
verbose output is requested or things go wrong, it will flood the logs
of CI and other systems with repeated messages that offer no valuable
insight. Reduce this by doing assertions when the condition actually
flips, and proceed to break out and exit the threads. We still assert
to mark the test as failed and print the expected and reported values.

Suggested-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Kumar Kartikeya Dwivedi <memxor@gmail.com>
Link: https://lore.kernel.org/r/20250403220841.66654-1-memxor@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>

diff --git a/tools/testing/selftests/bpf/prog_tests/res_spin_lock.c b/tools/testing/selftests/bpf/prog_tests/res_spin_lock.c
index 115287ba441bfd90ff6786e3851980d0949eb3f4..0703e987df8997680419e46dac429083900baf83 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/res_spin_lock.c
+++ b/tools/testing/selftests/bpf/prog_tests/res_spin_lock.c
@@ -25,8 +25,11 @@ static void *spin_lock_thread(void *arg)
 
        while (!READ_ONCE(skip)) {
                err = bpf_prog_test_run_opts(prog_fd, &topts);
-               ASSERT_OK(err, "test_run");
-               ASSERT_OK(topts.retval, "test_run retval");
+               if (err || topts.retval) {
+                       ASSERT_OK(err, "test_run");
+                       ASSERT_OK(topts.retval, "test_run retval");
+                       break;
+               }
        }
        pthread_exit(arg);
 }