AMD-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Sergio Callegari <sergio.callegari@gmail.com>
Cc: "Thorsten Leemhuis" <regressions@leemhuis.info>,
	stable@vger.kernel.org,
	"Mario Limonciello" <mario.limonciello@amd.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Linux kernel regressions list" <regressions@lists.linux.dev>,
	"amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>,
	"Christian König" <christian.koenig@amd.com>,
	"George Zhang" <george.zhang@amd.com>
Subject: Re: [REGRESSION] Please revert 1191285ecb42 from 7.1.y
Date: Tue, 11 Aug 2026 17:08:49 +0900	[thread overview]
Message-ID: <2026081151-reabsorb-calamari-7442@gregkh> (raw)
In-Reply-To: <1caea72e-b9c8-452b-a3ff-5690d948b4fc@gmail.com>

On Tue, Aug 11, 2026 at 09:28:52AM +0200, Sergio Callegari wrote:
> On 11/08/2026 07:24, Thorsten Leemhuis wrote:
> >
> > And yes, 6.18.y seems affected, too, according to a comment in the
> > ticket linked below.
> >
> 
> Yes, my bad I only mentioned 7.1.6 to be quick. My actual concern is that
> the troublesome commit got to the LTS kernels, which I think is the biggest
> issue right now. People may find their display completely black or almost
> black as soon as their DE starts. Getting to the conclusion that they need
> to /reduce/ the brightness to recover is not totally obvious and is not
> applicable at all if you have a system that is touchscreen only (since you
> won't be able to see where to press).
> 
> >> It needs to be reverted in Linus's tree first, otherwise the regression
> >> will be hit when you update to 7.2.
> 
> Indeed, but I think the biggest issue is with the LTS kernels right now.
> 
> > Normally I'd agree, but according to one reporter that problem does not
> > happen with 7.2-rc:
> > https://gitlab.freedesktop.org/drm/amd/-/work_items/5562#note_3603131
> 
> I'll see if I can try myself the 7.2 RCn series. From my superficial
> analysis of the commits that brought us to the current situation, the
> troublesome commit actually tried to patch on a true issue, namely that the
> user facing brightness data is not mapped symmetrically to the hardware
> data, but that the correct solution involves some more thorough rework of
> the brightness coordinate system. So there may well be some larger activity
> on it on upstream.
> >
> > Given all that I wonder if dropping that commit for now might be the
> > best move.
> 
> Personally, I think that dropping on the stable series would be beneficial,
> unless upstream can now offer a correct solution to pick. Behavior you get
> by dropping is not 100% ideal but certainly not as bad as what you get with
> the problematic commit.

We aim to be "bug compatible" with Linus's branch, otherwise the
"pressure" to fix those bugs goes down.  But I defer to the maintainers
of the code involved if it is being worked on, and they want it
reverted, great, we can do that.  Otherwise we prefer to leave it as-is.

thanks,

greg k-h

  reply	other threads:[~2026-08-11 12:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <9c6acb62-4080-48e6-9ba2-d1c457c4ec88@gmail.com>
     [not found] ` <2026081106-pebbly-cahoots-37a7@gregkh>
2026-08-11  5:24   ` [REGRESSION] Please revert 1191285ecb42 from 7.1.y Thorsten Leemhuis
2026-08-11  7:28     ` Sergio Callegari
2026-08-11  8:08       ` Greg KH [this message]
2026-08-11 12:26         ` Sergio Callegari
2026-08-11 17:25           ` Alex Deucher
2026-08-12  0:12             ` Greg KH
2026-08-12 14:49               ` Alex Deucher
2026-08-12 22:21             ` Sasha Levin

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=2026081151-reabsorb-calamari-7442@gregkh \
    --to=gregkh@linuxfoundation.org \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=christian.koenig@amd.com \
    --cc=george.zhang@amd.com \
    --cc=mario.limonciello@amd.com \
    --cc=regressions@leemhuis.info \
    --cc=regressions@lists.linux.dev \
    --cc=sergio.callegari@gmail.com \
    --cc=stable@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