selftests/bpf: Drop netns helpers in mptcp

New netns selftest helpers netns_new() and netns_free() has been added
in network_helpers.c, let's use them in mptcp selftests too instead of
using MPTCP's own helpers create_netns() and cleanup_netns().

Signed-off-by: Geliang Tang <tanggeliang@kylinos.cn>
Reviewed-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Link: https://lore.kernel.org/r/c02fda3177b34f9e74a044833fda9761627f4d07.1730338692.git.tanggeliang@kylinos.cn
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>

diff --git a/tools/testing/selftests/bpf/prog_tests/mptcp.c b/tools/testing/selftests/bpf/prog_tests/mptcp.c
index be3cad2aff774f268ddfc83082d5a362c2f57454..f8eb7f9d4fd20bbb7ee018728f7ae0f0a09d4d30 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/mptcp.c
+++ b/tools/testing/selftests/bpf/prog_tests/mptcp.c
@@ -69,24 +69,6 @@ struct mptcp_storage {
        char ca_name[TCP_CA_NAME_MAX];
 };
 
-static struct nstoken *create_netns(void)
-{
-       SYS(fail, "ip netns add %s", NS_TEST);
-       SYS(fail, "ip -net %s link set dev lo up", NS_TEST);
-
-       return open_netns(NS_TEST);
-fail:
-       return NULL;
-}
-
-static void cleanup_netns(struct nstoken *nstoken)
-{
-       if (nstoken)
-               close_netns(nstoken);
-
-       SYS_NOFAIL("ip netns del %s", NS_TEST);
-}
-
 static int start_mptcp_server(int family, const char *addr_str, __u16 port,
                              int timeout_ms)
 {
@@ -206,15 +188,15 @@ out:
 
 static void test_base(void)
 {
-       struct nstoken *nstoken = NULL;
+       struct netns_obj *netns = NULL;
        int server_fd, cgroup_fd;
 
        cgroup_fd = test__join_cgroup("/mptcp");
        if (!ASSERT_GE(cgroup_fd, 0, "test__join_cgroup"))
                return;
 
-       nstoken = create_netns();
-       if (!ASSERT_OK_PTR(nstoken, "create_netns"))
+       netns = netns_new(NS_TEST, true);
+       if (!ASSERT_OK_PTR(netns, "netns_new"))
                goto fail;
 
        /* without MPTCP */
@@ -237,7 +219,7 @@ with_mptcp:
        close(server_fd);
 
 fail:
-       cleanup_netns(nstoken);
+       netns_free(netns);
        close(cgroup_fd);
 }
 
@@ -322,21 +304,21 @@ out:
 
 static void test_mptcpify(void)
 {
-       struct nstoken *nstoken = NULL;
+       struct netns_obj *netns = NULL;
        int cgroup_fd;
 
        cgroup_fd = test__join_cgroup("/mptcpify");
        if (!ASSERT_GE(cgroup_fd, 0, "test__join_cgroup"))
                return;
 
-       nstoken = create_netns();
-       if (!ASSERT_OK_PTR(nstoken, "create_netns"))
+       netns = netns_new(NS_TEST, true);
+       if (!ASSERT_OK_PTR(netns, "netns_new"))
                goto fail;
 
        ASSERT_OK(run_mptcpify(cgroup_fd), "run_mptcpify");
 
 fail:
-       cleanup_netns(nstoken);
+       netns_free(netns);
        close(cgroup_fd);
 }
 
@@ -414,7 +396,7 @@ close_server:
 static void test_subflow(void)
 {
        struct mptcp_subflow *skel;
-       struct nstoken *nstoken;
+       struct netns_obj *netns;
        int cgroup_fd;
 
        cgroup_fd = test__join_cgroup("/mptcp_subflow");
@@ -437,8 +419,8 @@ static void test_subflow(void)
        if (!ASSERT_OK_PTR(skel->links._getsockopt_subflow, "attach _getsockopt_subflow"))
                goto skel_destroy;
 
-       nstoken = create_netns();
-       if (!ASSERT_OK_PTR(nstoken, "create_netns: mptcp_subflow"))
+       netns = netns_new(NS_TEST, true);
+       if (!ASSERT_OK_PTR(netns, "netns_new: mptcp_subflow"))
                goto skel_destroy;
 
        if (endpoint_init("subflow") < 0)
@@ -447,7 +429,7 @@ static void test_subflow(void)
        run_subflow();
 
 close_netns:
-       cleanup_netns(nstoken);
+       netns_free(netns);
 skel_destroy:
        mptcp_subflow__destroy(skel);
 close_cgroup: