[dpdk-dev] [dpdk-stable] [PATCH v3] net/ixgbe: fix UDP zero checksum error

Wang, Haiyue haiyue.wang at intel.com
Thu Apr 8 03:12:39 CEST 2021


> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Wednesday, April 7, 2021 19:54
> To: Wang, Haiyue <haiyue.wang at intel.com>; Zhang, Qi Z <qi.z.zhang at intel.com>
> Cc: dev <dev at dpdk.org>; Paolo Valerio <pvalerio at redhat.com>; Aaron Conole <aconole at redhat.com>; Rong,
> Leyi <leyi.rong at intel.com>; Tu, Lijuan <lijuan.tu at intel.com>; Ruifeng Wang (Arm Technology China)
> <Ruifeng.Wang at arm.com>; Feifei Wang <Feifei.Wang2 at arm.com>; dpdk stable <stable at dpdk.org>; Guo, Jia
> <jia.guo at intel.com>; Richardson, Bruce <bruce.richardson at intel.com>; Ananyev, Konstantin
> <konstantin.ananyev at intel.com>; Van Haaren, Harry <harry.van.haaren at intel.com>; Thomas Monjalon
> <thomas at monjalon.net>; Yigit, Ferruh <ferruh.yigit at intel.com>
> Subject: Re: [dpdk-stable] [PATCH v3] net/ixgbe: fix UDP zero checksum error
> 
> On Thu, Feb 25, 2021 at 11:54 AM David Marchand
> <david.marchand at redhat.com> wrote:
> >
> > Hello Haiyue,
> >
> > On Thu, Feb 4, 2021 at 3:56 PM Haiyue Wang <haiyue.wang at intel.com> wrote:
> > >
> > > There is an 82599 errata that UDP frames with a zero checksum are
> > > incorrectly marked as checksum invalid by the hardware.  This was
> > > leading to misleading PKT_RX_L4_CKSUM_BAD flag.
> > >
> > > This patch changes the bad UDP checksum to PKT_RX_L4_CKSUM_UNKNOWN,
> > > so the software application will then have to recompute the checksum
> > > itself if needed.
> > >
> > > Bugzilla ID: 629
> > > Fixes: af75078fece3 ("first public release")
> > > Cc: stable at dpdk.org
> > >
> > > Reported-by: Paolo Valerio <pvalerio at redhat.com>
> > > Signed-off-by: Haiyue Wang <haiyue.wang at intel.com>
> >
> > There was a previous mention of this issue in the past and it resulted
> > in dropping part of the hw statistics.
> > https://git.dpdk.org/dpdk/commit/?id=256ff05a9cae7484e2197cde4401dfa1f21d5a6f
> >
> > Does it make sense to restrict this "fix" to 82599 only?

Make sense.

> 
> ping.
> 
> 
> --
> David Marchand



More information about the dev mailing list