diff options
| author |   Eric W. Biederman <ebiederm@xmission.com> | 2014-03-14 18:00:06 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-03-14 22:52:47 -0400 |
| commit | 57ba34c9b068f314b219affafc19a39f8735d5e8 (patch) | |
| tree | acad4553c6cb2095deffbddc42b8d9a13a732b01 /drivers/net/ethernet/intel/igb | |
| parent | i40e: Don't receive packets when the napi budget == 0 (diff) | |
| download | linux-dev-57ba34c9b068f314b219affafc19a39f8735d5e8.tar.xz linux-dev-57ba34c9b068f314b219affafc19a39f8735d5e8.zip | |
igb: Don't receive packets when the napi budget == 0
Processing any incoming packets with a with a napi budget of 0
is incorrect driver behavior.
This matters as netpoll will shortly call drivers with a budget of 0
to avoid receive packet processing happening in hard irq context.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/intel/igb')
| -rw-r--r-- | drivers/net/ethernet/intel/igb/igb_main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethernet/intel/igb/igb_main.c index f81d87cfcc8d..d6b11522fed7 100644 --- a/drivers/net/ethernet/intel/igb/igb_main.c +++ b/drivers/net/ethernet/intel/igb/igb_main.c @@ -6946,7 +6946,7 @@ static bool igb_clean_rx_irq(struct igb_q_vector *q_vector, const int budget) unsigned int total_bytes = 0, total_packets = 0; u16 cleaned_count = igb_desc_unused(rx_ring); - do { + while (likely(total_packets < budget)) { union e1000_adv_rx_desc *rx_desc; /* return some buffers to hardware, one at a time is too slow */ @@ -6998,7 +6998,7 @@ static bool igb_clean_rx_irq(struct igb_q_vector *q_vector, const int budget) /* update budget accounting */ total_packets++; - } while (likely(total_packets < budget)); + } /* place incomplete frames back on ring for completion */ rx_ring->skb = skb; |