net: Fix issue with netdev_tx_reset_queue not resetting queue from XOFF state
authorAlexander Duyck <alexander.h.duyck@intel.com>
Tue, 7 Feb 2012 02:29:01 +0000 (02:29 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tue, 13 Mar 2012 03:16:53 +0000 (20:16 -0700)
commit5c4903549c05bbb373479e0ce2992573c120654a
tree32064a4a797dd19b286f921a83389980524a3918
parentb2d96e0ac07cf4929c6b0eb13121672048368117
net: Fix issue with netdev_tx_reset_queue not resetting queue from XOFF state

We are seeing dev_watchdog hangs on several drivers.  I suspect this is due
to the __QUEUE_STATE_STACK_XOFF bit being set prior to a reset for link
change, and then not being cleared by netdev_tx_reset_queue.  This change
corrects that.

In addition we were seeing dev_watchdog hangs on igb after running the
ethtool tests.  We found this to be due to the fact that the ethtool test
runs the same logic as ndo_start_xmit, but we were never clearing the XOFF
flag since the loopback test in ethtool does not do byte queue accounting.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Tested-by: Stephen Ko <stephen.s.ko@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igb/igb_main.c
include/linux/netdevice.h