PCI: output FW warning in pci_read/write_vpd
authorPrarit Bhargava <prarit@redhat.com>
Mon, 17 May 2010 18:25:14 +0000 (14:25 -0400)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 18 May 2010 22:00:25 +0000 (15:00 -0700)
pci_read/write_vpd() can fail due to a timeout.  Usually the command
times out because of firmware issues (incorrect vpd length, etc.) on the
PCI card.  Currently, the timeout occurs silently.

Output a message to the user indicating that they should check with
their vendor for new firmware.

Reviewed-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/access.c

index affb83b42ebb9c2ea8e43fc6bf88970bbfd0d230..531bc697d800910087d8ff2f125a00c03dd4abc9 100644 (file)
@@ -220,8 +220,13 @@ static int pci_vpd_pci22_wait(struct pci_dev *dev)
                        return 0;
                }
 
-               if (time_after(jiffies, timeout))
+               if (time_after(jiffies, timeout)) {
+                       dev_printk(KERN_DEBUG, &dev->dev,
+                                  "vpd r/w failed.  This is likely a firmware "
+                                  "bug on this device.  Contact the card "
+                                  "vendor for a firmware update.");
                        return -ETIMEDOUT;
+               }
                if (fatal_signal_pending(current))
                        return -EINTR;
                if (!cond_resched())