Multiple bits might be set in the Uncorrectable Error Status
register. But aer_print_error_source() only report a error of
the lowest bit set in the error status register.
So print strings for all bits unmasked and set.
And check First Error Pointer to mark the error occured first.
This FEP is not valid when the corresponing bit of the Uncorrectable
Error Status register is not set, or unimplemented or undefined.
Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
u16 id;
int severity; /* 0:NONFATAL | 1:FATAL | 2:COR */
int flags;
u16 id;
int severity; /* 0:NONFATAL | 1:FATAL | 2:COR */
int flags;
unsigned int status; /* COR/UNCOR Error Status */
unsigned int mask; /* COR/UNCOR Error Mask */
struct header_log_regs tlp; /* TLP Header */
unsigned int status; /* COR/UNCOR Error Status */
unsigned int mask; /* COR/UNCOR Error Mask */
struct header_log_regs tlp; /* TLP Header */
static int get_device_error_info(struct pci_dev *dev, struct aer_err_info *info)
{
static int get_device_error_info(struct pci_dev *dev, struct aer_err_info *info)
{
info->status = 0;
info->flags &= ~AER_TLP_HEADER_VALID_FLAG;
info->status = 0;
info->flags &= ~AER_TLP_HEADER_VALID_FLAG;
if (!(info->status & ~info->mask))
return AER_UNSUCCESS;
if (!(info->status & ~info->mask))
return AER_UNSUCCESS;
+ /* Get First Error Pointer */
+ pci_read_config_dword(dev, pos + PCI_ERR_CAP, &temp);
+ info->first = PCI_ERR_CAP_FEP(temp);
+
if (info->status & AER_LOG_TLP_MASKS) {
info->flags |= AER_TLP_HEADER_VALID_FLAG;
pci_read_config_dword(dev,
if (info->status & AER_LOG_TLP_MASKS) {
info->flags |= AER_TLP_HEADER_VALID_FLAG;
pci_read_config_dword(dev,
errmsg = aer_uncorrectable_error_string[i];
if (errmsg)
errmsg = aer_uncorrectable_error_string[i];
if (errmsg)
- AER_PR(info, "%s\t:\n", errmsg);
+ AER_PR(info, "%s\t: %s\n", errmsg,
+ info->first == i ? "First" : "");
- AER_PR(info, "Unknown Error Bit %2d \t:\n", i);
-
- break;
+ AER_PR(info, "Unknown Error Bit %2d \t: %s\n",
+ i, info->first == i ? "First" : "");