mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Cc: 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
Subject: Re: [PATCH] docs: submitting-patches: Require base tree information
Date: Wed, 21 Oct 2026 17:49:04 +0200	[thread overview]
Message-ID: <9aa3e170-3d03-456b-ba2d-f329b93ff071@kernel.org> (raw)
In-Reply-To: <20261021-origami-groundhog-of-radiance-78baf9@lemur>

On 21/10/2026 17:04, Konstantin Ryabitsev wrote:
> On Wed, Oct 21, 2026 at 01:55:37PM +0200, Krzysztof Kozlowski wrote:
>> Dependency is something else, but for every other patch:
>> 1. For testing/CI: you apply on top of next
>> 2. Every other case the maintainer knows where to apply it, because it
>> is his HEAD of next-branch.
>>
>> I am repeating myself, but I really had no single problem of lack of
>> knowledge where to apply a patch as a maintainer or reviewer of some
>> subsystem, thus maybe I do not understand the actual problem you are
>> experiencing.
> 
> From my perspective, I don't really want to force contributors to manually
> provide the tree information, but it's helpful if tooling includes it
> automatically.

Sure, I do not oppose this being helpful. I oppose this being a
requirement and that was the impression I got from this patch.

> 
> Specifically, I am working on a service that eager-builds git bundles for all
> submitted patch series -- the goal is to make it simple for CI systems to
> fetch patch submissions as bundles that are already applicable against
> mainline. Knowing which tree and commit the series is based on is really
> helpful when figuring this out.

This has limitations you will not be able to solve. For example, around
SoC code (ARM, ARM64, RISC-V) are often combination of minimum two
subsystems:
1. Driver code + Devicetree bindings, being picked up by driver
maintainer, e.g. drivers/clock/ or drivers/reset/
2. DTS (arch/*/boot/dts) being picked up by SoC platform maintainer who
later sends it to SoC folks (Arnd Bergmann and folks).

Such patchsets by definition target independent trees, thus the only
reasonable base to fully test it or even apply it is the linux-next.
Contributors would not be able to provide base-commit different than
linux-next, thus that information is redundant - you can just assume it
is linux-next.

And if by "tree and commit the series is based on" you meant maintainer
tree, excluding linux-next, then that information could not be even
provided, unless patchset is split.

Best regards,
Krzysztof

  reply	other threads:[~2026-10-21 15:49 UTC|newest]

Thread overview: 16+ 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 [this message]
2026-10-21 14:19   ` Jonathan Corbet
2026-10-21 15:59     ` 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=9aa3e170-3d03-456b-ba2d-f329b93ff071@kernel.org \
    --to=krzk@kernel.org \
    --cc=corbet@lwn.net \
    --cc=ebiggers@kernel.org \
    --cc=konstantin@linuxfoundation.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@infradead.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®