mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: Thorsten Leemhuis <regressions@leemhuis.info>
Cc: Hans de Goede <hansg@kernel.org>,
	 "platform-driver-x86@vger.kernel.org"
	<platform-driver-x86@vger.kernel.org>,
	 Linux kernel regressions list <regressions@lists.linux.dev>,
	 LKML <linux-kernel@vger.kernel.org>
Subject: Re: Policy: "routed through for-next, ... unless introduced in the most recent cycle"
Date: Tue, 28 Jul 2026 14:39:51 +0300 (EEST)	[thread overview]
Message-ID: <28ffe1e4-bbbd-3945-bb8f-f372d825d0ec@linux.intel.com> (raw)
In-Reply-To: <56f2c589-2133-417d-9bbd-2c0b11b83c87@leemhuis.info>

On Tue, 28 Jul 2026, Thorsten Leemhuis wrote:

> Hi Ilpo! I stumbled about a few messages of yours where you apply a
> commit for later mainlining were you wrote this:
> 
> > FYI [if applicable to your patch], as per Linus' policy change, also
> > fixes are mostly routed through for-next unless the fix is for a
> > commit introduced in the most recent cycle or is clearly a regression
> > fix.
> 
> Could you point me to the source of "per Linus' policy change"? I really
> wonder if I missed something, as from what I know Linus wants fixes for
> regressions in all recent released merged quickly (e.g. in the current
> cycle), unless very dangerous or so. My knowledge is among others based
> on this statement from Linus:
> https://lore.kernel.org/all/CAHk-=wis_qQy4oDNynNKi5b7Qhosmxtoj1jxo5wmB6SRUwQUBQ@mail.gmail.com/
> 
> To quote:
> 
> """
> And yes, I do consider "regression in an earlier release" to be a
> regression that needs fixing.
> 
> There's obviously a time limit: if that "regression in an earlier
> release" was a year or more ago, and just took forever for people to
> notice, and it had semantic changes that now mean that fixing the
> regression could cause a _new_ regression, then that can cause me to
> go "Oh, now the new semantics are what we have to live with".
> 
> But something like this, where the regression was in the previous
> release and it's just a clear fix with no semantic subtlety, I
> consider to be just a regular regression that should be expedited -
> partly to make it into stable, and partly to avoid having to put the
> fix into _another_ stable kernel..
> """
> 
> Is there something newer that invalidates this?

Hi Thorsten,

What I'm referring to is this policy change:

https://lore.kernel.org/lkml/CAHk-=wjt1NiKOdyAMz_DT7NmZ++SizPOhRSi492ukdTnpDzHQw@mail.gmail.com/


Perhaps it's just misunderstanding of my wording. My note says "unless". 
So to qualify for fixes as per the new policy the fix has to be either 
a) for something recently introduced (merged in this cycle) or b) a 
proved/tested regression fix to "any" version (not some theoretical 
regression somebody found with a tool or AI or reading the code; "any" is 
somewhat subjective when it comes to very old regressions).

The point of my note in that thank you email is not to set any policy but 
to just explain why less patches are these days merged through fixes 
branch. In pdx86 domain this largely relates to certain kind of patches:

- .probe rollback path fixes (a platform driver probe can certainly fail, 
but does it in practice?)
- .remove fixes (well yeah, removing a platform driver is technically 
possible, but why to do that?)
- new HW additions which previously went through fixes branch

Those issues tend to be quite old so they are not very critical and all 
these can easily wait a few weeks more.

I did differentiate based on criticality and complexity of a "fix" towards 
the end of cycle even before Linus sent that note but now I'm just making 
things easier to me and push most stuff through for-next (which leaves me 
less conflicts to deal with as well). That being said, Rafael Wysocki 
happened to add ACPI NULL checks to platform drivers (force match corner 
case) in that very -rc linked above, so perhaps I'm in part of the reason 
why Linus got tired of seeing so many "fixes". :-)


The way I read your email is that your main concern is that according to 
the new policy, only very recent regressions should be addressed through 
fixes, but that's not what my note (nor Linus, AFAICT) is saying.

I hope the link and explanation helped.

-- 
 i.


  reply	other threads:[~2026-07-28 11:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-28  6:10 Thorsten Leemhuis
2026-07-28 11:39 ` Ilpo Järvinen [this message]
2026-07-28 11:59   ` Thorsten Leemhuis

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=28ffe1e4-bbbd-3945-bb8f-f372d825d0ec@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=hansg@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=regressions@leemhuis.info \
    --cc=regressions@lists.linux.dev \
    /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®