[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] virtio-net: allow increasing rx queue size
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH v2] virtio-net: allow increasing rx queue size |
Date: |
Thu, 11 Aug 2016 13:48:31 +0200 |
On Wed, 10 Aug 2016 17:47:16 +0300
"Michael S. Tsirkin" <address@hidden> wrote:
> This allows increasing the rx queue size up to 1024: unlike with tx,
> guests don't put in huge S/G lists into RX so the risk of running into
> the max 1024 limitation due to some off-by-one seems small.
>
> It's helpful for users like OVS-DPDK which don't do any buffering on the
> host - 1K roughly matches 500 entries in tun + 256 in the current rx
> queue, which seems to work reasonably well. We could probably make do
> with ~750 entries but virtio spec limits us to powers of two.
> It might be a good idea to specify an s/g size limit in a future
> version.
>
> It also might be possible to make the queue size smaller down the road, 64
> seems like the minimal value which will still work (as guests seem to
> assume a queue full of 1.5K buffers is enough to process the largest
> incoming packet, which is ~64K). No one actually asked for this, and
> with virtio 1 guests can reduce ring size without need for host
> configuration, so don't bother with this for now.
>
> Cc: Cornelia Huck <address@hidden>
> Cc: Jason Wang <address@hidden>
> Suggested-by: Patrik Hermansson <address@hidden>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
>
> changes from v1:
> add macros as suggested by Cornelia
>
> include/hw/virtio/virtio-net.h | 1 +
> hw/net/virtio-net.c | 26 +++++++++++++++++++++++++-
> 2 files changed, 26 insertions(+), 1 deletion(-)
Reviewed-by: Cornelia Huck <address@hidden>