diff options
author | David S. Miller <davem@davemloft.net> | 2008-01-07 23:48:21 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-09 02:30:09 -0500 |
commit | 4ec2411980d0fd2995e8dea8a06fe57aa47523cb (patch) | |
tree | a4013547ff1bda8fc038623290debd62ad6dd7fe /drivers/net/ixgb/ixgb_main.c | |
parent | a0a46196cd98af5cc015842bba757571f02a8c30 (diff) |
[NET]: Do not check netif_running() and carrier state in ->poll()
Drivers do this to try to break out of the ->poll()'ing loop
when the device is being brought administratively down.
Now that we have a napi_disable() "pending" state we are going
to solve that problem generically.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ixgb/ixgb_main.c')
-rw-r--r-- | drivers/net/ixgb/ixgb_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ixgb/ixgb_main.c b/drivers/net/ixgb/ixgb_main.c index bf9085fe035a..a8bef52870fd 100644 --- a/drivers/net/ixgb/ixgb_main.c +++ b/drivers/net/ixgb/ixgb_main.c | |||
@@ -1794,7 +1794,7 @@ ixgb_clean(struct napi_struct *napi, int budget) | |||
1794 | ixgb_clean_rx_irq(adapter, &work_done, budget); | 1794 | ixgb_clean_rx_irq(adapter, &work_done, budget); |
1795 | 1795 | ||
1796 | /* if no Tx and not enough Rx work done, exit the polling mode */ | 1796 | /* if no Tx and not enough Rx work done, exit the polling mode */ |
1797 | if((!tx_cleaned && (work_done == 0)) || !netif_running(netdev)) { | 1797 | if((!tx_cleaned && (work_done == 0))) { |
1798 | netif_rx_complete(netdev, napi); | 1798 | netif_rx_complete(netdev, napi); |
1799 | ixgb_irq_enable(adapter); | 1799 | ixgb_irq_enable(adapter); |
1800 | } | 1800 | } |