diff options
| author | Auke Kok <auke-jan.h.kok@intel.com> | 2007-11-27 19:26:36 -0500 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-12-01 16:32:32 -0500 |
| commit | f7bbb9098315d712351aba7861a8c9fcf6bf0213 (patch) | |
| tree | ebe4936468e99b9f9124b5a8e6cfb461f59a576b /drivers/net | |
| parent | e970d1f8106514ea619d7a9005b1dc92c6049b32 (diff) | |
e1000: Fix NAPI state bug when Rx complete
Don't exit polling when we have not yet used our budget, this causes
the NAPI system to end up with a messed up poll list.
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/e1000/e1000_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c index cf39473ef90a..4f37506ad374 100644 --- a/drivers/net/e1000/e1000_main.c +++ b/drivers/net/e1000/e1000_main.c | |||
| @@ -3942,7 +3942,7 @@ e1000_clean(struct napi_struct *napi, int budget) | |||
| 3942 | &work_done, budget); | 3942 | &work_done, budget); |
| 3943 | 3943 | ||
| 3944 | /* If no Tx and not enough Rx work done, exit the polling mode */ | 3944 | /* If no Tx and not enough Rx work done, exit the polling mode */ |
| 3945 | if ((!tx_cleaned && (work_done < budget)) || | 3945 | if ((!tx_cleaned && (work_done == 0)) || |
| 3946 | !netif_running(poll_dev)) { | 3946 | !netif_running(poll_dev)) { |
| 3947 | quit_polling: | 3947 | quit_polling: |
| 3948 | if (likely(adapter->itr_setting & 3)) | 3948 | if (likely(adapter->itr_setting & 3)) |