Skip to content

Commit

Permalink
sky2: reboot fix
Browse files Browse the repository at this point in the history
The call to napi_disable() in the PCI shutdown handler is problematic,
and is aggravated by the new NAPI.
Also, make sure watchdog timer doesn't go off.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
  • Loading branch information
Stephen Hemminger authored and Linus Torvalds committed Oct 14, 2007
1 parent 4fa4350 commit 5c0d6b3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/sky2.c
Original file line number Diff line number Diff line change
Expand Up @@ -4396,7 +4396,7 @@ static void sky2_shutdown(struct pci_dev *pdev)
if (!hw)
return;

napi_disable(&hw->napi);
del_timer_sync(&hw->watchdog_timer);

for (i = 0; i < hw->ports; i++) {
struct net_device *dev = hw->dev[i];
Expand Down

0 comments on commit 5c0d6b3

Please sign in to comment.