From: "Theodore Tso" <tytso@mit.edu>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Eric Biggers <ebiggers@kernel.org>,
linux-doc@vger.kernel.org, workflows@vger.kernel.org,
Jonathan Corbet <corbet@lwn.net>,
Shuah Khan <skhan@linuxfoundation.org>,
Randy Dunlap <rdunlap@infradead.org>,
linux-kernel@vger.kernel.org,
Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Subject: Re: [PATCH] docs: submitting-patches: Require base tree information
Date: Tue, 27 Oct 2026 21:14:22 -0400 [thread overview]
Message-ID: <auFH8o9FbxprB40A@mit.edu> (raw)
In-Reply-To: <20261027122346.GA2133989@killaraus.ideasonboard.com>
On Tue, Oct 27, 2026 at 02:23:46PM -0500, Laurent Pinchart wrote:
> > The problem boils down to how to present work that touches multiple
> > target trees, and where some parts depend on other work that is
> > not yet finished. Do you post it as single series, so people get an
> > overview? Do you split it in multiple small series (which may still
> > have interdependencies), so everybody looses context?
>
> There's no one-size-fits-them-all solution to this problem. What I want
> in those cases is information provided by the person posting the patches
> about the dependencies and where to find them. I think there's value in
> specifying that information in a machine-parseable way.
The problem is that "one size fits all" and "machine-parseable way"
are in tension with one another. One approach might be that the
patche series fragments might be sent to different subsystem mailing
lists and reiewers, but the cover letter might include a pointer to a
unified git branch which has all of the patches touching all of the
different pieces.
Another approach might be to post it as a single series, but different
mailing lists and reviewers get cc'ed on the various patches in that
series --- and everyone gets cc'ed on the cover letter which also has
a pointer to the unified git branch.
A lot of this really depends on whether you are asking for reviews, or
whether you are asking people to apply the patch series in their
subsystem trees. In the case where there are a large number of
interdependencies, they probably can't do that via patch series
anyway, and it might be easier just to get relevant ack'ed by and then
send to Linus with an explanation that you've gotten acks from all of
the relevant reviewers.
In truth, this case is actually pretty rare. The more common case is
where there is a single set of changes that multiple patch series
might depend upon. For example, there might be an set of iomap
changes which will go on one branch, and then various file systems
(xfs, ext4, btrfs, etc.) might pull in that common dependency branch
with the iomap changes, and then put their ext4 changes on top of
those.
In that case, what's important is not just the base commit id, but an
explanation of where to find that base commit. Normally that just
goes in the cover letter, in human readable form, because I don't
think we have a lot of automation that tries to deal with that, as a
subsystem maintianer, reading the cover letter is just fine from my
perspective. But if someone wants to propsoe extending the base
commit with a repo and branch or tag where the commit can be found, I
guess we could do that.
- Ted
next prev parent reply other threads:[~2026-10-28 1:15 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-20 21:22 Eric Biggers
2026-10-20 22:07 ` Konstantin Ryabitsev
2026-10-21 6:07 ` Randy Dunlap
2026-10-21 6:35 ` Krzysztof Kozlowski
2026-10-21 6:44 ` Eric Biggers
2026-10-21 6:47 ` Krzysztof Kozlowski
2026-10-21 3:17 ` Weijie Yuan
2026-10-21 6:26 ` Krzysztof Kozlowski
2026-10-21 6:39 ` Eric Biggers
2026-10-21 6:41 ` Krzysztof Kozlowski
2026-10-21 6:49 ` Eric Biggers
2026-10-21 11:55 ` Krzysztof Kozlowski
2026-10-21 15:04 ` Konstantin Ryabitsev
2026-10-21 15:49 ` Krzysztof Kozlowski
2026-10-21 16:37 ` Konstantin Ryabitsev
2026-10-21 16:32 ` Laurent Pinchart
2026-10-22 6:57 ` Krzysztof Kozlowski
2026-10-22 7:25 ` Eric Biggers
2026-10-22 8:29 ` Krzysztof Kozlowski
2026-10-22 10:03 ` Laurent Pinchart
2026-10-26 10:40 ` Laurent Pinchart
2026-10-26 10:48 ` Krzysztof Kozlowski
2026-10-26 11:27 ` Mark Brown
2026-10-27 12:29 ` Laurent Pinchart
2026-10-27 14:31 ` Krzysztof Kozlowski
2026-10-27 19:57 ` Eric Biggers
2026-10-26 14:02 ` Geert Uytterhoeven
2026-10-26 14:53 ` Laurent Pinchart
2026-10-26 15:37 ` Geert Uytterhoeven
2026-10-26 20:19 ` Laurent Pinchart
2026-10-26 20:51 ` Mark Brown
2026-10-27 12:33 ` Laurent Pinchart
2026-10-27 18:57 ` Mark Brown
2026-10-26 20:12 ` Rob Herring
2026-10-27 7:28 ` Krzysztof Kozlowski
2026-10-27 11:55 ` Rob Herring
2026-10-27 8:10 ` Geert Uytterhoeven
2026-10-27 12:23 ` Laurent Pinchart
2026-10-28 1:14 ` Theodore Tso [this message]
2026-10-28 11:57 ` Rob Herring
2026-10-21 14:19 ` Jonathan Corbet
2026-10-21 15:59 ` Krzysztof Kozlowski
2026-10-22 10:04 ` Geert Uytterhoeven
2026-10-22 10:13 ` Krzysztof Kozlowski
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=auFH8o9FbxprB40A@mit.edu \
--to=tytso@mit.edu \
--cc=corbet@lwn.net \
--cc=ebiggers@kernel.org \
--cc=geert@linux-m68k.org \
--cc=konstantin@linuxfoundation.org \
--cc=krzk@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=robh@kernel.org \
--cc=skhan@linuxfoundation.org \
--cc=workflows@vger.kernel.org \
/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
all inboxes | Powered by JetHome®