[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 0/8] Packed virtqueue for virtio
From: |
Eugenio Perez Martin |
Subject: |
Re: [PATCH v7 0/8] Packed virtqueue for virtio |
Date: |
Mon, 28 Oct 2019 09:04:37 +0100 |
Hi Michael.
Totally agree with you. I investigated that for a short time but in
the end I tested manually with the coverage instrumentation, trying to
exercise the same "code" as current tests touches in split virtqueues.
I will automate at least these paths.
Thanks!
On Fri, Oct 25, 2019 at 1:35 PM Michael S. Tsirkin <address@hidden> wrote:
>
> On Fri, Oct 25, 2019 at 10:35:19AM +0200, Eugenio Pérez wrote:
> > Hi:
> >
> > This is an updated version of packed virtqueue support based on Wei and
> > Jason's
> > V6, mainly solving the clang leak detector error CI gave.
>
>
> Looks good, I will queue this up.
>
> It would be nice to add libqos based tests on top,
> based on Stefan's work.
>
>
> > Please review.
> >
> > Changes from V6:
> > - Commit reorder: Squash bugfix and sepparate big changes into smaller
> > commits.
> >
> > Changes from V5:
> > - Fix qemu's CI asan error.
> > - Move/copy rcu comments.
> > - Merge duplicated vdev->broken check between split and packet version.
> >
> > Eugenio Pérez (2):
> > virtio: Free blk virqueues at unrealize()
> > virtio: Free rnd virqueue at unrealize()
> >
> > Jason Wang (4):
> > virtio: basic packed virtqueue support
> > virtio: event suppression support for packed ring
> > vhost_net: enable packed ring support
> > virtio: add property to enable packed virtqueue
> >
> > Wei Xu (2):
> > virtio: basic structure for packed ring
> > virtio: device/driver area size calculation refactor for split ring
> >
> > hw/block/virtio-blk.c | 7 +-
> > hw/char/virtio-serial-bus.c | 2 +-
> > hw/net/vhost_net.c | 2 +
> > hw/scsi/virtio-scsi.c | 3 +-
> > hw/virtio/virtio-rng.c | 1 +
> > hw/virtio/virtio.c | 1154
> > ++++++++++++++++++++++++++++++++++++++-----
> > include/hw/virtio/virtio.h | 14 +-
> > 7 files changed, 1045 insertions(+), 138 deletions(-)
> >
> > --
> > 2.16.5
- [PATCH v7 1/8] virtio: basic structure for packed ring, (continued)
- [PATCH v7 1/8] virtio: basic structure for packed ring, Eugenio Pérez, 2019/10/25
- [PATCH v7 2/8] virtio: device/driver area size calculation refactor for split ring, Eugenio Pérez, 2019/10/25
- [PATCH v7 3/8] virtio: Free blk virqueues at unrealize(), Eugenio Pérez, 2019/10/25
- [PATCH v7 4/8] virtio: Free rnd virqueue at unrealize(), Eugenio Pérez, 2019/10/25
- [PATCH v7 6/8] virtio: event suppression support for packed ring, Eugenio Pérez, 2019/10/25
- [PATCH v7 5/8] virtio: basic packed virtqueue support, Eugenio Pérez, 2019/10/25
- [PATCH v7 7/8] vhost_net: enable packed ring support, Eugenio Pérez, 2019/10/25
- [PATCH v7 8/8] virtio: add property to enable packed virtqueue, Eugenio Pérez, 2019/10/25
- Re: [PATCH v7 0/8] Packed virtqueue for virtio, Michael S. Tsirkin, 2019/10/25