Re: [dpdk-dev] [PATCH v1] net/i40e: fix parse ptype issue for NEON vector

["Zhang, Qi Z" <qi.z.zhang@intel.com>]

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Kathleen Capella
> Sent: Thursday, March 11, 2021 3:51 AM
> To: Feifei Wang <Feifei.Wang2@arm.com>; jerinj@marvell.com; Ruifeng
> Wang <Ruifeng.Wang@arm.com>; Xing, Beilei <beilei.xing@intel.com>; Guo,
> Jia <jia.guo@intel.com>; Richardson, Bruce <bruce.richardson@intel.com>;
> Jianbo Liu <jianbo.liu@linaro.org>
> Cc: dev@dpdk.org; nd <nd@arm.com>; Feifei Wang
> <Feifei.Wang2@arm.com>; stable@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v1] net/i40e: fix parse ptype issue for NEON
> vector
> 
> Tested-by: Kathleen Capella <kathleen.capella@arm.com>
> 
> > -----Original Message-----
> > From: dev <dev-bounces@dpdk.org> On Behalf Of Feifei Wang
> > Sent: Tuesday, March 9, 2021 9:40 PM
> > To: jerinj@marvell.com; Ruifeng Wang <Ruifeng.Wang@arm.com>; Beilei
> > Xing <beilei.xing@intel.com>; Jeff Guo <jia.guo@intel.com>; Bruce
> > Richardson <bruce.richardson@intel.com>; Jianbo Liu
> > <jianbo.liu@linaro.org>
> > Cc: dev@dpdk.org; nd <nd@arm.com>; Feifei Wang
> <Feifei.Wang2@arm.com>;
> > stable@dpdk.org
> > Subject: [dpdk-dev] [PATCH v1] net/i40e: fix parse ptype issue for
> > NEON vector
> >
> > In i40e NEON vector rx path, the packet descs processing is incorrect.
> > This caused wrong packet type been filled in mbuf.
> >
> > To fix this, when shifting the pktlen field to be 16-bit aligned, it
> > only needs to process the high 16bit of the packet descs instead of the high
> 32bit.
> >
> > Test Results:
> > Architecture: arm64
> > NIC: XL710
> > Driver: i40e
> > Package: Ether()/IP()/
> >
> > Without this patch:
> > desc_to_ptype_v: ptype = 7 (error)
> >
> > With this patch:
> > desc_to_ptype_v: ptype = 23 (correct)
> >
> > Fixes: ae0eb310f253 ("net/i40e: implement vector PMD for ARM")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Feifei Wang <feifei.wang2@arm.com>
> > Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>

Applied to dpdk-next-net-intel.

Thanks
Qi