selftests/bpf: Drop cgroup_fd of run_mptcpify

Geliang Tang · matttbe · commit ce0756aad10e · 2025-03-10T05:50:18.000Z
The parameter 'cgroup_fd' of run_mptcpify() is useless, drop it.

Signed-off-by: Geliang Tang &lt;tanggeliang@kylinos.cn&gt;
Reviewed-by: Matthieu Baerts (NGI0) &lt;matttbe@kernel.org&gt;
diff --git a/tools/testing/selftests/bpf/prog_tests/mptcp.c b/tools/testing/selftests/bpf/prog_tests/mptcp.c
@@ -272,7 +272,7 @@ static int verify_mptcpify(int server_fd, int client_fd)
 	return err;
 }
 
-static int run_mptcpify(int cgroup_fd)
+static int run_mptcpify(void)
 {
 	int server_fd, client_fd, err = 0;
 	struct mptcpify *mptcpify_skel;
@@ -325,7 +325,7 @@ static void test_mptcpify(void)
 	if (!ASSERT_OK_PTR(netns, "netns_new"))
 		goto fail;
 
-	ASSERT_OK(run_mptcpify(cgroup_fd), "run_mptcpify");
+	ASSERT_OK(run_mptcpify(), "run_mptcpify");
 
 fail:
 	netns_free(netns);

Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,7 @@ static int verify_mptcpify(int server_fd, int client_fd)`
`272`	`272`	`return err;`
`273`	`273`	`}`
`274`	`274`
`275`		`-static int run_mptcpify(int cgroup_fd)`
	`275`	`+static int run_mptcpify(void)`
`276`	`276`	`{`
`277`	`277`	`int server_fd, client_fd, err = 0;`
`278`	`278`	`struct mptcpify *mptcpify_skel;`
`@@ -325,7 +325,7 @@ static void test_mptcpify(void)`
`325`	`325`	`if (!ASSERT_OK_PTR(netns, "netns_new"))`
`326`	`326`	`goto fail;`
`327`	`327`
`328`		`- ASSERT_OK(run_mptcpify(cgroup_fd), "run_mptcpify");`
	`328`	`+ ASSERT_OK(run_mptcpify(), "run_mptcpify");`
`329`	`329`
`330`	`330`	`fail:`
`331`	`331`	`netns_free(netns);`