Re: [PATCH v6 2/3] aerdrv: Enhanced AER logging

From: Mauro Carvalho Chehab
Date: Tue Dec 04 2012 - 15:31:36 EST

Em Tue, 4 Dec 2012 20:14:10 +0000
"Ortiz, Lance E" <lance.oritz@xxxxxx> escreveu:

> > > + if (!dev) {
> > > + pr_info("PCI AER Cannot get PCI device
> > %04x:%02x:%02x.%d\n",
> > > + pcie->device_id.segment, pcie->device_id.bus,
> > > + pcie->device_id.slot, pcie->device_id.function);
> >
> > Hmm... please correct if I'm wrong, but an error happened at PCI, and
> > also a
> > kernel bug that prevented it to get the proper PCI device...
> >
> > IMHO, the message here should be stronger, and likely printed via
> > pr_err().
> >
> Mauro,
> I modeled this message after other places in the kernel where this function failed. So I figured it would be safe to be consistent there. I agree though that it should be pr_err(). I can make that change.

I understand. On most cases, this may not be a critical issue.

However, in this particular case, if PCI AER got an error, but the device
is not found when trying to handle it, it can be an indication that
the PCI device has a more serious issue. So, I'm in favor of changing it,
and likely be more verbose at the error message, saying that the device
was not found while trying to report an error condition that happened
there. It could make sense to even send a trace for the daemon to be
aware of the error, on some pci device that vanished likely due to the

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at