From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Matthieu Baerts <matttbe@kernel.org>,
Theodore Tso <tytso@mit.edu>, Dan Carpenter <error27@gmail.com>,
Konstantin Ryabitsev <mricon@kernel.org>,
Greg KH <greg@kroah.com>, Jonathan Corbet <corbet@lwn.net>,
ksummit@lists.linux.dev
Subject: Re: [MAINTAINERS SUMMIT] Coping with the new-developer flood
Date: Fri, 7 Aug 2026 15:40:17 +0300 [thread overview]
Message-ID: <20260807124017.GK2201016@killaraus.ideasonboard.com> (raw)
In-Reply-To: <04b4e9abab2831b177228752860bedab1f15a736.camel@HansenPartnership.com>
On Fri, Aug 07, 2026 at 08:28:08AM -0400, James Bottomley wrote:
> On Fri, 2026-08-07 at 14:56 +0300, Laurent Pinchart wrote:
> > On Fri, Aug 07, 2026 at 11:20:39AM +0200, Matthieu Baerts wrote:
> > > On 07/08/2026 05:23, Theodore Tso wrote:
> > > > There are some few exceptions, where a fix can't be easily
> > > > backported to the LTS branch, and so a different way of fixing a
> > > > particular serious patch needs to be applied to the LTS branch.
> > > > But normally this will be explicitly stated in the commit
> > > > description, but we don't do that in a structured way, so it
> > > > might be hard for a non-LLM bot to figure this out.
> > >
> > > Good point, but instead of looking at the patch description, it
> > > might be enough to look at the header: an exception could be added
> > > for patches sent to the stable ML with a version in the prefix.
> > >
> > > But back to the idea of checking if the patch can apply: it might
> > > not be easy to deal with all the different trees and branches.
> > > There might be some dependences that are still in review, or
> > > applied somewhere but not in linux-next yet, or conflicts with
> > > linux-next, etc.
> >
> > This is an issue that isn't limited to newcomers. As a maintainer, I
> > sometimes don't know what I should apply a patch series on. We have
> > the base-commit and prerequisite-patch-id trailers added by b4 that
> > help a lot there, but not everybody uses them. If a mechanism that
> > flags the issue for newcomers could also ensure that every patch
> > series carries proper base information, that would be a very nice
> > side effect. This may include adding new trailers if some use cases
> > are not currently covered.
>
> There are two reasons not to insist too rigidly on base commit: firstly
> because a lot of series are invariant (they apply to any tree because
> there's been no commits in the areas patched)
Then the author could pick any base commit available in Linus' tree
the relevant subsystem tree (typically an upstream release of the next
branch of the subsystem). Would that have any drawback compared to not
specifying any base commit at all ?
> and secondly because the
> series has a dependency on another posted but not yet applied patch
> series (i.e. no published base commit at all). At least make it a
> Maintainer decision whether to insist on base commit or not.
Isn't that what prerequisite-patch-id is for ?
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2026-08-07 12:40 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-05 23:26 [MAINTAINERS SUMMIT] Coping with the new-developer flood Jonathan Corbet
2026-08-06 9:24 ` Matthieu Baerts
2026-08-06 10:10 ` Dan Carpenter
2026-08-06 15:42 ` Theodore Tso
2026-08-06 15:57 ` Greg KH
2026-08-06 17:28 ` Konstantin Ryabitsev
2026-08-06 18:56 ` Shuah Khan
2026-08-06 19:32 ` Dan Carpenter
2026-08-06 21:14 ` Mark Brown
2026-08-07 3:23 ` Theodore Tso
2026-08-07 9:20 ` Matthieu Baerts
2026-08-07 11:56 ` Laurent Pinchart
2026-08-07 12:28 ` James Bottomley
2026-08-07 12:40 ` Laurent Pinchart [this message]
2026-08-07 12:47 ` Arnaldo Carvalho de Melo
2026-08-07 13:36 ` Laurent Pinchart
2026-08-07 13:56 ` Rafael J. Wysocki (Intel)
2026-08-07 16:38 ` Miguel Ojeda
2026-08-07 12:51 ` James Bottomley
2026-08-07 13:34 ` Laurent Pinchart
2026-08-07 13:42 ` Mark Brown
2026-08-07 13:02 ` Mimi Zohar
2026-08-07 13:12 ` Dan Carpenter
2026-08-07 16:37 ` Miguel Ojeda
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=20260807124017.GK2201016@killaraus.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=corbet@lwn.net \
--cc=error27@gmail.com \
--cc=greg@kroah.com \
--cc=ksummit@lists.linux.dev \
--cc=matttbe@kernel.org \
--cc=mricon@kernel.org \
--cc=tytso@mit.edu \
/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