diff options
| author | Thomas Falcon <tlfalcon@linux.ibm.com> | 2019-01-24 20:17:01 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2019-01-25 09:48:15 +0300 |
| commit | e95d22c69b2c130ccce257b84daf283fd82d611e (patch) | |
| tree | 54cb193da4fe1f79dc338dedd83a8627e861b111 | |
| parent | 3b707c3008cad04604c1f50e39f456621821c414 (diff) | |
| download | linux-e95d22c69b2c130ccce257b84daf283fd82d611e.tar.xz | |
ibmveth: Do not process frames after calling napi_reschedule
The IBM virtual ethernet driver's polling function continues
to process frames after rescheduling NAPI, resulting in a warning
if it exhausted its budget. Do not restart polling after calling
napi_reschedule. Instead let frames be processed in the following
instance.
Signed-off-by: Thomas Falcon <tlfalcon@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/ibm/ibmveth.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/ethernet/ibm/ibmveth.c b/drivers/net/ethernet/ibm/ibmveth.c index 098d8764c0ea..dd71d5db7274 100644 --- a/drivers/net/ethernet/ibm/ibmveth.c +++ b/drivers/net/ethernet/ibm/ibmveth.c @@ -1313,7 +1313,6 @@ static int ibmveth_poll(struct napi_struct *napi, int budget) unsigned long lpar_rc; u16 mss = 0; -restart_poll: while (frames_processed < budget) { if (!ibmveth_rxq_pending_buffer(adapter)) break; @@ -1401,7 +1400,6 @@ restart_poll: napi_reschedule(napi)) { lpar_rc = h_vio_signal(adapter->vdev->unit_address, VIO_IRQ_DISABLE); - goto restart_poll; } } |