On Thu, Apr 03, 2025 at 01:14:25PM +0530, Raag Jadav wrote: > If an error is triggered while system suspend is in progress, any bus > level power state transition will result in unpredictable error handling. > Mark skip_bus_pm flag as true to avoid this. > > Signed-off-by: Raag Jadav <raag.jadav@xxxxxxxxx> > --- > > Ideally we'd want to defer recovery until system resume, but this is > good enough to prevent device suspend. > > More discussion at [1]. > [1] https://lore.kernel.org/r/Z-38rPeN_j7YGiEl@xxxxxxxxxxxxxxxxxx I just realized I messed up the link, please use [2] instead. Apologies for the inconvenience. [2] https://lore.kernel.org/all/CAJZ5v0g-aJXfVH+Uc=9eRPuW08t-6PwzdyMXsC6FZRKYJtY03Q@xxxxxxxxxxxxxx > drivers/pci/pcie/aer.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c > index 508474e17183..5acf4efc2df3 100644 > --- a/drivers/pci/pcie/aer.c > +++ b/drivers/pci/pcie/aer.c > @@ -1108,6 +1108,12 @@ static void pci_aer_handle_error(struct pci_dev *dev, struct aer_err_info *info) > > static void handle_error_source(struct pci_dev *dev, struct aer_err_info *info) > { > + /* > + * Avoid any power state transition if an error is triggered during > + * system suspend. > + */ > + dev->skip_bus_pm = true; > + > cxl_rch_handle_error(dev, info); > pci_aer_handle_error(dev, info); > pci_dev_put(dev); > -- > 2.34.1 >
