From: Stephen Hemminger <stephen@networkplumber.org>
To: David Ahern <dsahern@kernel.org>
Cc: roopa@nvidia.com, razor@blackwall.org, bridge@lists.linux.dev,
netdev@vger.kernel.org, liuhangbin@gmail.com,
Tobias Waldekranz <tobias@waldekranz.com>
Subject: Re: [PATCH v3 iproute2 0/4] Multiple Spanning Tree (MST) Support
Date: Sat, 6 Jul 2024 12:56:16 -0700 [thread overview]
Message-ID: <20240706125616.690e7b98@hermes.local> (raw)
In-Reply-To: <547c13c8-c3c3-495e-8ca9-d87156bfe3f5@kernel.org>
On Sat, 6 Jul 2024 09:26:46 -0600
David Ahern <dsahern@kernel.org> wrote:
> On 7/5/24 9:49 PM, Stephen Hemminger wrote:
> > On Fri, 5 Jul 2024 18:53:47 -0600
> > David Ahern <dsahern@kernel.org> wrote:
> >
> >> On 7/5/24 11:31 AM, patchwork-bot+netdevbpf@kernel.org wrote:
> >>> Hello:
> >>>
> >>> This series was applied to iproute2/iproute2.git (main)
> >>> by Stephen Hemminger <stephen@networkplumber.org>:
> >>>
> >>
> >> Why was this merged to the main repro? As a new feature to iproute2 this
> >> should be committed to next and only put in main on the next dev cycle.
> >
> > Because the kernel support was already added, I prefer to not force waiting
> > for code that is non-intrusive and kernel support is already present.
>
> I have told multiple people - with you in CC - that is not how iproute2
> branching works. People need to send userspace patches for iproute2 in
> the same dev cycle as the kernel patches. You are now selectively
> undermining that process. What is the point of -next branch then?
The original point was to have kernel -next and iproute2 -next branches
and have support arrive at same time on both sides. The problem is when
developers get behind, and the iproute2 patches arrive after the kernel cycle
and then would end up get delayed another 3 to 4 months.
Example:
If mst had been submitted during 6.9 -next open window, then
it would have arrived in iproute2 when -next was merged in May 2024 and
would get released concurrently with 6.10 (July 2024).
When MST was submitted later, if it goes through -next, then it would
get merged to main in August 2024 and released concurrently with 6.11
in October. By merging to main, it will be in July.
I understand your concern, and probably better not to have done it.
The problem with accepting things early is the review process gets
truncated, and new features often have lots of feedback.
next prev parent reply other threads:[~2024-07-06 19:56 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-02 12:08 [PATCH v3 iproute2 0/4] Multiple Spanning Tree (MST) Support Tobias Waldekranz
2024-07-02 12:08 ` [PATCH v3 iproute2 1/4] ip: bridge: add support for mst_enabled Tobias Waldekranz
2024-07-02 12:08 ` [PATCH v3 iproute2 2/4] bridge: Remove duplicated textification macros Tobias Waldekranz
2024-07-02 12:08 ` [PATCH v3 iproute2 3/4] bridge: vlan: Add support for setting a VLANs MSTI Tobias Waldekranz
2024-07-02 12:08 ` [PATCH v3 iproute2 4/4] bridge: mst: Add get/set support for MST states Tobias Waldekranz
2024-07-05 17:31 ` [PATCH v3 iproute2 0/4] Multiple Spanning Tree (MST) Support patchwork-bot+netdevbpf
2024-07-06 0:53 ` David Ahern
2024-07-06 3:49 ` Stephen Hemminger
2024-07-06 15:26 ` David Ahern
2024-07-06 19:56 ` Stephen Hemminger [this message]
2024-07-07 16:16 ` David Ahern
2024-07-07 21:16 ` Stephen Hemminger
2024-07-08 22:40 ` patchwork-bot+netdevbpf
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=20240706125616.690e7b98@hermes.local \
--to=stephen@networkplumber.org \
--cc=bridge@lists.linux.dev \
--cc=dsahern@kernel.org \
--cc=liuhangbin@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=razor@blackwall.org \
--cc=roopa@nvidia.com \
--cc=tobias@waldekranz.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).