On Thu, Jan 16, 2025 at 6:47 AM Karolina Stolarek <karolina.stolarek@xxxxxxxxxx> wrote: > On 15/01/2025 08:42, Jon Pan-Doh wrote: > > diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c > > index ba40800b5494..4bb0b3840402 100644 > > --- a/drivers/pci/pcie/aer.c > > +++ b/drivers/pci/pcie/aer.c > > @@ -695,7 +695,6 @@ static void __aer_print_error(struct pci_dev *dev, > > pci_printk(level, dev, " [%2d] %-22s%s\n", i, errmsg, > > info->first_error == i ? " (First)" : ""); > > } > > - pci_dev_aer_stats_incr(dev, info); > > } > > With this change, we stop calling pci_dev_aer_stats_incr() in > dpc_process_error(). Is this intended? No, this should be fixed. We can call pci_dev_aer_stats_incr() in dpc_process_error() (similar to aer_process_error). Thanks, Jon
