diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2007-11-19 11:20:43 -0500 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2007-11-18 19:20:44 -0500 |
| commit | 8329d98e480250ef5f5a083f9c3af50510b5e65d (patch) | |
| tree | 183f79256e8c37cbc68ea115174f390e99120a77 | |
| parent | 9a4b9708f1f2eaf5edd619df578cf3afec36eb82 (diff) | |
virtio: fix net driver loop case where we fail to restart
skb is only NULL the first time around: it's more correct to test for
being under-budget.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
| -rw-r--r-- | drivers/net/virtio_net.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index d74e6f4aa248..5413dbf3d4ac 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c | |||
| @@ -198,8 +198,8 @@ again: | |||
| 198 | if (vi->num < vi->max / 2) | 198 | if (vi->num < vi->max / 2) |
| 199 | try_fill_recv(vi); | 199 | try_fill_recv(vi); |
| 200 | 200 | ||
| 201 | /* All done? */ | 201 | /* Out of packets? */ |
| 202 | if (!skb) { | 202 | if (received < budget) { |
| 203 | netif_rx_complete(vi->dev, napi); | 203 | netif_rx_complete(vi->dev, napi); |
| 204 | if (unlikely(!vi->rvq->vq_ops->restart(vi->rvq)) | 204 | if (unlikely(!vi->rvq->vq_ops->restart(vi->rvq)) |
| 205 | && netif_rx_reschedule(vi->dev, napi)) | 205 | && netif_rx_reschedule(vi->dev, napi)) |
