RE: [PATCH v4 net-next 1/1] net: update comments for SKB_GSO_TCP_ECN and SKB_GSO_TCP_ACCECN

"Chia-Yu Chang (Nokia)" <[email protected]> Tue, 4 Aug 2026 12:38:10 +0000
Newsgroups org.kernel.vger.linux-doc,dev.linux.lists.virtualization,org.kernel.vger.bpf,org.kernel.vger.linux-kselftest,org.kernel.vger.linux-rdma,org.kernel.vger.netdev
Message-ID <PAXPR07MB798489DFC97547F586618BEEA3D42@PAXPR07MB7984.eurprd07.prod.outlook.com>
> -----Original Message-----
> From: Jakub Kicinski <[email protected]> 
> Sent: Tuesday, August 4, 2026 2:47 AM
> To: Chia-Yu Chang (Nokia) <[email protected]>
> Cc: [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; Koen De Schepper (Nokia) <[email protected]>; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; [email protected]; Parav Pandit <[email protected]>
> Subject: Re: [PATCH v4 net-next 1/1] net: update comments for SKB_GSO_TCP_ECN and SKB_GSO_TCP_ACCECN
> 
> 
> CAUTION: This is an external email. Please be very careful when clicking links or opening attachments. See the URL nok.it/ext for additional information.
> 
> 
> 
> On Tue, 28 Jul 2026 18:10:07 +0200 [email protected]
> wrote:
> > For RX, SKB_GSO_TCP_ECN shall NOT be used
> 
> but it _is_ still used in hns3, what's your plan there?
> Documenting something we adhere to at the outset seems like a red flag.
> 
Hello,

I would think to remove the SKB_GSO_TCP_ECN in hns3.
Because hns3 will break the whole TOS byte based on Jijie's email: https://lore.kernel.org/all/[email protected]/#t

Also, the virtio related changes will be submitted only after the approval from virtio-spec.

Thanks.
Chia-Yu