On Tue, Mar 25, 2025 at 8:08 AM Karolina Stolarek
<karolina.stolarek@xxxxxxxxxx> wrote:
> diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c
> index a1cf8c7ef628..ec34bc9b2332 100644
> --- a/drivers/pci/pcie/aer.c
> +++ b/drivers/pci/pcie/aer.c
> @@ -760,47 +760,42 @@ int cper_severity_to_aer(int cper_severity)
> EXPORT_SYMBOL_GPL(cper_severity_to_aer);
> #endif
>
> -void pci_print_aer(struct pci_dev *dev, int aer_severity,
> - struct aer_capability_regs *aer)
> +static void populate_aer_err_info(struct aer_err_info *info, int severity,
> + struct aer_capability_regs *aer_regs)
> {
> [...]
> + if (severity == AER_CORRECTABLE) {
> + info->id = aer_regs->cor_err_source;
> + info->status = aer_regs->cor_status;
> + info->mask = aer_regs->cor_mask;
> + } else {
> + info->id = aer_regs->uncor_err_source;
info->id isn't filled in pci_print_aer(). Is the addition due to
aer_print_error() having a conditional/log for info->id == id? From a
brief look at the code, FW-first (e.g. errors from GHES), I think it
will always be true. However, that doesn't always seem to be the case
for CXL (e.g. when cxl_dev_state.rcd == true).
Disclaimer: not a CXL/GHES expert though.
> +void aer_print_platform_error(struct pci_dev *pdev, int severity,
> + struct aer_capability_regs *aer_regs)
I like the encapsulation.
Reviewed-by: Jon Pan-Doh <pandoh@xxxxxxxxxx>
Thanks,
Jon
