net/mlx5: add GENEVE in tunnel offloads cap query

Message ID 1574777165-10517-1-git-send-email-suanmingm@mellanox.com (mailing list archive)
State Accepted, archived
Delegated to: Raslan Darawsheh
Headers
Series net/mlx5: add GENEVE in tunnel offloads cap query |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
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

Commit Message

Suanming Mou Nov. 26, 2019, 2:06 p.m. UTC
  GENEVE is available in tunnel offloads. Add it as the default support
option.

Signed-off-by: Suanming Mou <suanmingm@mellanox.com>
Acked-by: Ori Kam <orika@mellanox.com>
---
 drivers/net/mlx5/mlx5.c     | 4 +++-
 drivers/net/mlx5/mlx5_txq.c | 3 ++-
 2 files changed, 5 insertions(+), 2 deletions(-)
  

Comments

Raslan Darawsheh Nov. 26, 2019, 3:22 p.m. UTC | #1
Hi,

> -----Original Message-----
> From: Suanming Mou <suanmingm@mellanox.com>
> Sent: Tuesday, November 26, 2019 4:06 PM
> To: Slava Ovsiienko <viacheslavo@mellanox.com>; Matan Azrad
> <matan@mellanox.com>
> Cc: Ori Kam <orika@mellanox.com>; Raslan Darawsheh
> <rasland@mellanox.com>; dev@dpdk.org
> Subject: [PATCH] net/mlx5: add GENEVE in tunnel offloads cap query
> 
> GENEVE is available in tunnel offloads. Add it as the default support option.
> 
> Signed-off-by: Suanming Mou <suanmingm@mellanox.com>
> Acked-by: Ori Kam <orika@mellanox.com>
> ---
>  drivers/net/mlx5/mlx5.c     | 4 +++-
>  drivers/net/mlx5/mlx5_txq.c | 3 ++-
>  2 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index
> 59ae408..949da68 100644
> --- a/drivers/net/mlx5/mlx5.c
> +++ b/drivers/net/mlx5/mlx5.c
> @@ -2256,7 +2256,9 @@ struct mlx5_flow_id_pool *
>  		tunnel_en = ((dv_attr.tunnel_offloads_caps &
> 
> MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_VXLAN) &&
>  			     (dv_attr.tunnel_offloads_caps &
> -
> MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GRE));
> +
> MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GRE) &&
> +			     (dv_attr.tunnel_offloads_caps &
> +
> MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GENEVE));
>  	}
>  	DRV_LOG(DEBUG, "tunnel offloading is %ssupported",
>  		tunnel_en ? "" : "not ");
> diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c
> index 8160516..d265f98 100644
> --- a/drivers/net/mlx5/mlx5_txq.c
> +++ b/drivers/net/mlx5/mlx5_txq.c
> @@ -127,7 +127,8 @@
>  			offloads |=
> DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM;
>  		if (config->tso)
>  			offloads |= (DEV_TX_OFFLOAD_VXLAN_TNL_TSO |
> -				     DEV_TX_OFFLOAD_GRE_TNL_TSO);
> +				     DEV_TX_OFFLOAD_GRE_TNL_TSO |
> +				     DEV_TX_OFFLOAD_GENEVE_TNL_TSO);
>  	}
>  	return offloads;
>  }
> --
> 1.8.3.1


Patch applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh
  

Patch

diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c
index 59ae408..949da68 100644
--- a/drivers/net/mlx5/mlx5.c
+++ b/drivers/net/mlx5/mlx5.c
@@ -2256,7 +2256,9 @@  struct mlx5_flow_id_pool *
 		tunnel_en = ((dv_attr.tunnel_offloads_caps &
 			      MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_VXLAN) &&
 			     (dv_attr.tunnel_offloads_caps &
-			      MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GRE));
+			      MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GRE) &&
+			     (dv_attr.tunnel_offloads_caps &
+			      MLX5DV_RAW_PACKET_CAP_TUNNELED_OFFLOAD_GENEVE));
 	}
 	DRV_LOG(DEBUG, "tunnel offloading is %ssupported",
 		tunnel_en ? "" : "not ");
diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c
index 8160516..d265f98 100644
--- a/drivers/net/mlx5/mlx5_txq.c
+++ b/drivers/net/mlx5/mlx5_txq.c
@@ -127,7 +127,8 @@ 
 			offloads |= DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM;
 		if (config->tso)
 			offloads |= (DEV_TX_OFFLOAD_VXLAN_TNL_TSO |
-				     DEV_TX_OFFLOAD_GRE_TNL_TSO);
+				     DEV_TX_OFFLOAD_GRE_TNL_TSO |
+				     DEV_TX_OFFLOAD_GENEVE_TNL_TSO);
 	}
 	return offloads;
 }