selftests/bpf: Simplify test_exec in test_sockmap

Signed-off-by: Geliang Tang <tanggeliang@kylinos.cn>
geliangtang · Jan 20, 2025 · 911c00b · 911c00b
1 parent 986e6cc
commit 911c00b
Showing 1 changed file with 3 additions and 12 deletions.
diff --git a/tools/testing/selftests/bpf/test_sockmap.c b/tools/testing/selftests/bpf/test_sockmap.c
@@ -1375,19 +1375,10 @@ static int __test_exec(int cgrp, int test, struct sockmap_options *opt)
 static void test_exec(int cgrp, struct sockmap_options *opt)
 {
 	int type = strcmp(opt->map, BPF_SOCKMAP_FILENAME);
-	int err;
 
-	if (type == 0) {
-		test_start();
-		err = __test_exec(cgrp, SENDMSG, opt);
-		if (err)
-			test_fail();
-	} else {
-		test_start();
-		err = __test_exec(cgrp, SENDPAGE, opt);
-		if (err)
-			test_fail();
-	}
+	test_start();
+	if (__test_exec(cgrp, type ? SENDPAGE : SENDMSG, opt))
+		test_fail();
 }
 
 static void test_send_one(struct sockmap_options *opt, int cgrp)