Re: [PATCH] net/i40e: support GTP in tunnel parameters parsing

[Bruce Richardson <bruce.richardson@intel.com>]

On Fri, Aug 08, 2025 at 10:25:43AM +0100, Bruce Richardson wrote:
> On Wed, Jul 09, 2025 at 04:32:14PM +0800, Yijun Geng wrote:
> > i40e I40E_TX_OFFLOAD_MASK indicates that all tunnel types are supported,
> > but gtp type is not considered in i40e_parse_tunneling_params(),
> > tunneling parameter: L4TUNLEN is not set. During TX checksum offloading,
> > the incorrect L3 header offset was used when calculating the inner IP
> > checksum,  resulting in modifying the length field of the GTP header.
> > 
> > Signed-off-by: Yijun Geng <1204565380@qq.com>
> > ---
> >  drivers/net/intel/i40e/i40e_rxtx.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/net/intel/i40e/i40e_rxtx.c b/drivers/net/intel/i40e/i40e_rxtx.c
> > index aba3c11ee5..d59d81474e 100644
> > --- a/drivers/net/intel/i40e/i40e_rxtx.c
> > +++ b/drivers/net/intel/i40e/i40e_rxtx.c
> > @@ -274,6 +274,7 @@ i40e_parse_tunneling_params(uint64_t ol_flags,
> >  		break;
> >  	case RTE_MBUF_F_TX_TUNNEL_VXLAN:
> >  	case RTE_MBUF_F_TX_TUNNEL_GENEVE:
> > +	case RTE_MBUF_F_TX_TUNNEL_GTP:
> >  		*cd_tunneling |= I40E_TXD_CTX_UDP_TUNNELING;
> >  		break;
> >  	case RTE_MBUF_F_TX_TUNNEL_GRE:
> > -- 
> 
> Seems ok to me.
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> 
Applied to dpdk-next-net-intel.

Thanks,
/Bruce