diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2014-03-14 21:10:50 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-03-14 22:52:48 -0400 |
commit | c7b82cc8d95e69afe2dc1038fb73abc048a3bca5 (patch) | |
tree | 841f71d7ec0f20ca4573064a531292ab3918d313 /drivers/net/ethernet/neterion/vxge/vxge-main.c | |
parent | 176f792f571ffef6d86b4254424cc05bb3874c9e (diff) |
vxge: 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/neterion/vxge/vxge-main.c')
-rw-r--r-- | drivers/net/ethernet/neterion/vxge/vxge-main.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/ethernet/neterion/vxge/vxge-main.c b/drivers/net/ethernet/neterion/vxge/vxge-main.c index c5bb1ace4a74..11adc89959c1 100644 --- a/drivers/net/ethernet/neterion/vxge/vxge-main.c +++ b/drivers/net/ethernet/neterion/vxge/vxge-main.c | |||
@@ -368,6 +368,9 @@ vxge_rx_1b_compl(struct __vxge_hw_ring *ringh, void *dtr, | |||
368 | vxge_debug_entryexit(VXGE_TRACE, "%s: %s:%d", | 368 | vxge_debug_entryexit(VXGE_TRACE, "%s: %s:%d", |
369 | ring->ndev->name, __func__, __LINE__); | 369 | ring->ndev->name, __func__, __LINE__); |
370 | 370 | ||
371 | if (ring->budget <= 0) | ||
372 | goto out; | ||
373 | |||
371 | do { | 374 | do { |
372 | prefetch((char *)dtr + L1_CACHE_BYTES); | 375 | prefetch((char *)dtr + L1_CACHE_BYTES); |
373 | rx_priv = vxge_hw_ring_rxd_private_get(dtr); | 376 | rx_priv = vxge_hw_ring_rxd_private_get(dtr); |
@@ -525,6 +528,7 @@ vxge_rx_1b_compl(struct __vxge_hw_ring *ringh, void *dtr, | |||
525 | if (first_dtr) | 528 | if (first_dtr) |
526 | vxge_hw_ring_rxd_post_post_wmb(ringh, first_dtr); | 529 | vxge_hw_ring_rxd_post_post_wmb(ringh, first_dtr); |
527 | 530 | ||
531 | out: | ||
528 | vxge_debug_entryexit(VXGE_TRACE, | 532 | vxge_debug_entryexit(VXGE_TRACE, |
529 | "%s:%d Exiting...", | 533 | "%s:%d Exiting...", |
530 | __func__, __LINE__); | 534 | __func__, __LINE__); |