> -----Original Message-----
> From: dev <[email protected]> On Behalf Of Thomas Monjalon
> Sent: Friday, July 30, 2021 8:04 PM
> To: Gregory Etelson <[email protected]>
> Cc: [email protected]; Ajit Khaparde <[email protected]>; Andrew
> Rybchenko <[email protected]>; Yigit, Ferruh
> <[email protected]>; [email protected]; Li, Xiaoyun
> <[email protected]>; Tomasz Kulasek <[email protected]>;
> Ananyev, Konstantin <[email protected]>; Olivier Matz
> <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH v4] app/testpmd: fix TX checksum
> calculation for tunnel
>
> 30/07/2021 10:39, Olivier Matz:
> > On Thu, Jul 29, 2021 at 08:01:41PM +0300, Gregory Etelson wrote:
> > > csumonly engine calculates TX checksum of a tunnelled packet for
> > > outer headers only or separately for outer and inner headers. The
> > > calculation method is determined by checksum configuration options.
> > > If TX checksum calculation is separated, the inner headers are
> > > processed before outer headers.
> > >
> > > Inner headers processing sets checksum values to 0 unconditionally.
> > > If TX configuration offloads inner checksums only, outer checksum
> > > calculation in software will read 0 instead of real values and
> > > produce wrong result.
> > >
> > > The patch zeroes inner checksums only before software calculation.
> > >
> > > Fixes: 6b520d54ebfe ("app/testpmd: use Tx preparation in checksum
> > > engine")
> > > Cc: [email protected]
> > >
> > > Signed-off-by: Gregory Etelson <[email protected]>
> >
> > Acked-by: Olivier Matz <[email protected]>
> >
>
> The previous acks were forgotten (it should be added manually in the patch):
>
> Acked-by: Ori Kam <[email protected]>
> Acked-by: Ajit Khaparde <[email protected]>
> Acked-by: Xiaoyun Li <[email protected]>
>
> Applied, thanks.
>
Hi Gregory and All,
When we test checksum_offload related test cases based on dpdk21.08-rc3, we
find that received pkts with bad ip-checksum when send ip-checksum=0 pkts.
We find this patch is the first bad commit id. Could you pls have a quick look?
More detailed refer to https://bugs.dpdk.org/show_bug.cgi?id=768
