app/testpmd: cleanup tunnel protocols parsing

This is a cleanup commit.
It assembles all tunnel outer updates into one function call to avoid
code duplications.
It defines RTE_VXLAN_GPE_DEFAULT_PORT (4790) in accordance with all
other tunnel protocol definitions.
It replaces all numeric values 4789 in their corresponding definition
RTE_VXLAN_GPE_DEFAULT_PORT.
It updates the 'csum parse-tunnel' documentation.

Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
index 6e04d53..761ac5a 100644 (file)
--- a/app/test-pmd/cmdline_flow.c
+++ b/app/test-pmd/cmdline_flow.c
@@ -23,6 +23,7 @@
 #include <cmdline_parse_num.h>
 #include <rte_flow.h>
 #include <rte_hexdump.h>
+#include <rte_vxlan.h>
 
 #include "testpmd.h"
 
@@ -421,7 +422,7 @@ struct vxlan_encap_conf vxlan_encap_conf = {
        .select_tos_ttl = 0,
        .vni = "\x00\x00\x00",
        .udp_src = 0,
-       .udp_dst = RTE_BE16(4789),
+       .udp_dst = RTE_BE16(RTE_VXLAN_DEFAULT_PORT),
        .ipv4_src = RTE_IPV4(127, 0, 0, 1),
        .ipv4_dst = RTE_IPV4(255, 255, 255, 255),
        .ipv6_src = "\x00\x00\x00\x00\x00\x00\x00\x00"

diff --git a/app/test-pmd/csumonly.c b/app/test-pmd/csumonly.c
index a9f33c6..8f2f840 100644 (file)
--- a/app/test-pmd/csumonly.c
+++ b/app/test-pmd/csumonly.c
@@ -63,7 +63,7 @@
 #define _htons(x) (x)
 #endif
 
-uint16_t vxlan_gpe_udp_port = 4790;
+uint16_t vxlan_gpe_udp_port = RTE_VXLAN_GPE_DEFAULT_PORT;
 uint16_t geneve_udp_port = RTE_GENEVE_DEFAULT_PORT;
 
 /* structure that caches offload info for the current packet */
@@ -181,6 +181,17 @@ parse_ethernet(struct rte_ether_hdr *eth_hdr, struct testpmd_offload_info *info)
        }
 }
 
+/* Fill in outer layers length */
+static void
+update_tunnel_outer(struct testpmd_offload_info *info)
+{
+       info->is_tunnel = 1;
+       info->outer_ethertype = info->ethertype;
+       info->outer_l2_len = info->l2_len;
+       info->outer_l3_len = info->l3_len;
+       info->outer_l4_proto = info->l4_proto;
+}
+
 /*
  * Parse a GTP protocol header.
  * No optional fields and next extension header type.
@@ -201,11 +212,7 @@ parse_gtp(struct rte_udp_hdr *udp_hdr,
            udp_hdr->dst_port != _htons(RTE_GTPU_UDP_PORT))
                return;
 
-       info->is_tunnel = 1;
-       info->outer_ethertype = info->ethertype;
-       info->outer_l2_len = info->l2_len;
-       info->outer_l3_len = info->l3_len;
-       info->outer_l4_proto = info->l4_proto;
+       update_tunnel_outer(info);
        info->l2_len = 0;
 
        gtp_hdr = (struct rte_gtp_hdr *)((char *)udp_hdr +
@@ -250,18 +257,15 @@ parse_vxlan(struct rte_udp_hdr *udp_hdr,
 {
        struct rte_ether_hdr *eth_hdr;
 
-       /* check udp destination port, 4789 is the default vxlan port
-        * (rfc7348) or that the rx offload flag is set (i40e only
-        * currently) */
-       if (udp_hdr->dst_port != _htons(4789) &&
+       /* check udp destination port, RTE_VXLAN_DEFAULT_PORT (4789) is the
+        * default vxlan port (rfc7348) or that the rx offload flag is set
+        * (i40e only currently)
+        */
+       if (udp_hdr->dst_port != _htons(RTE_VXLAN_DEFAULT_PORT) &&
                RTE_ETH_IS_TUNNEL_PKT(pkt_type) == 0)
                return;
 
-       info->is_tunnel = 1;
-       info->outer_ethertype = info->ethertype;
-       info->outer_l2_len = info->l2_len;
-       info->outer_l3_len = info->l3_len;
-       info->outer_l4_proto = info->l4_proto;
+       update_tunnel_outer(info);
 
        eth_hdr = (struct rte_ether_hdr *)((char *)udp_hdr +
                sizeof(struct rte_udp_hdr) +
@@ -291,11 +295,7 @@ parse_vxlan_gpe(struct rte_udp_hdr *udp_hdr,
 
        if (!vxlan_gpe_hdr->proto || vxlan_gpe_hdr->proto ==
            RTE_VXLAN_GPE_TYPE_IPV4) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                ipv4_hdr = (struct rte_ipv4_hdr *)((char *)vxlan_gpe_hdr +
                           vxlan_gpe_len);
@@ -305,11 +305,7 @@ parse_vxlan_gpe(struct rte_udp_hdr *udp_hdr,
                info->l2_len = 0;
 
        } else if (vxlan_gpe_hdr->proto == RTE_VXLAN_GPE_TYPE_IPV6) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                ipv6_hdr = (struct rte_ipv6_hdr *)((char *)vxlan_gpe_hdr +
                           vxlan_gpe_len);
@@ -319,11 +315,7 @@ parse_vxlan_gpe(struct rte_udp_hdr *udp_hdr,
                info->l2_len = 0;
 
        } else if (vxlan_gpe_hdr->proto == RTE_VXLAN_GPE_TYPE_ETH) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                eth_hdr = (struct rte_ether_hdr *)((char *)vxlan_gpe_hdr +
                          vxlan_gpe_len);
@@ -335,17 +327,6 @@ parse_vxlan_gpe(struct rte_udp_hdr *udp_hdr,
        info->l2_len += RTE_ETHER_VXLAN_GPE_HLEN;
 }
 
-/* Fill in outer layers length */
-static void
-update_tunnel_outer(struct testpmd_offload_info *info)
-{
-       info->is_tunnel = 1;
-       info->outer_ethertype = info->ethertype;
-       info->outer_l2_len = info->l2_len;
-       info->outer_l3_len = info->l3_len;
-       info->outer_l4_proto = info->l4_proto;
-}
-
 /* Parse a geneve header */
 static void
 parse_geneve(struct rte_udp_hdr *udp_hdr,
@@ -412,11 +393,7 @@ parse_gre(struct simple_gre_hdr *gre_hdr, struct testpmd_offload_info *info)
                gre_len += GRE_EXT_LEN;
 
        if (gre_hdr->proto == _htons(RTE_ETHER_TYPE_IPV4)) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                ipv4_hdr = (struct rte_ipv4_hdr *)((char *)gre_hdr + gre_len);
 
@@ -425,11 +402,7 @@ parse_gre(struct simple_gre_hdr *gre_hdr, struct testpmd_offload_info *info)
                info->l2_len = 0;
 
        } else if (gre_hdr->proto == _htons(RTE_ETHER_TYPE_IPV6)) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                ipv6_hdr = (struct rte_ipv6_hdr *)((char *)gre_hdr + gre_len);
 
@@ -438,11 +411,7 @@ parse_gre(struct simple_gre_hdr *gre_hdr, struct testpmd_offload_info *info)
                info->l2_len = 0;
 
        } else if (gre_hdr->proto == _htons(RTE_ETHER_TYPE_TEB)) {
-               info->is_tunnel = 1;
-               info->outer_ethertype = info->ethertype;
-               info->outer_l2_len = info->l2_len;
-               info->outer_l3_len = info->l3_len;
-               info->outer_l4_proto = info->l4_proto;
+               update_tunnel_outer(info);
 
                eth_hdr = (struct rte_ether_hdr *)((char *)gre_hdr + gre_len);
 

diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 7d9a00f..795c739 100644 (file)
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -1149,11 +1149,11 @@ Where:
 * ``ip|udp|tcp|sctp`` always relate to  the inner layer.
 
 * ``outer-ip`` relates to the outer IP layer (only for IPv4) in the case where the packet is recognized
-  as a tunnel packet by the forwarding engine (vxlan, gre, ipip and geneve are
+  as a tunnel packet by the forwarding engine (geneve, gre, gtp, ipip, vxlan and vxlan-gpe are
   supported). See also the ``csum parse-tunnel`` command.
 
 * ``outer-udp`` relates to the outer UDP layer in the case where the packet is recognized
-  as a tunnel packet by the forwarding engine (vxlan, vxlan-gpe, geneve are
+  as a tunnel packet by the forwarding engine (geneve, gtp, vxlan and vxlan-gpe are
   supported). See also the ``csum parse-tunnel`` command.
 
 .. note::
@@ -1213,7 +1213,7 @@ engine::
    testpmd> csum parse-tunnel (on|off) (tx_port_id)
 
 If enabled, the csum forward engine will try to recognize supported
-tunnel headers (vxlan, gre, ipip, geneve).
+tunnel headers (geneve, gtp, gre, ipip, vxlan, vxlan-gpe).
 
 If disabled, treat tunnel packets as non-tunneled packets (a inner
 header is handled as a packet payload).

diff --git a/lib/librte_net/rte_vxlan.h b/lib/librte_net/rte_vxlan.h
index c23c10c..2ad6061 100644 (file)
--- a/lib/librte_net/rte_vxlan.h
+++ b/lib/librte_net/rte_vxlan.h
@@ -22,6 +22,7 @@ extern "C" {
 
 /** VXLAN default port. */
 #define RTE_VXLAN_DEFAULT_PORT 4789
+#define RTE_VXLAN_GPE_DEFAULT_PORT 4790
 
 /**
  * VXLAN protocol header.