Re: [PATCH] net/ice: fix tso tunnel setting to not take effect

[David Marchand <david.marchand@redhat.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?


-- 
David Marchand