RE: [PATCH v2 net] skbuff.h: Improve the checksum related comments

From: Dexuan Cui
Date: Sun Apr 05 2020 - 22:00:17 EST


> From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Sent: Sunday, April 5, 2020 6:43 PM
> On 4/5/20 6:26 PM, Dexuan Cui wrote:
> > @@ -211,9 +211,9 @@
> > * is implied by the SKB_GSO_* flags in gso_type. Most obviously, if the
> > * gso_type is SKB_GSO_TCPV4 or SKB_GSO_TCPV6, TCP checksum offload
> as
> > * part of the GSO operation is implied. If a checksum is being offloaded
> > - * with GSO then ip_summed is CHECKSUM_PARTIAL, csum_start and
> csum_offset
> > - * are set to refer to the outermost checksum being offload (two offloaded
> > - * checksums are possible with UDP encapsulation).
> > + * with GSO then ip_summed is CHECKSUM_PARTIAL, and both csum_start
> and
> > + * csum_offset are set to refer to the outermost checksum being offload
> (two
>
>
> being offloaded

Thanks for spotting this! :-)

>
> Reviewed-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> --
> ~Randy

Will post a v3 shortly.

Thanks,
-- Dexuan