bnx2: Handle error condition in ->slot_reset()
authorMichael Chan <mchan@broadcom.com>
Tue, 6 Aug 2013 22:50:07 +0000 (15:50 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 7 Aug 2013 17:44:03 +0000 (10:44 -0700)
by closing the device if necessary.  Otherwise, since NAPI state is
already disabled, a subsequent close will hang the system.

Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2.c

index 6a2de1d79ff6c011188f2ccfb8e48c315d93825c..3baf8b5631f9e5a10fd8e5fcea4fb4601fc48ddd 100644 (file)
@@ -8694,14 +8694,13 @@ static pci_ers_result_t bnx2_io_slot_reset(struct pci_dev *pdev)
 {
        struct net_device *dev = pci_get_drvdata(pdev);
        struct bnx2 *bp = netdev_priv(dev);
-       pci_ers_result_t result;
-       int err;
+       pci_ers_result_t result = PCI_ERS_RESULT_DISCONNECT;
+       int err = 0;
 
        rtnl_lock();
        if (pci_enable_device(pdev)) {
                dev_err(&pdev->dev,
                        "Cannot re-enable PCI device after reset\n");
-               result = PCI_ERS_RESULT_DISCONNECT;
        } else {
                pci_set_master(pdev);
                pci_restore_state(pdev);
@@ -8709,9 +8708,15 @@ static pci_ers_result_t bnx2_io_slot_reset(struct pci_dev *pdev)
 
                if (netif_running(dev)) {
                        bnx2_set_power_state(bp, PCI_D0);
-                       bnx2_init_nic(bp, 1);
+                       err = bnx2_init_nic(bp, 1);
                }
-               result = PCI_ERS_RESULT_RECOVERED;
+               if (!err)
+                       result = PCI_ERS_RESULT_RECOVERED;
+       }
+
+       if (result != PCI_ERS_RESULT_RECOVERED && netif_running(dev)) {
+               bnx2_napi_enable(bp);
+               dev_close(dev);
        }
        rtnl_unlock();