From: Geliang Tang <tanggeli...@kylinos.cn>

Each MPTCP subtest tests test__start_subtest(suffix), then invokes
test_suffix(). It makes sense to add a new macro RUN_MPTCP_TEST to
simpolify the code.

Signed-off-by: Geliang Tang <tanggeli...@kylinos.cn>
Reviewed-by: Mat Martineau <martin...@kernel.org>
Signed-off-by: Matthieu Baerts (NGI0) <matt...@kernel.org>
---
 tools/testing/selftests/bpf/prog_tests/mptcp.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/mptcp.c 
b/tools/testing/selftests/bpf/prog_tests/mptcp.c
index baf976a7a1dd..9d1b255bb654 100644
--- a/tools/testing/selftests/bpf/prog_tests/mptcp.c
+++ b/tools/testing/selftests/bpf/prog_tests/mptcp.c
@@ -347,10 +347,14 @@ static void test_mptcpify(void)
        close(cgroup_fd);
 }
 
+#define RUN_MPTCP_TEST(suffix)                                 \
+do {                                                           \
+       if (test__start_subtest(#suffix))                       \
+               test_##suffix();                                \
+} while (0)
+
 void test_mptcp(void)
 {
-       if (test__start_subtest("base"))
-               test_base();
-       if (test__start_subtest("mptcpify"))
-               test_mptcpify();
+       RUN_MPTCP_TEST(base);
+       RUN_MPTCP_TEST(mptcpify);
 }

-- 
2.43.0


Reply via email to