vhost-user: Add and consolidate qdev properties

[PATCH 0/2] vhost-user: Add and consolidate qdev properties

ConKite posted 2 patches 3 days, 22 hours ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260603223430.3311916-1-connorkite@gmail.com

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Stefano Garzarella <sgarzare@redhat.com>, Raphael Norwitz <rnorwitz@nvidia.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>, "Alex Bennée" <alex.bennee@linaro.org>, Stefan Hajnoczi <stefanha@redhat.com>, Viresh Kumar <viresh.kumar@linaro.org>, Gerd Hoffmann <kraxel@redhat.com>, Mathieu Poirier <mathieu.poirier@linaro.org>, Manos Pitsidianakis <manos.pitsidianakis@linaro.org>, Milan Zamazal <mzamazal@redhat.com>, Haixu Cui <quic_haixcui@quicinc.com>, Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp>, Dmitry Osipenko <dmitry.osipenko@collabora.com>

hw/block/vhost-user-blk.c            |  1 +
hw/display/vhost-user-gpu.c          |  1 +
hw/scsi/vhost-user-scsi.c            |  1 +
hw/virtio/vhost-user-base.c          | 10 ++++++++++
hw/virtio/vhost-user-fs.c            |  1 +
hw/virtio/vhost-user-gpio.c          |  4 ----
hw/virtio/vhost-user-i2c.c           |  5 -----
hw/virtio/vhost-user-input.c         |  5 -----
hw/virtio/vhost-user-rng.c           |  5 -----
hw/virtio/vhost-user-rtc.c           |  4 ----
hw/virtio/vhost-user-scmi.c          |  1 +
hw/virtio/vhost-user-snd.c           |  1 -
hw/virtio/vhost-user-spi.c           |  5 -----
hw/virtio/vhost-user-test-device.c   |  1 -
hw/virtio/vhost-user-vsock.c         |  1 +
include/hw/virtio/vhost-user-base.h  |  1 +
include/hw/virtio/vhost-user-blk.h   |  1 +
include/hw/virtio/vhost-user-fs.h    |  1 +
include/hw/virtio/vhost-user-scmi.h  |  1 +
include/hw/virtio/vhost-user-vsock.h |  1 +
include/hw/virtio/virtio-gpu.h       |  1 +
include/hw/virtio/virtio-scsi.h      |  1 +
22 files changed, 23 insertions(+), 30 deletions(-)

Expand all Fold all

[PATCH 0/2] vhost-user: Add and consolidate qdev properties

Posted by ConKite 3 days, 22 hours ago

[1/2] Consolidate chardev qdev property definitions in the
VHostUserBase family of devices.  Now, the definition only takes place
within VHostUserBase initialization and is inherited by children.

[2/2] In support of future work to add a memory isolation mode to vhost-user
devices, a qdev property, "memory-isolation", is added to all vhost-user
devices.  This is simply a bool representing whether the mode is active
or not.

These patches are grouped together as they both touch on qdev properties
held in common by all vhost-user devices.  Taken together, these patches
ensure that commmon properties are defined in the same location and way.

Signed-off-by: Connor Kite <connorkite@gmail.com>
---
ConKite (2):
  vhost-user: Consolidate chardev property definitions
  vhost-user: Add memory-isolation qdev property to vhost-user devices

 hw/block/vhost-user-blk.c            |  1 +
 hw/display/vhost-user-gpu.c          |  1 +
 hw/scsi/vhost-user-scsi.c            |  1 +
 hw/virtio/vhost-user-base.c          | 10 ++++++++++
 hw/virtio/vhost-user-fs.c            |  1 +
 hw/virtio/vhost-user-gpio.c          |  4 ----
 hw/virtio/vhost-user-i2c.c           |  5 -----
 hw/virtio/vhost-user-input.c         |  5 -----
 hw/virtio/vhost-user-rng.c           |  5 -----
 hw/virtio/vhost-user-rtc.c           |  4 ----
 hw/virtio/vhost-user-scmi.c          |  1 +
 hw/virtio/vhost-user-snd.c           |  1 -
 hw/virtio/vhost-user-spi.c           |  5 -----
 hw/virtio/vhost-user-test-device.c   |  1 -
 hw/virtio/vhost-user-vsock.c         |  1 +
 include/hw/virtio/vhost-user-base.h  |  1 +
 include/hw/virtio/vhost-user-blk.h   |  1 +
 include/hw/virtio/vhost-user-fs.h    |  1 +
 include/hw/virtio/vhost-user-scmi.h  |  1 +
 include/hw/virtio/vhost-user-vsock.h |  1 +
 include/hw/virtio/virtio-gpu.h       |  1 +
 include/hw/virtio/virtio-scsi.h      |  1 +
 22 files changed, 23 insertions(+), 30 deletions(-)

-- 
2.43.0

Re: [PATCH 0/2] vhost-user: Add and consolidate qdev properties

Posted by Stefan Hajnoczi 3 days, 7 hours ago

On Wed, Jun 3, 2026 at 6:34 PM ConKite <connorkite@gmail.com> wrote:
>
> [1/2] Consolidate chardev qdev property definitions in the
> VHostUserBase family of devices.  Now, the definition only takes place
> within VHostUserBase initialization and is inherited by children.

This patch can be merged right away. It's a cleanup for vhost-user-base.

>
> [2/2] In support of future work to add a memory isolation mode to vhost-user
> devices, a qdev property, "memory-isolation", is added to all vhost-user
> devices.  This is simply a bool representing whether the mode is active
> or not.

This patch will need to wait until the memory isolation work is
completed since the qdev property currently does nothing.

>
> These patches are grouped together as they both touch on qdev properties
> held in common by all vhost-user devices.  Taken together, these patches
> ensure that commmon properties are defined in the same location and way.
>
> Signed-off-by: Connor Kite <connorkite@gmail.com>
> ---
> ConKite (2):
>   vhost-user: Consolidate chardev property definitions
>   vhost-user: Add memory-isolation qdev property to vhost-user devices
>
>  hw/block/vhost-user-blk.c            |  1 +
>  hw/display/vhost-user-gpu.c          |  1 +
>  hw/scsi/vhost-user-scsi.c            |  1 +
>  hw/virtio/vhost-user-base.c          | 10 ++++++++++
>  hw/virtio/vhost-user-fs.c            |  1 +
>  hw/virtio/vhost-user-gpio.c          |  4 ----
>  hw/virtio/vhost-user-i2c.c           |  5 -----
>  hw/virtio/vhost-user-input.c         |  5 -----
>  hw/virtio/vhost-user-rng.c           |  5 -----
>  hw/virtio/vhost-user-rtc.c           |  4 ----
>  hw/virtio/vhost-user-scmi.c          |  1 +
>  hw/virtio/vhost-user-snd.c           |  1 -
>  hw/virtio/vhost-user-spi.c           |  5 -----
>  hw/virtio/vhost-user-test-device.c   |  1 -
>  hw/virtio/vhost-user-vsock.c         |  1 +
>  include/hw/virtio/vhost-user-base.h  |  1 +
>  include/hw/virtio/vhost-user-blk.h   |  1 +
>  include/hw/virtio/vhost-user-fs.h    |  1 +
>  include/hw/virtio/vhost-user-scmi.h  |  1 +
>  include/hw/virtio/vhost-user-vsock.h |  1 +
>  include/hw/virtio/virtio-gpu.h       |  1 +
>  include/hw/virtio/virtio-scsi.h      |  1 +
>  22 files changed, 23 insertions(+), 30 deletions(-)
>
> --
> 2.43.0
>