selftests/bpf: Drop cgroup_fd of run_mptcpify

Geliang Tang · intel-lab-lkp · commit 07ce1ab9720d · 2025-01-15T15:31:15.000+08:00
The parameter 'cgroup_fd' of run_mptcpify() is useless, drop it.

Signed-off-by: Geliang Tang &lt;tanggeliang@kylinos.cn&gt;
diff --git a/tools/testing/selftests/bpf/prog_tests/mptcp.c b/tools/testing/selftests/bpf/prog_tests/mptcp.c
@@ -283,7 +283,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;
@@ -336,7 +336,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
`@@ -283,7 +283,7 @@ static int verify_mptcpify(int server_fd, int client_fd)`
`283`	`283`	`return err;`
`284`	`284`	`}`
`285`	`285`
`286`		`-static int run_mptcpify(int cgroup_fd)`
	`286`	`+static int run_mptcpify(void)`
`287`	`287`	`{`
`288`	`288`	`int server_fd, client_fd, err = 0;`
`289`	`289`	`struct mptcpify *mptcpify_skel;`
`@@ -336,7 +336,7 @@ static void test_mptcpify(void)`
`336`	`336`	`if (!ASSERT_OK_PTR(netns, "netns_new"))`
`337`	`337`	`goto fail;`
`338`	`338`
`339`		`- ASSERT_OK(run_mptcpify(cgroup_fd), "run_mptcpify");`
	`339`	`+ ASSERT_OK(run_mptcpify(), "run_mptcpify");`
`340`	`340`
`341`	`341`	`fail:`
`342`	`342`	`netns_free(netns);`