diff options
| author |   Pierre Pfister <ppfister@cisco.com> | 2017-10-05 14:24:05 +0200 |
|---|---|---|
| committer |   Neale Ranns <nranns@cisco.com> | 2017-10-10 19:09:35 +0000 |
| commit | 30ef35e636ac8946c80f710098b6333afa199e64 (patch) | |
| tree | 2a85ee62ecdc24a5244b1425c2a45cc8934f6caf | |
| parent | aa5df48cb233b377b5910694e2440a16e5973864 (diff) | |
fix buffer allocation for sparse jumbo frames in vhost
A bug was reported where a jumbo packet would stay in vhost
queue forever or until a large enough number of other packets
arrived in the queue too.
This is due to a bug in vhost input node buffer allocation.
The fix is to make sure that vhost always allocates at least
enough buffers for one single big packet. '40' is used to
account for 65kB frames.
Change-Id: I1d293028854165083e30cd798fab9d4140230b78
Signed-off-by: Pierre Pfister <ppfister@cisco.com>
| -rw-r--r-- | src/vnet/devices/virtio/vhost-user.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/vnet/devices/virtio/vhost-user.c b/src/vnet/devices/virtio/vhost-user.c index 4db5a23398d..4cbabe2eed8 100644 --- a/src/vnet/devices/virtio/vhost-user.c +++ b/src/vnet/devices/virtio/vhost-user.c @@ -1559,8 +1559,10 @@ vhost_user_if_input (vlib_main_t * vm, * per packet. In case packets are bigger, we will just yeld at some point * in the loop and come back later. This is not an issue as for big packet, * processing cost really comes from the memory copy. + * The assumption is that even big packets will fit in 40 buffers. */ - if (PREDICT_FALSE (vum->cpus[thread_index].rx_buffers_len < n_left + 1)) + if (PREDICT_FALSE (vum->cpus[thread_index].rx_buffers_len < n_left + 1 || + vum->cpus[thread_index].rx_buffers_len < 40)) { u32 curr_len = vum->cpus[thread_index].rx_buffers_len; vum->cpus[thread_index].rx_buffers_len += |