net/mlx5: add missing ptype for geneve tunnel

Message ID 1573033119-32015-1-git-send-email-wisamm@mellanox.com (mailing list archive)
State Accepted, archived
Delegated to: Raslan Darawsheh
Headers
Series net/mlx5: add missing ptype for geneve tunnel |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-compilation success Compile Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/travis-robot warning Travis build: failed
ci/Intel-compilation fail Compilation issues

Commit Message

Wisam Jaddo Nov. 6, 2019, 9:38 a.m. UTC
  HW ptype are missing TUNNEL_GENEVE support

Fixes: e59a5dbcfd07 ("net/mlx5: add flow match on GENEVE item")
Cc: motih@mellanox.com

Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>
---
 drivers/net/mlx5/mlx5_flow.c | 4 ++++
 drivers/net/mlx5/mlx5_rxtx.h | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)
  

Comments

Slava Ovsiienko Nov. 6, 2019, 5:49 p.m. UTC | #1
> -----Original Message-----
> From: Wisam Monther <wisamm@mellanox.com>
> Sent: Wednesday, November 6, 2019 11:39
> To: Slava Ovsiienko <viacheslavo@mellanox.com>
> Cc: Wisam Monther <wisamm@mellanox.com>; dev@dpdk.org; Moti
> Haimovsky <motih@mellanox.com>
> Subject: [PATCH] net/mlx5: add missing ptype for geneve tunnel
> 
> HW ptype are missing TUNNEL_GENEVE support
> 
> Fixes: e59a5dbcfd07 ("net/mlx5: add flow match on GENEVE item")
> Cc: motih@mellanox.com
> 
> Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
  
Raslan Darawsheh Nov. 7, 2019, 2:28 p.m. UTC | #2
Hi,

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Wisam Monther
> Sent: Wednesday, November 6, 2019 11:39 AM
> To: Slava Ovsiienko <viacheslavo@mellanox.com>
> Cc: Wisam Monther <wisamm@mellanox.com>; dev@dpdk.org; Moti
> Haimovsky <motih@mellanox.com>
> Subject: [dpdk-dev] [PATCH] net/mlx5: add missing ptype for geneve tunnel
> 
> HW ptype are missing TUNNEL_GENEVE support
> 
> Fixes: e59a5dbcfd07 ("net/mlx5: add flow match on GENEVE item")
> Cc: motih@mellanox.com
> 
> Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>


Changed author name to match Signed-off-by tag,

Patch applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh
  

Patch

diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index b4b08f4..8f2dc57 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -287,6 +287,10 @@  static struct mlx5_flow_tunnel_info tunnels_info[] = {
 		.ptype = RTE_PTYPE_TUNNEL_VXLAN | RTE_PTYPE_L4_UDP,
 	},
 	{
+		.tunnel = MLX5_FLOW_LAYER_GENEVE,
+		.ptype = RTE_PTYPE_TUNNEL_GENEVE | RTE_PTYPE_L4_UDP,
+	},
+	{
 		.tunnel = MLX5_FLOW_LAYER_VXLAN_GPE,
 		.ptype = RTE_PTYPE_TUNNEL_VXLAN_GPE | RTE_PTYPE_L4_UDP,
 	},
diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_rxtx.h
index d4ba25f..559d225 100644
--- a/drivers/net/mlx5/mlx5_rxtx.h
+++ b/drivers/net/mlx5/mlx5_rxtx.h
@@ -40,7 +40,7 @@ 
 #include "mlx5_glue.h"
 
 /* Support tunnel matching. */
-#define MLX5_FLOW_TUNNEL 8
+#define MLX5_FLOW_TUNNEL 9
 
 struct mlx5_rxq_stats {
 #ifdef MLX5_PMD_SOFT_COUNTERS