net/ice: fix tso tunnel setting to not take effect

  The Tx offload capabilities of ICE ethdev doesn't include
tso tunnel, which will result in tso tunnel setting to
not take effect.

This commit will add tso tunnel capabilities in ice_dev_info_get().

Fixes: 295968d17407 ("ethdev: add namespace")
Cc: stable@dpdk.org

Signed-off-by: Kaiwen Deng <kaiwenx.deng@intel.com>
---
 drivers/net/ice/ice_ethdev.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
  

please add Bugzilla ID

在 2023/11/24 14:44, Kaiwen Deng 写道:
> The Tx offload capabilities of ICE ethdev doesn't include
> tso tunnel, which will result in tso tunnel setting to
> not take effect.
>
> This commit will add tso tunnel capabilities in ice_dev_info_get().
>
> Fixes: 295968d17407 ("ethdev: add namespace")
> Cc: stable@dpdk.org
>
> Signed-off-by: Kaiwen Deng <kaiwenx.deng@intel.com>
> ---
>   drivers/net/ice/ice_ethdev.c | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> index 3ccba4db80..fbc957fcd8 100644
> --- a/drivers/net/ice/ice_ethdev.c
> +++ b/drivers/net/ice/ice_ethdev.c
> @@ -3876,7 +3876,11 @@ ice_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
>   			RTE_ETH_TX_OFFLOAD_TCP_CKSUM |
>   			RTE_ETH_TX_OFFLOAD_SCTP_CKSUM |
>   			RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM |
> -			RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM;
> +			RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM |
> +			RTE_ETH_TX_OFFLOAD_VXLAN_TNL_TSO |
> +			RTE_ETH_TX_OFFLOAD_GRE_TNL_TSO |
> +			RTE_ETH_TX_OFFLOAD_IPIP_TNL_TSO |
> +			RTE_ETH_TX_OFFLOAD_GENEVE_TNL_TSO;
>   		dev_info->flow_type_rss_offloads |= ICE_RSS_OFFLOAD_ALL;
>   	}
>   
correct,
Reviewed-by: Huisong Li <lihuisong@huawei.com>
  

On Fri, Nov 24, 2023 at 8:27 AM Kaiwen Deng <kaiwenx.deng@intel.com> wrote:
>
> The Tx offload capabilities of ICE ethdev doesn't include
> tso tunnel, which will result in tso tunnel setting to
> not take effect.
>
> This commit will add tso tunnel capabilities in ice_dev_info_get().
>
> Fixes: 295968d17407 ("ethdev: add namespace")

This Fixes: tag is wrong.

> Cc: stable@dpdk.org
>
> Signed-off-by: Kaiwen Deng <kaiwenx.deng@intel.com>
> ---
>  drivers/net/ice/ice_ethdev.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> index 3ccba4db80..fbc957fcd8 100644
> --- a/drivers/net/ice/ice_ethdev.c
> +++ b/drivers/net/ice/ice_ethdev.c
> @@ -3876,7 +3876,11 @@ ice_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
>                         RTE_ETH_TX_OFFLOAD_TCP_CKSUM |
>                         RTE_ETH_TX_OFFLOAD_SCTP_CKSUM |
>                         RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM |
> -                       RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM;
> +                       RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM |
> +                       RTE_ETH_TX_OFFLOAD_VXLAN_TNL_TSO |
> +                       RTE_ETH_TX_OFFLOAD_GRE_TNL_TSO |
> +                       RTE_ETH_TX_OFFLOAD_IPIP_TNL_TSO |
> +                       RTE_ETH_TX_OFFLOAD_GENEVE_TNL_TSO;
>                 dev_info->flow_type_rss_offloads |= ICE_RSS_OFFLOAD_ALL;
>         }

Is this tunnel + tso feature supported with vector drivers?

Is there some documentation to update to reflect this addition?