[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 00/16] packed ring virtio-net backend support
From: |
Wei Xu |
Subject: |
Re: [Qemu-devel] [PATCH v1 00/16] packed ring virtio-net backend support |
Date: |
Sun, 25 Nov 2018 21:59:00 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Nov 23, 2018 at 01:57:37PM +0800, Wei Xu wrote:
> On Thu, Nov 22, 2018 at 06:57:31PM +0100, Maxime Coquelin wrote:
> > Hi Wei,
> >
> > I just tested your series with Tiwei's v3, and it fails
> > with ctrl vq enabled:
> > qemu-system-x86_64: virtio-net ctrl missing headers
>
> OK, I haven't tried Tiwei's v3 yet, will give it a try.
Hi Maxime,
It is caused by the _F_NEXT flag bit for indirect descriptor as
mentioned by tiwei, this patch is needed to fix it.
diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
index 7487d3d..8e61e6f 100644
--- a/hw/virtio/virtio.c
+++ b/hw/virtio/virtio.c
@@ -1478,7 +1478,11 @@ static void *virtqueue_packed_pop(VirtQueue *vq, size_t
sz)
i -= vq->vring.num;
}
- if (desc.flags & VRING_DESC_F_NEXT) {
+ if (cache == &indirect_desc_cache) {
+ if (i == max)
+ break;
+ vring_packed_desc_read(vq->vdev, &desc, cache, i);
+ } else if (desc.flags & VRING_DESC_F_NEXT) {
vring_packed_desc_read(vq->vdev, &desc, cache, i);
} else {
- [Qemu-devel] [PATCH v1 10/16] virtio: event suppression support for packed ring, (continued)
- [Qemu-devel] [PATCH v1 10/16] virtio: event suppression support for packed ring, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 12/16] virtio: add userspace migration of packed ring, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 11/16] virtio-net: fill head desc after done all in a chain, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 13/16] virtio: add vhost-net migration of packed ring, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 14/16] virtio: packed ring feature bit for userspace backend, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 16/16] virtio: enable packed ring via a new command line, wexu, 2018/11/22
- [Qemu-devel] [PATCH v1 15/16] vhost: enable packed ring, wexu, 2018/11/22
- Re: [Qemu-devel] [PATCH v1 00/16] packed ring virtio-net backend support, Maxime Coquelin, 2018/11/22
- Re: [Qemu-devel] [PATCH v1 00/16] packed ring virtio-net backend support, no-reply, 2018/11/23
- Re: [Qemu-devel] [PATCH v1 00/16] packed ring virtio-net backend support, no-reply, 2018/11/23