[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/2] virtio: Add vhost-user-gpio device's support
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH 0/2] virtio: Add vhost-user-gpio device's support |
Date: |
Thu, 20 Jan 2022 06:25:32 -0500 |
On Mon, Jan 17, 2022 at 10:11:21AM +0000, Alex Bennée wrote:
>
> "Michael S. Tsirkin" <mst@redhat.com> writes:
>
> > On Wed, Jan 12, 2022 at 05:04:57PM +0530, Viresh Kumar wrote:
> >> Hello,
> >>
> >> This patchset adds vhost-user-gpio device's support in Qemu. The support
> >> for the
> >> same has already been added to virtio specification and Linux Kernel.
> >>
> >> A Rust based backend is also in progress and is tested against this
> >> patchset:
> >>
> >> https://github.com/rust-vmm/vhost-device/pull/76
> >
> >
> > I'm reluctant to add this with no tests in tree.
> > Want to write a minimal libhost-user based backend?
>
> This is going to be a problem going forward as we have more out-of-tree
> backends written as a first preference. While the first couple of vhost
> devices have C implementations in contrib before we worked on the rust
> version I think we are getting to the point of skipping a first C
> version for future devices.
Not sure what to do then. Integrate using submodules somehow?
> However I notice we have qtest/vhost-user-test.c so would that be enough
> to ensure we can instantiate the device and a basic vhost-user
> initialisation sequence doesn't cause it to crap out. This obviously
> won't be exercising the virtq processing itself but does that really
> exercise any of QEMU's boilerplate anyway?
we do have tests e.g. for network based on vhost-user, yes.
> > We also need some maintainers to step up.
>
> You mean more reviewers for the vhost and virtio sections of QEMU's
> maintainers?
>
> >
> >
> >> --
> >> Viresh
> >>
> >> Viresh Kumar (2):
> >> hw/virtio: add boilerplate for vhost-user-gpio device
> >> hw/virtio: add vhost-user-gpio-pci boilerplate
> >>
> >> hw/virtio/Kconfig | 5 +
> >> hw/virtio/meson.build | 2 +
> >> hw/virtio/vhost-user-gpio-pci.c | 69 ++++++
> >> hw/virtio/vhost-user-gpio.c | 343 ++++++++++++++++++++++++++++
> >> include/hw/virtio/vhost-user-gpio.h | 35 +++
> >> 5 files changed, 454 insertions(+)
> >> create mode 100644 hw/virtio/vhost-user-gpio-pci.c
> >> create mode 100644 hw/virtio/vhost-user-gpio.c
> >> create mode 100644 include/hw/virtio/vhost-user-gpio.h
> >>
> >> --
> >> 2.31.1.272.g89b43f80a514
>
>
> --
> Alex Bennée
- Re: [PATCH 1/2] hw/virtio: add boilerplate for vhost-user-gpio device, (continued)
[PATCH 2/2] hw/virtio: add vhost-user-gpio-pci boilerplate, Viresh Kumar, 2022/01/12
Re: [PATCH 0/2] virtio: Add vhost-user-gpio device's support, Michael S. Tsirkin, 2022/01/17
Re: [PATCH 0/2] virtio: Add vhost-user-gpio device's support,
Michael S. Tsirkin <=