Commit 7782c8c4 authored by Stephen Hemminger's avatar Stephen Hemminger Committed by Jeff Garzik

sky2: don't use AER routines

Using PCIE advanced error recovery stuff creates more user problems than it's worth.
The AER stuff depends on MMCONFIG and in many configurations it just doesn't work.
Plus it doesn't add any real functionality to the driver. The sky2
driver handles its own errors fine as is.

This reverts 555382cbSigned-off-by: default avatarStephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
parent b32f40c4
...@@ -31,7 +31,6 @@ ...@@ -31,7 +31,6 @@
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/ethtool.h> #include <linux/ethtool.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/aer.h>
#include <linux/ip.h> #include <linux/ip.h>
#include <net/ip.h> #include <net/ip.h>
#include <linux/tcp.h> #include <linux/tcp.h>
...@@ -2432,26 +2431,15 @@ static void sky2_hw_intr(struct sky2_hw *hw) ...@@ -2432,26 +2431,15 @@ static void sky2_hw_intr(struct sky2_hw *hw)
if (status & Y2_IS_PCI_EXP) { if (status & Y2_IS_PCI_EXP) {
/* PCI-Express uncorrectable Error occurred */ /* PCI-Express uncorrectable Error occurred */
int aer = pci_find_aer_capability(hw->pdev);
u32 err; u32 err;
if (aer) { err = sky2_read32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS);
pci_read_config_dword(pdev, aer + PCI_ERR_UNCOR_STATUS, sky2_write32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS,
&err); 0xfffffffful);
pci_cleanup_aer_uncorrect_error_status(pdev);
} else {
/* Either AER not configured, or not working
* because of bad MMCONFIG, so just do recover
* manually.
*/
err = sky2_read32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS);
sky2_write32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS,
0xfffffffful);
}
if (net_ratelimit()) if (net_ratelimit())
dev_err(&pdev->dev, "PCI Express error (0x%x)\n", err); dev_err(&pdev->dev, "PCI Express error (0x%x)\n", err);
sky2_read32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS);
} }
if (status & Y2_HWE_L1_MASK) if (status & Y2_HWE_L1_MASK)
...@@ -2806,24 +2794,13 @@ static void sky2_reset(struct sky2_hw *hw) ...@@ -2806,24 +2794,13 @@ static void sky2_reset(struct sky2_hw *hw)
cap = pci_find_capability(pdev, PCI_CAP_ID_EXP); cap = pci_find_capability(pdev, PCI_CAP_ID_EXP);
if (cap) { if (cap) {
if (pci_find_aer_capability(pdev)) { sky2_write32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS,
/* Check for advanced error reporting */ 0xfffffffful);
pci_cleanup_aer_uncorrect_error_status(pdev);
pci_cleanup_aer_correct_error_status(pdev);
} else {
dev_warn(&pdev->dev,
"PCI Express Advanced Error Reporting"
" not configured or MMCONFIG problem?\n");
sky2_write32(hw, Y2_CFG_AER + PCI_ERR_UNCOR_STATUS,
0xfffffffful);
}
/* If error bit is stuck on ignore it */ /* If error bit is stuck on ignore it */
if (sky2_read32(hw, B0_HWE_ISRC) & Y2_IS_PCI_EXP) if (sky2_read32(hw, B0_HWE_ISRC) & Y2_IS_PCI_EXP)
dev_info(&pdev->dev, "ignoring stuck error report bit\n"); dev_info(&pdev->dev, "ignoring stuck error report bit\n");
else
else if (pci_enable_pcie_error_reporting(pdev))
hwe_mask |= Y2_IS_PCI_EXP; hwe_mask |= Y2_IS_PCI_EXP;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment