selftests/bpf: Skip tc_tunnel subtest if its setup fails

A subtest setup can fail in a wide variety of ways, so make sure not to
run it if an issue occurs during its setup. The return value is
already representing whether the setup succeeds or fails, it is just
about wiring it.

Signed-off-by: Alexis Lothoré (eBPF Foundation) <alexis.lothore@bootlin.com>
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
Link: https://patch.msgid.link/20251031-tc_tunnel_improv-v1-1-0ffe44d27eda@bootlin.com

diff --git a/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c b/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
index cf2e088bfe8efc139fb39788b42b15d4a95d8493..1d8d38e67f8bb131643210db09ac7cfffce7222d 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
+++ b/tools/testing/selftests/bpf/prog_tests/test_tc_tunnel.c
@@ -666,8 +666,8 @@ void test_tc_tunnel(void)
                ret = build_subtest_name(cfg, cfg->name, TEST_NAME_MAX_LEN);
                if (ret < 0 || !test__start_subtest(cfg->name))
                        continue;
-               subtest_setup(skel, cfg);
-               run_test(cfg);
+               if (subtest_setup(skel, cfg) == 0)
+                       run_test(cfg);
                subtest_cleanup(cfg);
        }
        cleanup();