selftests/bpf: Optionally open a dedicated namespace to run test in it
authorBastien Curutchet (eBPF Foundation) <bastien.curutchet@bootlin.com>
Wed, 19 Feb 2025 14:53:01 +0000 (15:53 +0100)
committerAlexei Starovoitov <ast@kernel.org>
Wed, 19 Feb 2025 17:46:02 +0000 (09:46 -0800)
Some tests are serialized to prevent interference with others.

Open a dedicated network namespace when a test name starts with 'ns_' to
allow more test parallelization. Use the test name as namespace name to
avoid conflict between namespaces.

Signed-off-by: Bastien Curutchet (eBPF Foundation) <bastien.curutchet@bootlin.com>
Link: https://lore.kernel.org/r/20250219-b4-tc_links-v2-2-14504db136b7@bootlin.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/test_progs.c

index c9e745d49493e594b55d79ac26061b0466f9a039..0cb759632225d27ad8df7bc478bf1adb5651bf33 100644 (file)
@@ -1365,10 +1365,19 @@ static int recv_message(int sock, struct msg *msg)
        return ret;
 }
 
+static bool ns_is_needed(const char *test_name)
+{
+       if (strlen(test_name) < 3)
+               return false;
+
+       return !strncmp(test_name, "ns_", 3);
+}
+
 static void run_one_test(int test_num)
 {
        struct prog_test_def *test = &prog_test_defs[test_num];
        struct test_state *state = &test_states[test_num];
+       struct netns_obj *ns = NULL;
 
        env.test = test;
        env.test_state = state;
@@ -1376,10 +1385,13 @@ static void run_one_test(int test_num)
        stdio_hijack(&state->log_buf, &state->log_cnt);
 
        watchdog_start();
+       if (ns_is_needed(test->test_name))
+               ns = netns_new(test->test_name, true);
        if (test->run_test)
                test->run_test();
        else if (test->run_serial_test)
                test->run_serial_test();
+       netns_free(ns);
        watchdog_stop();
 
        /* ensure last sub-test is finalized properly */