The XDP redirection is tested without any flag provided to the
xdp_attach() function.

Add two subtests that check the correct behaviour with
XDP_FLAGS_{DRV/SKB}_MODE flags

Signed-off-by: Bastien Curutchet (eBPF Foundation) 
<bastien.curutc...@bootlin.com>
---
 .../selftests/bpf/prog_tests/test_xdp_veth.c       | 27 ++++++++++++++++------
 1 file changed, 20 insertions(+), 7 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 
0aa1583b741b15f573bbb979b2047a0109070544..62a315d01e51739a7f13dddda3d5a569eab24ceb
 100644
--- a/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c
+++ b/tools/testing/selftests/bpf/prog_tests/test_xdp_veth.c
@@ -33,6 +33,7 @@
 #include "xdp_dummy.skel.h"
 #include "xdp_redirect_map.skel.h"
 #include "xdp_tx.skel.h"
+#include <uapi/linux/if_link.h>
 
 #define VETH_PAIRS_COUNT       3
 #define NS_SUFFIX_LEN          6
@@ -185,26 +186,26 @@ static int check_ping(void)
 }
 
 #define VETH_REDIRECT_SKEL_NB  3
-void test_xdp_veth_redirect(void)
+static void xdp_veth_redirect(u32 flags)
 {
        struct prog_configuration ping_config[VETH_PAIRS_COUNT] = {
                {
                        .local_name = "xdp_redirect_map_0",
                        .remote_name = "xdp_dummy_prog",
-                       .local_flags = 0,
-                       .remote_flags = 0,
+                       .local_flags = flags,
+                       .remote_flags = flags,
                },
                {
                        .local_name = "xdp_redirect_map_1",
                        .remote_name = "xdp_tx",
-                       .local_flags = 0,
-                       .remote_flags = 0,
+                       .local_flags = flags,
+                       .remote_flags = flags,
                },
                {
                        .local_name = "xdp_redirect_map_2",
                        .remote_name = "xdp_dummy_prog",
-                       .local_flags = 0,
-                       .remote_flags = 0,
+                       .local_flags = flags,
+                       .remote_flags = flags,
                }
        };
        struct bpf_object *bpf_objs[VETH_REDIRECT_SKEL_NB];
@@ -262,3 +263,15 @@ void test_xdp_veth_redirect(void)
 
        cleanup_network();
 }
+
+void test_xdp_veth_redirect(void)
+{
+       if (test__start_subtest("0"))
+               xdp_veth_redirect(0);
+
+       if (test__start_subtest("DRV_MODE"))
+               xdp_veth_redirect(XDP_FLAGS_DRV_MODE);
+
+       if (test__start_subtest("SKB_MODE"))
+               xdp_veth_redirect(XDP_FLAGS_SKB_MODE);
+}

-- 
2.47.1


Reply via email to