diff options
author | Emil Tantilov <emil.s.tantilov@intel.com> | 2010-10-27 20:59:49 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-10-28 13:19:00 -0400 |
commit | ec43a81cfa85b1ba0c9ac1aa995fe652f30fd4d6 (patch) | |
tree | 110963d7857fe29750394485c4c4e404ca637f33 /drivers/net/ixgb | |
parent | 9806307a1c5a2a79e268ae4e78b437d38c8adf7f (diff) |
ixgb: call pci_disable_device in ixgb_remove
ixgb fails to work after reload on recent kernels:
rmmod ixgb (dev->current_state = PCI_UNKNOWN)
modprobe ixgb (pci_enable_device will bail leaving current_state to PCI_UNKNOWN)
ifup eth0
do_IRQ: 2.82 No irq handler for vector (irq -1)
The issue was exposed by commit fcd097f31a6ee207cc0c3da9cccd2a86d4334785
PCI: MSI: Remove unsafe and unnecessary hardware access
which avoids HW writes for power states != PCI_D0
CC: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com>
Tested-by: Jeff Pieper <jeffrey.e.pieper@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ixgb')
-rw-r--r-- | drivers/net/ixgb/ixgb_main.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ixgb/ixgb_main.c b/drivers/net/ixgb/ixgb_main.c index 666207a9c039..caa8192fff2a 100644 --- a/drivers/net/ixgb/ixgb_main.c +++ b/drivers/net/ixgb/ixgb_main.c | |||
@@ -533,6 +533,7 @@ ixgb_remove(struct pci_dev *pdev) | |||
533 | pci_release_regions(pdev); | 533 | pci_release_regions(pdev); |
534 | 534 | ||
535 | free_netdev(netdev); | 535 | free_netdev(netdev); |
536 | pci_disable_device(pdev); | ||
536 | } | 537 | } |
537 | 538 | ||
538 | /** | 539 | /** |