All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Abeni <pabeni@redhat.com>
To: Lynne <dev@lynne.ee>, Florian Westphal <fw@strlen.de>
Cc: Netdev <netdev@vger.kernel.org>, Kuniyu <kuniyu@amazon.com>,
	 Willemdebruijn Kernel <willemdebruijn.kernel@gmail.com>
Subject: Re: Regarding UDP-Lite deprecation and removal
Date: Thu, 21 Mar 2024 11:31:40 +0100	[thread overview]
Message-ID: <e5e430e6c6fd079847cb7547b96c2cab70906abb.camel@redhat.com> (raw)
In-Reply-To: <NtHhf_6--3-9@lynne.ee>

On Mon, 2024-03-18 at 18:58 +0100, Lynne wrote:
> Mar 18, 2024, 14:18 by fw@strlen.de:
> 
> > Lynne <dev@lynne.ee> wrote:
> > 
> > > UDP-Lite was scheduled to be removed in 2025 in commit
> > > be28c14ac8bbe1ff due to a lack of real-world users, and
> > > a long-outstanding security bug being left undiscovered.
> > > 
> > > I would like to open a discussion to perhaps either avoid this,
> > > or delay it, conditionally.
> > > 
> > 
> > Is there any evidence UDP-Lite works in practice?
> > 
> > I am not aware of any HW that will peek into L3/L4 payload to figure out
> > that the 'udplite' payload should be passed up even though it has bad csum.
> > 
> > So, AFAIU L2 FCS/CRC essentially renders entire 'partial csum' premise moot,
> > stack will never receive udplite frames that are damaged.
> > 
> > Did things change?
> > 
> 
> I do somehow get CRC errors past the Ethernet layer on consumer rtl cards,
> by default, with no ethtool changes, so maybe things did change.
> 
> I haven't sacrificed a good cable yet to get a definitive proof.
> The cargo-culted way to be sure is to enable rx-all.

I did not consider the mac-level csum - thanks Florian for bringing
that up.

Yes, you can disable FCS checking on the local host - for some NIC at
least - and AFAIK that is the only way to receive csum corrupted
packets. Delivery packets with bad FCS without F_RXALL set would be a
bug.

And you need to set F_RXALL on all the intermediate hops.

All in all, the use-case looks very thin at best.

In any case, to  step-in to maintain a specific protocol you should
start first contributing. I *guess* a reasonable good start would be
implementing UDP-lite selftests.

Cheers,

Paolo



  reply	other threads:[~2024-03-21 10:31 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-17  0:34 Regarding UDP-Lite deprecation and removal Lynne
2024-03-18 10:36 ` Paolo Abeni
2024-03-18 10:49   ` Denis Kirjanov
2024-03-18 14:10 ` Florian Westphal
2024-03-18 17:58   ` Lynne
2024-03-21 10:31     ` Paolo Abeni [this message]
2024-03-21 15:41     ` Jakub Kicinski
2024-03-21  8:29 ` Lynne
2024-03-21 11:43   ` Denis Kirjanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e5e430e6c6fd079847cb7547b96c2cab70906abb.camel@redhat.com \
    --to=pabeni@redhat.com \
    --cc=dev@lynne.ee \
    --cc=fw@strlen.de \
    --cc=kuniyu@amazon.com \
    --cc=netdev@vger.kernel.org \
    --cc=willemdebruijn.kernel@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.