selftests/bpf: Drop cgroup_fd of run_mptcpify

Geliang Tang · matttbe · commit 4fa28fe6035c · 2025-08-01T06:05:36.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
@@ -268,7 +268,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;
@@ -321,7 +321,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
`@@ -268,7 +268,7 @@ static int verify_mptcpify(int server_fd, int client_fd)`
`268`	`268`	`return err;`
`269`	`269`	`}`
`270`	`270`
`271`		`-static int run_mptcpify(int cgroup_fd)`
	`271`	`+static int run_mptcpify(void)`
`272`	`272`	`{`
`273`	`273`	`int server_fd, client_fd, err = 0;`
`274`	`274`	`struct mptcpify *mptcpify_skel;`
`@@ -321,7 +321,7 @@ static void test_mptcpify(void)`
`321`	`321`	`if (!ASSERT_OK_PTR(netns, "netns_new"))`
`322`	`322`	`goto fail;`
`323`	`323`
`324`		`- ASSERT_OK(run_mptcpify(cgroup_fd), "run_mptcpify");`
	`324`	`+ ASSERT_OK(run_mptcpify(), "run_mptcpify");`
`325`	`325`
`326`	`326`	`fail:`
`327`	`327`	`netns_free(netns);`