aboutsummaryrefslogtreecommitdiff
path: root/drivers/net/lib82596.c
diff options
context:
space:
mode:
authorZhu Yi <yi.zhu@intel.com>2008-11-27 13:42:20 +0800
committerJohn W. Linville <linville@tuxdriver.com>2008-12-05 09:18:35 -0500
commitb8ddafd759e356c839774a8b87011b768deb53b3 (patch)
tree9913a6a2510c70b2ba6a61f84f1d9384731df491 /drivers/net/lib82596.c
parent40a9a8299116297429298e8fcee08235134883f7 (diff)
ipw2200: fix netif_*_queue() removal regression
In "ipw2200: Call netif_*_queue() interfaces properly", netif_stop_queue() and netif_wake_queue() were removed with the reason "netif_carrier_{on,off}() handles starting and stopping packet flow into the driver". The patch also removes a valid condition check that ipw_tx_skb() cannot be called if device is not in STATUS_ASSOCIATED state. But netif_carrier_off() doesn't guarantee netdev->hard_start_xmit won't be called because linkwatch event is handled in a delayed workqueue. This caused a kernel oops reported by Frank Seidel: https://bugzilla.novell.com/show_bug.cgi?id=397390 This patch fixes the problem by moving the STATUS_ASSOCIATED check back to ipw_tx_skb(). It also adds a missing netif_carrier_off() call to ipw_disassociate(). Signed-off-by: Zhu Yi <yi.zhu@intel.com> Signed-off-by: Chatre, Reinette <reinette.chatre@intel.com> Tested-by: Frank Seidel <fseidel@suse.de> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/lib82596.c')
0 files changed, 0 insertions, 0 deletions