Re: [PATCH] docs: submitting-patches: Require base tree information

[Krzysztof Kozlowski]

On 21/10/2026 08:07, Randy Dunlap wrote:
> 
> 
> On 10/20/26 3:07 PM, Konstantin Ryabitsev wrote:
>> On Tue, Oct 20, 2026 at 09:22:35PM +0000, Eric Biggers wrote:
>>> Many patches still don't include base tree information and can't be
>>> applied.  Such patches can't be properly reviewed or tested, nor can
>>> they be accepted.  submitting-patches.rst already describes how to
>>> include base tree information, but it's characterized as optional,
>>> appears only towards the end of the document, and is missing details.
>>
>> I'm happy with this change, but I'd disambiguate the title a bit. With b4, we
>> started providing both a base-commit and base-tree footer, specifically
>> because often times the "base-commit:" line by itself is not sufficient.
>>
>> Imagine this scenario: a developer is working on some code in a feature branch
>> and come across a pre-existing bug. They create a fork off of their feature
>> branch to fix the bug and send it in. In this situation, the "base-commit"
>> footer would be meaningless, because it references some transient commit-id in
>> their feature branch. In an ideal scenario, the developer would rebase that
>> bugfix branch on a known tag, but that doesn't always happen.
>>
>> This observation doesn't really preclude this patch from being accepted, I'm
>> just recording my own experience that, by itself, base-commit is often
>> insufficient.
> 
> Yeah, I was wondering about specifying tree as well as commit, so thanks
> answering before I even asked the question.
> 
> Most of my patches are made against linux-next.  So: is this going to be
> acceptable?
> 
> (for linux-next-20261020:)
> base-commit: 053966c344dbd346e71305f530e91ea77916189f
> base-tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master

According to commit msg which claims that without base information "nor
can they be accepted. " (ridiculous statement on its own, since base is
for these contributions obvious and known), next would not be
acceptable, because maintainers DO NOT work on next. So if base is
necessary for the patch "to be accepted", then base pointing to
something irrelevant to the maintainer does not fit that requirement.

This patch is just harmful to contribution process, just to fix
limitations of bots/testing tools which apparently do not understand we
have something like linux-next.

Nacked-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>

Best regards,
Krzysztof