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

Acked-by: Stanislav Fomichev <s...@fomichev.me>
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 
b609cff198e44f71815e4b13a4b7c907223403d6..64f72068a0100e05bfd327f8291cfc454bc6f518
 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 VETH_NAME_MAX_LEN      32
@@ -188,26 +189,26 @@ static void cleanup_network(struct veth_configuration 
*net_config)
 }
 
 #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 veth_configuration net_config[VETH_PAIRS_COUNT];
@@ -272,3 +273,15 @@ void test_xdp_veth_redirect(void)
 
        cleanup_network(net_config);
 }
+
+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