Created attachment 301753 [details] dmesg with pci=earlydump (v5.19.5 + ltr debug patch) Split off bug 216373 as this is a different issue than what it was initially about. To quote Bjorn: > The issue you're seeing is an Unsupported Request error logged by a Switch > Downstream Port when it received an LTR message sent by 44:00.0 when the > Switch has LTR disabled: > > pcieport 0000:43:00.0: device [1022:1471] error > status/mask=00100000/00000000 > pcieport 0000:43:00.0: [20] UnsupReq (First) > pcieport 0000:43:00.0: AER: TLP Header: 34000000 44000010 00000000 > 84288428 The errors themselves can be masked by providing pci=noaer to the kernel. The amdgpu.aspm=0 kernel option makes this issue disappear.
Created attachment 301754 [details] lspci -vvnn on vega10 system
Hardware: CPU: Ryzen Threadripper 1950X MB: Asrock X399 Taichi GPU: Radeon Vega 64 [1002:687f]
Created attachment 301760 [details] LTR Fix Does the attached patch help?
Yes, it does. LTR+ is no longer being enabled with this patch.
(In reply to Lijo Lazar from comment #3) > Created attachment 301760 [details] > LTR Fix > > Does the attached patch help? The ASPM code in vi.c and cik.c and si.c should be similarly protected.
(In reply to Alex Deucher from comment #5) > (In reply to Lijo Lazar from comment #3) > > Created attachment 301760 [details] > > LTR Fix > > > > Does the attached patch help? > > The ASPM code in vi.c and cik.c and si.c should be similarly protected. Just checked. Actually, LTR settings are not changed in them.
(In reply to Lijo Lazar from comment #6) > (In reply to Alex Deucher from comment #5) > > (In reply to Lijo Lazar from comment #3) > > > Created attachment 301760 [details] > > > LTR Fix > > > > > > Does the attached patch help? > > > > The ASPM code in vi.c and cik.c and si.c should be similarly protected. > > Just checked. Actually, LTR settings are not changed in them. Yes, but they do setup ASPM which should be protected with CONFIG_PCIEASPM?
Patch: https://patchwork.freedesktop.org/patch/501912/