> -----Original Message----- > From: David Marchand <[email protected]> > Sent: Wednesday, February 3, 2021 4:32 PM > To: Zhang, Qi Z <[email protected]> > Cc: Wang, Haiyue <[email protected]>; [email protected]; > [email protected]; [email protected]; Rong, Leyi > <[email protected]>; Tu, Lijuan <[email protected]>; > [email protected]; [email protected]; [email protected]; Guo, Jia > <[email protected]>; Richardson, Bruce <[email protected]>; > Ananyev, Konstantin <[email protected]> > Subject: Re: [PATCH v2] net/ixgbe: fix UDP zero checksum error > > On Wed, Feb 3, 2021 at 9:07 AM Zhang, Qi Z <[email protected]> wrote: > > > -----Original Message----- > > > From: Wang, Haiyue <[email protected]> > > > Sent: Wednesday, February 3, 2021 10:19 AM > > > To: [email protected] > > > Cc: [email protected]; [email protected]; > > > [email protected]; Zhang, Qi Z <[email protected]>; Rong, > > > Leyi <[email protected]>; Tu, Lijuan <[email protected]>; > > > [email protected]; [email protected]; Wang, Haiyue > > > <[email protected]>; [email protected]; Guo, Jia > > > <[email protected]>; Richardson, Bruce <[email protected]>; > > > Ananyev, Konstantin <[email protected]> > > > Subject: [PATCH v2] net/ixgbe: fix UDP zero checksum error > > > > > > 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: [email protected] > > > > > > Reported-by: Paolo Valerio <[email protected]> > > > Signed-off-by: Haiyue Wang <[email protected]> > > > > Applied to dpdk-next-net-intel. > > Why the rush for applying? > The ARM part is not done, Paolo said he would test the patch and I am pretty > sure a review can't hurt. > > > -- > David Marchand
Oops, it should be a mistake, the patch has not been acked and reviewed yet. Thanks for capture, will revert. Qi
