From: "Deucher, Alexander" <Alexander.Deucher@amd.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Alex Deucher <alexdeucher@gmail.com>,
"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: RE: [PATCH] Revert "drm/amdgpu: Fix NULL dereference in dpm sysfs handlers"
Date: Thu, 15 Oct 2020 14:11:17 +0000 [thread overview]
Message-ID: <MN2PR12MB44881689BB508017AE3174E8F7020@MN2PR12MB4488.namprd12.prod.outlook.com> (raw)
In-Reply-To: <20201015140617.GA4039567@kroah.com>
[AMD Public Use]
> -----Original Message-----
> From: Greg KH <gregkh@linuxfoundation.org>
> Sent: Thursday, October 15, 2020 10:06 AM
> To: Deucher, Alexander <Alexander.Deucher@amd.com>
> Cc: Alex Deucher <alexdeucher@gmail.com>; stable@vger.kernel.org
> Subject: Re: [PATCH] Revert "drm/amdgpu: Fix NULL dereference in dpm
> sysfs handlers"
>
> On Thu, Oct 15, 2020 at 01:18:34PM +0000, Deucher, Alexander wrote:
> > [AMD Public Use]
> >
> > > -----Original Message-----
> > > From: Greg KH <gregkh@linuxfoundation.org>
> > > Sent: Thursday, October 15, 2020 1:15 AM
> > > To: Alex Deucher <alexdeucher@gmail.com>
> > > Cc: stable@vger.kernel.org; Deucher, Alexander
> > > <Alexander.Deucher@amd.com>
> > > Subject: Re: [PATCH] Revert "drm/amdgpu: Fix NULL dereference in dpm
> > > sysfs handlers"
> > >
> > > On Wed, Oct 14, 2020 at 04:28:36PM -0400, Alex Deucher wrote:
> > > > This regressed some working configurations so revert it. Will fix
> > > > this properly for 5.9 and backport then.
> > >
> > > What do you mean "backport then"?
> > >
> > > >
> > > > This reverts commit 38e0c89a19fd13f28d2b4721035160a3e66e270b.
> > > >
> > > > This needs to be applied to 5.9 as well. -next (5.10) has this
> > > > already, but 5.9 missed it.
> > >
> > > What is the real fix for this? Is it in Linus's tree and I can just
> > > backport that fix?
> > >
> >
> > This is no real fix. The revert is the fix. Sorry, I should have clarified that.
>
> Is it also reverted in Linus's tree? If so, what's that commit id?
>
> If not, shouldn't it be?
It is reverted in Linus' tree. I cherry-picked the revert from Linus' tree:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=87004abfbc27261edd15716515d89ab42198b405
Alex
>
> thanks,
>
> greg k-h
next prev parent reply other threads:[~2020-10-15 14:11 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-14 20:28 [PATCH] Revert "drm/amdgpu: Fix NULL dereference in dpm sysfs handlers" Alex Deucher
2020-10-15 5:14 ` Greg KH
2020-10-15 13:18 ` Deucher, Alexander
2020-10-15 14:06 ` Greg KH
2020-10-15 14:11 ` Deucher, Alexander [this message]
2020-10-16 7:43 ` Greg KH
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=MN2PR12MB44881689BB508017AE3174E8F7020@MN2PR12MB4488.namprd12.prod.outlook.com \
--to=alexander.deucher@amd.com \
--cc=alexdeucher@gmail.com \
--cc=gregkh@linuxfoundation.org \
--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;
as well as URLs for NNTP newsgroup(s).