[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 0/9] Packed virtqueue for virtio
From: |
Eugenio Perez Martin |
Subject: |
Re: [PATCH v6 0/9] Packed virtqueue for virtio |
Date: |
Fri, 25 Oct 2019 09:04:27 +0200 |
Hi Jason!
I can post a new version. You will have it in a moment.
Thanks!
On Fri, Oct 25, 2019 at 5:20 AM Jason Wang <address@hidden> wrote:
>
>
> On 2019/10/25 上午1:13, Eugenio Pérez wrote:
> > Hi:
> >
> > This is an updated version of packed virtqueue support based on Wei and
> > Jason's
> > V5, mainly solving the clang leak detector error CI gave.
> >
> > Please review.
> >
> > 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 (3):
> > virtio: Free rng and blk virqueues
> > virtio: add some rcu comments
> > virtio: Move vdev->broken check to dispatch drop_all
> >
> > 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/driverr area size calculation refactor for split ring
>
>
> Looks good to me.
>
> Just two nits:
>
> I tend to squash patch 8 and patch 9 into the patch that introduces
> those issues and split patch 3 into two parts.
>
> Btw, if you wish you can add your s-o-b to the series.
>
> Do you want to post a new version or I can tweak them by myself?
>
> Thanks
>
>
> >
> > 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(-)
> >
- [PATCH v6 1/9] virtio: basic structure for packed ring, (continued)
- [PATCH v6 1/9] virtio: basic structure for packed ring, Eugenio Pérez, 2019/10/24
- [PATCH v6 7/9] virtio: add property to enable packed virtqueue, Eugenio Pérez, 2019/10/24
- [PATCH v6 5/9] virtio: event suppression support for packed ring, Eugenio Pérez, 2019/10/24
- [PATCH v6 4/9] virtio: basic packed virtqueue support, Eugenio Pérez, 2019/10/24
- [PATCH v6 6/9] vhost_net: enable packed ring support, Eugenio Pérez, 2019/10/24
- [PATCH v6 3/9] virtio: Free rng and blk virqueues, Eugenio Pérez, 2019/10/24
- [PATCH v6 2/9] virtio: device/driver area size calculation refactor for split ring, Eugenio Pérez, 2019/10/24
- [PATCH v6 8/9] virtio: add some rcu comments, Eugenio Pérez, 2019/10/24
- [PATCH v6 9/9] virtio: Move vdev->broken check to dispatch drop_all, Eugenio Pérez, 2019/10/24
- Re: [PATCH v6 0/9] Packed virtqueue for virtio, Jason Wang, 2019/10/24
- Re: [PATCH v6 0/9] Packed virtqueue for virtio,
Eugenio Perez Martin <=