selftests/bpf: convert bpf tunnel test to encap modes
authorWillem de Bruijn <willemb@google.com>
Fri, 22 Mar 2019 18:33:00 +0000 (14:33 -0400)
committerAlexei Starovoitov <ast@kernel.org>
Fri, 22 Mar 2019 20:52:45 +0000 (13:52 -0700)
Make the tests correctly annotate skbs with tunnel metadata.

This makes the gso tests succeed. Enable them.

Signed-off-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/progs/test_tc_tunnel.c
tools/testing/selftests/bpf/test_tc_tunnel.sh

index 3b79dffb8103755196b05e570aefe89b3e9be836..f541c2de947d2bbacadf9a3a626ce3c097e8ec50 100644 (file)
@@ -70,8 +70,13 @@ static __always_inline int encap_ipv4(struct __sk_buff *skb, bool with_gre)
        if (tcph.dest != __bpf_constant_htons(cfg_port))
                return TC_ACT_OK;
 
-       flags = BPF_F_ADJ_ROOM_FIXED_GSO;
-       olen = with_gre ? sizeof(h_outer) : sizeof(h_outer.ip);
+       flags = BPF_F_ADJ_ROOM_FIXED_GSO | BPF_F_ADJ_ROOM_ENCAP_L3_IPV4;
+       if (with_gre) {
+               flags |= BPF_F_ADJ_ROOM_ENCAP_L4_GRE;
+               olen = sizeof(h_outer);
+       } else {
+               olen = sizeof(h_outer.ip);
+       }
 
        /* add room between mac and network header */
        if (bpf_skb_adjust_room(skb, olen, BPF_ADJ_ROOM_MAC, flags))
@@ -119,8 +124,14 @@ static __always_inline int encap_ipv6(struct __sk_buff *skb, bool with_gre)
        if (tcph.dest != __bpf_constant_htons(cfg_port))
                return TC_ACT_OK;
 
-       flags = BPF_F_ADJ_ROOM_FIXED_GSO;
-       olen = with_gre ? sizeof(h_outer) : sizeof(h_outer.ip);
+       flags = BPF_F_ADJ_ROOM_FIXED_GSO | BPF_F_ADJ_ROOM_ENCAP_L3_IPV6;
+       if (with_gre) {
+               flags |= BPF_F_ADJ_ROOM_ENCAP_L4_GRE;
+               olen = sizeof(h_outer);
+       } else {
+               olen = sizeof(h_outer.ip);
+       }
+
 
        /* add room between mac and network header */
        if (bpf_skb_adjust_room(skb, olen, BPF_ADJ_ROOM_MAC, flags))
index cda5317790d241507c21b6e87a7c9792d3f32b63..dcf3206269314aab3ff5dee91ecf8787e33339b8 100755 (executable)
@@ -97,13 +97,11 @@ if [[ "$#" -eq "0" ]]; then
        echo "ip6 gre"
        $0 ipv6 ip6gre 100
 
-       # disabled until passes SKB_GSO_DODGY checks
-       # echo "ip gre gso"
-       # $0 ipv4 gre 2000
+       echo "ip gre gso"
+       $0 ipv4 gre 2000
 
-       # disabled until passes SKB_GSO_DODGY checks
-       # echo "ip6 gre gso"
-       # $0 ipv6 ip6gre 2000
+       echo "ip6 gre gso"
+       $0 ipv6 ip6gre 2000
 
        echo "OK. All tests passed"
        exit 0