Question related to GSO6 checksum magic

From: Heiner Kallweit <hkallweit1@gmail.com>
To: Eric Dumazet <edumazet@google.com>,
	Alexander Duyck <alexander.h.duyck@linux.intel.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Question related to GSO6 checksum magic
Date: Tue, 11 Feb 2020 20:48:52 +0100	[thread overview]
Message-ID: <29eb3035-1777-8b9a-c744-f2996fc5fae1@gmail.com> (raw)

Few network drivers like Intel e1000e or r8169 have the following in the
GSO6 tx path:

ipv6_hdr(skb)->payload_len = 0;
tcp_hdr(skb)->check = ~csum_ipv6_magic(&ipv6_hdr(skb)->saddr,
				       &ipv6_hdr(skb)->daddr,
				       0, IPPROTO_TCP, 0);
(partially also w/o the payload_len assignment)

This sounds like we should factor it out to a helper.
The code however leaves few questions to me, but I'm not familiar enough
with the net core low-level details to answer them:

- This code is used in a number of drivers, so is it something that
  should be moved to the core? If yes, where would it belong to?

- Is clearing payload_len needed? IOW, can it be a problem if drivers
  miss this?

Thanks, Heiner