diff options
author | Don Skidmore <donald.c.skidmore@intel.com> | 2009-09-18 05:45:43 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-09-21 14:32:29 -0400 |
commit | 0a1f87cbdbf26e1257b36cc5e9c00a08c2236ff2 (patch) | |
tree | 410d4028ec4f13b6da9ecae1f6c2decae698906a | |
parent | dedcf2971c250473e724b642c0100d3621116616 (diff) |
ixgbe: fix sfp_timer clean up in ixgbe_down
We weren't stoping the sfp_timer after the device was brought down.
This patch properly cleans up.
Signed-off-by: Shannon Nelson <shannon.nelson@intel.com>
Signed-off-by: Don Skidmore <donald.c.skidmore@intel.com>
Acked-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ixgbe/ixgbe_main.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c index 59ad9590e700..056434c9831c 100644 --- a/drivers/net/ixgbe/ixgbe_main.c +++ b/drivers/net/ixgbe/ixgbe_main.c | |||
@@ -2926,6 +2926,8 @@ void ixgbe_down(struct ixgbe_adapter *adapter) | |||
2926 | 2926 | ||
2927 | ixgbe_napi_disable_all(adapter); | 2927 | ixgbe_napi_disable_all(adapter); |
2928 | 2928 | ||
2929 | clear_bit(__IXGBE_SFP_MODULE_NOT_FOUND, &adapter->state); | ||
2930 | del_timer_sync(&adapter->sfp_timer); | ||
2929 | del_timer_sync(&adapter->watchdog_timer); | 2931 | del_timer_sync(&adapter->watchdog_timer); |
2930 | cancel_work_sync(&adapter->watchdog_task); | 2932 | cancel_work_sync(&adapter->watchdog_task); |
2931 | 2933 | ||