Re: [PATCH RFC 02/15] vhost-user: Add memory-isolation qdev property to vhost-user devices

Fri, 24 Jul 2026 04:04:48 -0700 

On 2026/07/24 7:30, ConKite wrote:

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.

Signed-off-by: Connor Kite <[email protected]>
Reviewed-by: Stefan Hajnoczi <[email protected]>
This and the next patch add interfaces before implementation. Perhaps it's better reversed. 

Regards,
Akihiko Odaki


---
  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          | 3 ++-
  hw/virtio/vhost-user-fs.c            | 1 +
  hw/virtio/vhost-user-scmi.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 +
  14 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
index 2e5b3ae1b1..9830dc6c35 100644
--- a/hw/block/vhost-user-blk.c
+++ b/hw/block/vhost-user-blk.c
@@ -631,6 +631,7 @@ static const VMStateDescription vmstate_vhost_user_blk = {
static const Property vhost_user_blk_properties[] = { 
      DEFINE_PROP_CHR("chardev", VHostUserBlk, chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VHostUserBlk, memory_isolation, 
false),
      DEFINE_PROP_UINT16("num-queues", VHostUserBlk, num_queues,
                         VHOST_USER_BLK_AUTO_NUM_QUEUES),
      DEFINE_PROP_UINT32("queue-size", VHostUserBlk, queue_size, 128),
diff --git a/hw/display/vhost-user-gpu.c b/hw/display/vhost-user-gpu.c
index 57360898ca..6d0ede672a 100644
--- a/hw/display/vhost-user-gpu.c
+++ b/hw/display/vhost-user-gpu.c
@@ -659,6 +659,7 @@ static struct vhost_dev 
*vhost_user_gpu_get_vhost(VirtIODevice *vdev)
static const Property vhost_user_gpu_properties[] = { 
      VIRTIO_GPU_BASE_PROPERTIES(VhostUserGPU, parent_obj.conf),
+    DEFINE_PROP_BOOL("memory-isolation", VhostUserGPU, memory_isolation, 
false),
  };
static void 
diff --git a/hw/scsi/vhost-user-scsi.c b/hw/scsi/vhost-user-scsi.c
index 5070178dc2..c1a0393220 100644
--- a/hw/scsi/vhost-user-scsi.c
+++ b/hw/scsi/vhost-user-scsi.c
@@ -346,6 +346,7 @@ static void vhost_user_scsi_unrealize(DeviceState *dev)
static const Property vhost_user_scsi_properties[] = { 
      DEFINE_PROP_CHR("chardev", VirtIOSCSICommon, conf.chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VirtIOSCSICommon, 
conf.memory_isolation, false),
      DEFINE_PROP_UINT32("boot_tpgt", VirtIOSCSICommon, conf.boot_tpgt, 0),
      DEFINE_PROP_UINT32("num_queues", VirtIOSCSICommon, conf.num_queues,
                         VIRTIO_SCSI_AUTO_NUM_QUEUES),
diff --git a/hw/virtio/vhost-user-base.c b/hw/virtio/vhost-user-base.c
index 90bd2903db..d2a74a2860 100644
--- a/hw/virtio/vhost-user-base.c
+++ b/hw/virtio/vhost-user-base.c
@@ -409,7 +409,8 @@ static void vub_device_unrealize(DeviceState *dev)
/*Define common qdev properties. Inherited by all children*/ 
  static const Property vub_properties[] = {
-    DEFINE_PROP_CHR("chardev", VHostUserBase, chardev)
+    DEFINE_PROP_CHR("chardev", VHostUserBase, chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VHostUserBase, memory_isolation, 
false)
  };
diff --git a/hw/virtio/vhost-user-fs.c b/hw/virtio/vhost-user-fs.c 
index 209993918a..3d980d814b 100644
--- a/hw/virtio/vhost-user-fs.c
+++ b/hw/virtio/vhost-user-fs.c
@@ -410,6 +410,7 @@ static const VMStateDescription vuf_backend_vmstate = {
static const Property vuf_properties[] = { 
      DEFINE_PROP_CHR("chardev", VHostUserFS, conf.chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VHostUserFS, conf.memory_isolation, 
false),
      DEFINE_PROP_STRING("tag", VHostUserFS, conf.tag),
      DEFINE_PROP_UINT16("num-request-queues", VHostUserFS,
                         conf.num_request_queues, 1),
diff --git a/hw/virtio/vhost-user-scmi.c b/hw/virtio/vhost-user-scmi.c
index 02dc088ea9..d514d89392 100644
--- a/hw/virtio/vhost-user-scmi.c
+++ b/hw/virtio/vhost-user-scmi.c
@@ -287,6 +287,7 @@ static const VMStateDescription vu_scmi_vmstate = {
static const Property vu_scmi_properties[] = { 
      DEFINE_PROP_CHR("chardev", VHostUserSCMI, chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VHostUserSCMI, memory_isolation, 
false),
  };
static void vu_scmi_class_init(ObjectClass *klass, const void *data) 
diff --git a/hw/virtio/vhost-user-vsock.c b/hw/virtio/vhost-user-vsock.c
index c2cd376e73..89aa789e22 100644
--- a/hw/virtio/vhost-user-vsock.c
+++ b/hw/virtio/vhost-user-vsock.c
@@ -154,6 +154,7 @@ static void vuv_device_unrealize(DeviceState *dev)
static const Property vuv_properties[] = { 
      DEFINE_PROP_CHR("chardev", VHostUserVSock, conf.chardev),
+    DEFINE_PROP_BOOL("memory-isolation", VHostUserVSock, 
conf.memory_isolation, false),
  };
static void vuv_class_init(ObjectClass *klass, const void *data) 
diff --git a/include/hw/virtio/vhost-user-base.h 
b/include/hw/virtio/vhost-user-base.h
index d5d2f081ae..29bc40736c 100644
--- a/include/hw/virtio/vhost-user-base.h
+++ b/include/hw/virtio/vhost-user-base.h
@@ -25,6 +25,7 @@ struct VHostUserBase {
      uint32_t num_vqs;
      uint32_t vq_size; /* can't exceed VIRTIO_QUEUE_MAX */
      uint32_t config_size;
+    bool memory_isolation;
      /* State tracking */
      VhostUserState vhost_user;
      struct vhost_virtqueue *vhost_vq;
diff --git a/include/hw/virtio/vhost-user-blk.h 
b/include/hw/virtio/vhost-user-blk.h
index dee848cfd8..8126447a86 100644
--- a/include/hw/virtio/vhost-user-blk.h
+++ b/include/hw/virtio/vhost-user-blk.h
@@ -30,6 +30,7 @@ OBJECT_DECLARE_SIMPLE_TYPE(VHostUserBlk, VHOST_USER_BLK)
  struct VHostUserBlk {
      VirtIODevice parent_obj;
      CharFrontend chardev;
+    bool memory_isolation;
      int32_t bootindex;
      struct virtio_blk_config blkcfg;
      uint16_t num_queues;
diff --git a/include/hw/virtio/vhost-user-fs.h 
b/include/hw/virtio/vhost-user-fs.h
index 5699ea03a1..bf71990b9c 100644
--- a/include/hw/virtio/vhost-user-fs.h
+++ b/include/hw/virtio/vhost-user-fs.h
@@ -28,6 +28,7 @@ typedef struct {
      char *tag;
      uint16_t num_request_queues;
      uint16_t queue_size;
+    bool memory_isolation;
  } VHostUserFSConf;
struct VHostUserFS { 
diff --git a/include/hw/virtio/vhost-user-scmi.h 
b/include/hw/virtio/vhost-user-scmi.h
index 30ab0d7ab7..d346b42ed8 100644
--- a/include/hw/virtio/vhost-user-scmi.h
+++ b/include/hw/virtio/vhost-user-scmi.h
@@ -19,6 +19,7 @@ OBJECT_DECLARE_SIMPLE_TYPE(VHostUserSCMI, VHOST_USER_SCMI);
  struct VHostUserSCMI {
      VirtIODevice parent;
      CharFrontend chardev;
+    bool memory_isolation;
      struct vhost_virtqueue *vhost_vqs;
      struct vhost_dev vhost_dev;
      VhostUserState vhost_user;
diff --git a/include/hw/virtio/vhost-user-vsock.h 
b/include/hw/virtio/vhost-user-vsock.h
index 5db6938b8b..6288b26fa4 100644
--- a/include/hw/virtio/vhost-user-vsock.h
+++ b/include/hw/virtio/vhost-user-vsock.h
@@ -21,6 +21,7 @@ OBJECT_DECLARE_SIMPLE_TYPE(VHostUserVSock, VHOST_USER_VSOCK)
typedef struct { 
      CharFrontend chardev;
+    bool memory_isolation;
  } VHostUserVSockConf;
struct VHostUserVSock { 
diff --git a/include/hw/virtio/virtio-gpu.h b/include/hw/virtio/virtio-gpu.h
index b9bad27c97..707612c3a4 100644
--- a/include/hw/virtio/virtio-gpu.h
+++ b/include/hw/virtio/virtio-gpu.h
@@ -277,6 +277,7 @@ struct VhostUserGPU {
      CharFrontend vhost_chr;
      QemuDmaBuf *dmabuf[VIRTIO_GPU_MAX_SCANOUTS];
      bool backend_blocked;
+    bool memory_isolation;
  };
#define MAX_SLOTS 4096 
diff --git a/include/hw/virtio/virtio-scsi.h b/include/hw/virtio/virtio-scsi.h
index b6028bb5cd..47fdc3785c 100644
--- a/include/hw/virtio/virtio-scsi.h
+++ b/include/hw/virtio/virtio-scsi.h
@@ -59,6 +59,7 @@ struct VirtIOSCSIConf {
      char *vhostfd;
      char *wwpn;
      CharFrontend chardev;
+    bool memory_isolation;
      uint32_t boot_tpgt;
      IOThread *iothread;
      IOThreadVirtQueueMappingList *iothread_vq_mapping_list;

Reply via email to