Re: [PATCH v7 4/5] PCI/AER: Clear both AER fatal and non-fatal status
From: Jonathan Cameron <hidden>
Date: 2026-01-27 10:40:09
Also in:
linux-pci, lkml
On Sat, 24 Jan 2026 15:45:56 +0800 Shuai Xue [off-list ref] wrote:
The DPC driver clears AER fatal status for the port that reported the
error, but not for the downstream device that deteced the error. The
current recovery code only clears non-fatal AER status, leaving fatal
status bits set in the error device.
Use pci_aer_raw_clear_status() to clear both fatal and non-fatal error
status in the error device, ensuring all AER status bits are properly
cleared after recovery.
Fixes: aa344bc8b727 ("PCI/ERR: Clear AER status only when we control AER")
Cc: stable@vger.kernel.org
Signed-off-by: Shuai Xue <xueshuai@linux.alibaba.com>Shouldn't this be first patch in series to make it easier to backport? Otherwise seems reasonable to me, but others know these flows better than me so hopefully we'll get some more review. Reviewed-by: Jonathan Cameron <redacted>
quoted hunk ↗ jump to hunk
--- drivers/pci/pcie/err.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/pci/pcie/err.c b/drivers/pci/pcie/err.c index 0780ea09478b..5e463efc3d05 100644 --- a/drivers/pci/pcie/err.c +++ b/drivers/pci/pcie/err.c@@ -285,7 +285,7 @@ pci_ers_result_t pcie_do_recovery(struct pci_dev *dev, */ if (host->native_aer || pcie_ports_native) { pcie_clear_device_status(dev); - pci_aer_clear_nonfatal_status(dev); + pci_aer_raw_clear_status(dev); } pci_walk_bridge(bridge, pci_pm_runtime_put, NULL);