[PATCH bpf-next v2 02/10] selftests/bpf: Remove unused argument

From: Bastien Curutchet (eBPF Foundation)
Date: Tue Jan 21 2025 - 08:02:41 EST


check_ping() doesn't use the struct skeletons it has as input.

Remove the unused input.

Signed-off-by: Bastien Curutchet (eBPF Foundation) <bastien.curutchet@xxxxxxxxxxx>
---
tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c b/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c
index 8dc28274a6e8fc75b05781d827a04f01e03a6ebb..8507863e61bbea99c906c60ed4535e23d530588c 100644
--- a/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c
+++ b/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c
@@ -165,7 +165,7 @@ static void cleanup_network(void)
SYS_NOFAIL("ip netns del %s", config[i].namespace);
}

-static int check_ping(struct skeletons *skeletons)
+static int check_ping(void)
{
/* Test: if all interfaces are properly configured, we must be able to ping
* veth33 from veth11
@@ -214,7 +214,7 @@ void test_xdp_veth_redirect(void)
goto destroy_xdp_redirect_map;
}

- ASSERT_OK(check_ping(&skeletons), "ping");
+ ASSERT_OK(check_ping(), "ping");

destroy_xdp_redirect_map:
xdp_redirect_map__destroy(skeletons.xdp_redirect_maps);

--
2.47.1