On Fri, Oct 21, 2022 at 08:14:04AM +0200, Salvatore Bonaccorso wrote:
> Hi,
>
> On Fri, Oct 21, 2022 at 02:29:22AM +0200, Diederik de Haas wrote:
> > On Thursday, 20 October 2022 17:38:56 CEST Alex Deucher wrote:
> > > This reverts commit 9f55f36f749a7608eeef57d7d72991a9bd557341.
> > >
> > > This patch was backported incorrectly when Sasha backported it and
> > > the patch that caused the regression that this patch set fixed
> > > was reverted in commit 412b844143e3 ("Revert "PCI/portdrv: Don't disable AER
> > > reporting in get_port_device_capability()""). This isn't necessary and
> > > causes a regression so drop it.
> > >
> > > Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2216
> > > Cc: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
> > > Cc: Sasha Levin <sashal@xxxxxxxxxx>
> > > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
> > > Cc: <stable@xxxxxxxxxxxxxxx> # 5.10
> > > ---
> >
> > I build a kernel with these 2 patches reverted and can confirm that that fixes
> > the issue on my machine with a Radeon RX Vega 64 GPU.
> > # lspci -nn | grep VGA
> > 0b:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/
> > ATI] Vega 10 XL/XT [Radeon RX Vega 56/64] [1002:687f] (rev c1)
> >
> > So feel free to add
> >
> > Tested-By: Diederik de Haas <didi.debian@xxxxxxxxx>
>
> Note additionally (probably only relevant for Greg while reviewing),
> that the first of the commits which need to be reverted is already
> queued as revert in queue-5.10.
Argh, that caused me to drop both of these from the review queue.
Can someone verify that this really still is needed on the latest
5.10-rc that was just sent out? And if so, please send me whatever is
really needed?
this got way too confusing...
greg k-h
