Commit d9aee591 authored by Yuval Mintz's avatar Yuval Mintz Committed by David S. Miller

bnx2x: Don't release PCI bars on shutdown

The bnx2x driver in its pci shutdown() callback releases its pci bars (in the
same manner it does during its pci remove() callback).
During a system reboot while VFs are enabled, its possible for the VF's remove
to be called (as a result of pci_disable_sriov()) after its shutdown callback
has already finished running; This will cause a paging request fault as the VF
tries to access the pci bar which it has previously released, crashing the
system.

This patch further differentiates the shutdown and remove callbacks, preventing the
pci release procedures from being called during shutdown.
Signed-off-by: default avatarYuval Mintz <yuvalmin@broadcom.com>
Signed-off-by: default avatarAriel Elior <ariele@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a926592f
...@@ -12942,25 +12942,26 @@ static void __bnx2x_remove(struct pci_dev *pdev, ...@@ -12942,25 +12942,26 @@ static void __bnx2x_remove(struct pci_dev *pdev,
pci_set_power_state(pdev, PCI_D3hot); pci_set_power_state(pdev, PCI_D3hot);
} }
if (bp->regview) if (remove_netdev) {
iounmap(bp->regview); if (bp->regview)
iounmap(bp->regview);
/* for vf doorbells are part of the regview and were unmapped along with /* For vfs, doorbells are part of the regview and were unmapped
* it. FW is only loaded by PF. * along with it. FW is only loaded by PF.
*/ */
if (IS_PF(bp)) { if (IS_PF(bp)) {
if (bp->doorbells) if (bp->doorbells)
iounmap(bp->doorbells); iounmap(bp->doorbells);
bnx2x_release_firmware(bp); bnx2x_release_firmware(bp);
} }
bnx2x_free_mem_bp(bp); bnx2x_free_mem_bp(bp);
if (remove_netdev)
free_netdev(dev); free_netdev(dev);
if (atomic_read(&pdev->enable_cnt) == 1) if (atomic_read(&pdev->enable_cnt) == 1)
pci_release_regions(pdev); pci_release_regions(pdev);
}
pci_disable_device(pdev); pci_disable_device(pdev);
} }
......
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