netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kuniyuki Iwashima <kuniyu@amazon.com>
To: <pablo@netfilter.org>
Cc: <davem@davemloft.net>, <dsahern@kernel.org>,
	<edumazet@google.com>, <horms@kernel.org>, <jmorris@namei.org>,
	<kadlec@netfilter.org>, <kuba@kernel.org>, <kuni1840@gmail.com>,
	<kuniyu@amazon.com>, <ncardwell@google.com>,
	<netdev@vger.kernel.org>, <pabeni@redhat.com>,
	<paul@paul-moore.com>, <serge@hallyn.com>, <willemb@google.com>
Subject: Re: [PATCH v1 net-next 2/4] net: Retire DCCP.
Date: Tue, 8 Apr 2025 10:31:20 -0700	[thread overview]
Message-ID: <20250408173246.38170-1-kuniyu@amazon.com> (raw)
In-Reply-To: <Z_VQ0KlCRkqYWXa-@calendula>

From: Pablo Neira Ayuso <pablo@netfilter.org>
Date: Tue, 8 Apr 2025 18:37:36 +0200
> Hi Kuniyuki,
> 
> On Mon, Apr 07, 2025 at 04:17:49PM -0700, Kuniyuki Iwashima wrote:
> > DCCP was orphaned in 2021 by commit 054c4610bd05 ("MAINTAINERS: dccp:
> > move Gerrit Renker to CREDITS"), which noted that the last maintainer
> > had been inactive for five years.
> > 
> > In recent years, it has become a playground for syzbot, and most changes
> > to DCCP have been odd bug fixes triggered by syzbot.  Apart from that,
> > the only changes have been driven by treewide or networking API updates
> > or adjustments related to TCP.
> > 
> > Thus, in 2023, we announced we would remove DCCP in 2025 via commit
> > b144fcaf46d4 ("dccp: Print deprecation notice.").
> > 
> > Since then, only one individual has contacted the netdev mailing list. [0]
> > 
> > There is ongoing research for Multipath DCCP.  The repository is hosted
> > on GitHub [1], and development is not taking place through the upstream
> > community.  While the repository is published under the GPLv2 license,
> > the scheduling part remains proprietary, with a LICENSE file [2] stating:
> > 
> >   "This is not Open Source software."
> >
> > The researcher mentioned a plan to address the licensing issue, upstream
> > the patches, and step up as a maintainer, but there has been no further
> > communication since then.
> > 
> > Maintaining DCCP for a decade without any real users has become a burden.
> >
> > Therefore, it's time to remove it.
> > 
> > Removing DCCP will also provide significant benefits to TCP.  It allows
> > us to freely reorganize the layout of struct inet_connection_sock, which
> > is currently shared with DCCP, and optimize it to reduce the number of
> > cachelines accessed in the TCP fast path.
> 
> Netfilter parses skbuffs, in that sense, it is a middlebox. Main
> concern on my side is that it could break rulesets, even for people
> that don't really see dccp traffic ever, ruleset could stop loading.
> 
> I would keep the netfilter bits aside by now, based on your netfilter
> updates, we can internally discuss how to phase out dccp support to
> get aligned with netdev maintainers, as it seems there is a wish to
> reduce debt in this front.

Fair enough.  I'll drop the netfilter stuff in v2.


> 
> Having said, this Netfilter does not rely on any of this socket
> structures, I think it should not be an impediment for your tree
> spring cleanup (IIRC we have no hard dependencies on CONFIG_DCCP).

Yes, actually I split changes for net/dccp, netfilter, and LSM
initially, and then, even without net/dccp, netfilter compiled
successfully if include/linux/dccp.h (dccp_hdr etc) was not removed.

Thanks!

---
pw-bot: cr

  reply	other threads:[~2025-04-08 17:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-07 23:17 [PATCH v1 net-next 0/4] net: Retire DCCP Kuniyuki Iwashima
2025-04-07 23:17 ` [PATCH v1 net-next 1/4] selftest: net: Remove DCCP bits Kuniyuki Iwashima
2025-04-07 23:17 ` [PATCH v1 net-next 2/4] net: Retire DCCP Kuniyuki Iwashima
2025-04-08  1:34   ` Paul Moore
2025-04-08  5:22     ` Christian Göttsche
2025-04-08  8:12       ` Paul Moore
2025-04-08 16:28       ` Casey Schaufler
2025-04-08 16:37   ` Pablo Neira Ayuso
2025-04-08 17:31     ` Kuniyuki Iwashima [this message]
2025-04-07 23:17 ` [PATCH v1 net-next 3/4] net: Unexport shared functions for DCCP Kuniyuki Iwashima
2025-04-08 23:25   ` Jakub Kicinski
2025-04-08 23:53     ` Kuniyuki Iwashima
2025-04-07 23:17 ` [PATCH v1 net-next 4/4] tcp: Rename tcp_or_dccp_get_hashinfo() Kuniyuki Iwashima

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=20250408173246.38170-1-kuniyu@amazon.com \
    --to=kuniyu@amazon.com \
    --cc=davem@davemloft.net \
    --cc=dsahern@kernel.org \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=jmorris@namei.org \
    --cc=kadlec@netfilter.org \
    --cc=kuba@kernel.org \
    --cc=kuni1840@gmail.com \
    --cc=ncardwell@google.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=pablo@netfilter.org \
    --cc=paul@paul-moore.com \
    --cc=serge@hallyn.com \
    --cc=willemb@google.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).