Skip to content

Commit

Permalink
e100: fix irq leak on suspend/resume
Browse files Browse the repository at this point in the history
e100: fix irq leak on suspend/resume

From: Frederik Deweerdt <frederik.deweerdt@gmail.com>

The e100_resume() function should be calling netif_device_detach and
free_irq. This fixes multiple irq's being allocated after resume.

Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
  • Loading branch information
Auke Kok authored and Jeff Garzik committed Jan 30, 2007
1 parent f8a8ccd commit 518d833
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions drivers/net/e100.c
Original file line number Diff line number Diff line change
Expand Up @@ -2725,6 +2725,7 @@ static int e100_suspend(struct pci_dev *pdev, pm_message_t state)
del_timer_sync(&nic->watchdog);
netif_carrier_off(nic->netdev);

netif_device_detach(netdev);
pci_save_state(pdev);

if ((nic->flags & wol_magic) | e100_asf(nic)) {
Expand All @@ -2736,6 +2737,7 @@ static int e100_suspend(struct pci_dev *pdev, pm_message_t state)
}

pci_disable_device(pdev);
free_irq(pdev->irq, netdev);
pci_set_power_state(pdev, PCI_D3hot);

return 0;
Expand Down

0 comments on commit 518d833

Please sign in to comment.