All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Linux regression tracking (Thorsten Leemhuis)" <regressions@leemhuis.info>
To: Jarkko Sakkinen <jarkko@kernel.org>,
	Linux regressions mailing list <regressions@lists.linux.dev>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Peter Huewe <peterhuewe@gmx.de>, Jason Gunthorpe <jgg@ziepe.ca>,
	Dominik Brodowski <linux@dominikbrodowski.net>,
	Herbert Xu <herbert@gondor.apana.org.au>,
	stable@vger.kernel.org,
	James Bottomley <James.Bottomley@hansenpartnership.com>,
	reach622@mailcuk.com, Bell <1138267643@qq.com>,
	"Jason A . Donenfeld" <Jason@zx2c4.com>,
	linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org,
	Mario Limonciello <mario.limonciello@amd.com>
Subject: Re: [PATCH v3] tpm: disable hwrng for fTPM on some AMD designs
Date: Sun, 12 Mar 2023 11:35:57 +0100	[thread overview]
Message-ID: <6a916781-7b84-924f-bdc6-0f284610bead@leemhuis.info> (raw)
In-Reply-To: <ZA0sScO47IMKPhtG@kernel.org>

On 12.03.23 02:35, Jarkko Sakkinen wrote:
> On Fri, Mar 10, 2023 at 06:43:47PM +0100, Thorsten Leemhuis wrote:
>> [adding Linux to the list of recipients]
>>
>> On 08.03.23 10:42, Linux regression tracking (Thorsten Leemhuis) wrote:
>>> Hi, Thorsten here, the Linux kernel's regression tracker. Top-posting
>>> for once, to make this easily accessible to everyone.
>>>
>>> Jarkko, thx for reviewing and picking below fix up. Are you planning to
>>> send this to Linus anytime soon, now that the patch was a few days in
>>> next? It would be good to get this 6.1 regression finally fixed, it
>>> already took way longer then the time frame
>>> Documentation/process/handling-regressions.rst outlines for a case like
>>> this. But well, that's how it is sometimes...
>>
>> Linus, would you consider picking this fix up directly from here or from
>> linux-next (8699d5244e37)? It's been in the latter for 9 days now
>> afaics. And the issue seems to bug more than just one or two users, so
>> it IMHO would be good to get this finally resolved.
>>
>> Jarkko didn't reply to my inquiry, guess something else keeps him busy.
> 
> That's a bit arrogant. You emailed only 4 days ago.

My deepest apologies if that "guess something else keeps him busy"
triggered your response, what I wanted to say is "I don't consider the
lack of a response a problem, that's how it is for all of us sometimes".
Sorry, that might not have been the best way to express that.

If my prodding itself was the cause: well, I think that's what I should
do in this case. That stance developed quickly when I started doing
regression tracking, as I noticed one thing:

Image a regression caused by a commit merged for 5.11-rc1 is reported a
day or two after 5.11-rc7 is released. Imagine further a fix is posted
for review two or three days after 5.11-rc8 is out. From what I noticed
quite a few of those fixes (not all of course) make it to mainline in
time for the release of 5.11. But the picture looked totally different
when the fix was posted for review shortly *after* 5.11 was out, as I
noticed quite a few of those were only mainlined 9 or 10 weeks later for
5.13-rc1 (and only then can be backported to 5.11.y and 5.12.y).

[above was just a hypothetical example with the worst timing to
illustrate the core problem, the timelines are different in case of the
fTPM issue]

From my understanding of things that's not how it should be (unless
there are strong reasons in the individual case). That's why I'm working
against that. Still working on optimizing when/how I ask, as I'm not yet
happy with how I do that.

Don't worry, I use my best judgment in that process; if the fix is
complex and the next merge window is near, I might let it slip – OTOH if
it's something that apparently bugs quite a few people, I prod
developers and maintainers more quickly & often, like I did in this case.

In the end situations like the one outlined above lead me to writing the
section "Prioritize work on fixing regressions" in
Documentation/process/handling-regressions.rst (
https://docs.kernel.org/process/handling-regressions.html ). Greg acked
it; Linus never commented on it, not sure if he looked at it when he
merged that. But I have no idea how developers actually have seen it
and/or take it seriously. But from what I saw it already helped somewhat.

> I'm open to do PR for rc3 with the fix, if it cannot wait to v6.4 pr.

From later in this thread I see that you plan to do that now, thus:

many thx!

Ciao, Thorsten

      parent reply	other threads:[~2023-03-12 10:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-28  2:44 [PATCH v3] tpm: disable hwrng for fTPM on some AMD designs Mario Limonciello
2023-02-28  3:10 ` Jarkko Sakkinen
2023-03-08  9:42   ` Linux regression tracking (Thorsten Leemhuis)
2023-03-10 17:43     ` Thorsten Leemhuis
2023-03-12  1:35       ` Jarkko Sakkinen
2023-03-12  1:42         ` Jarkko Sakkinen
2023-03-12  1:49           ` Jason A. Donenfeld
2023-03-12  1:55             ` Jarkko Sakkinen
2023-03-12  1:57               ` Jarkko Sakkinen
2023-03-12 10:40           ` Linux regression tracking (Thorsten Leemhuis)
2023-03-12 10:35         ` Linux regression tracking (Thorsten Leemhuis) [this message]

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=6a916781-7b84-924f-bdc6-0f284610bead@leemhuis.info \
    --to=regressions@leemhuis.info \
    --cc=1138267643@qq.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=Jason@zx2c4.com \
    --cc=herbert@gondor.apana.org.au \
    --cc=jarkko@kernel.org \
    --cc=jgg@ziepe.ca \
    --cc=linux-integrity@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@dominikbrodowski.net \
    --cc=mario.limonciello@amd.com \
    --cc=peterhuewe@gmx.de \
    --cc=reach622@mailcuk.com \
    --cc=regressions@lists.linux.dev \
    --cc=stable@vger.kernel.org \
    --cc=torvalds@linux-foundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.