[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 6/7] hw/virtio: add started_vu status field to vhost-user-gpio
From: |
Alex Bennée |
Subject: |
[PATCH v3 6/7] hw/virtio: add started_vu status field to vhost-user-gpio |
Date: |
Mon, 28 Nov 2022 16:41:04 +0000 |
As per the fix to vhost-user-blk in f5b22d06fb (vhost: recheck dev
state in the vhost_migration_log routine) we really should track the
connection and starting separately.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
include/hw/virtio/vhost-user-gpio.h | 10 ++++++++++
hw/virtio/vhost-user-gpio.c | 11 ++++-------
2 files changed, 14 insertions(+), 7 deletions(-)
diff --git a/include/hw/virtio/vhost-user-gpio.h
b/include/hw/virtio/vhost-user-gpio.h
index 4fe9aeecc0..a9305c5e6c 100644
--- a/include/hw/virtio/vhost-user-gpio.h
+++ b/include/hw/virtio/vhost-user-gpio.h
@@ -28,7 +28,17 @@ struct VHostUserGPIO {
VhostUserState vhost_user;
VirtQueue *command_vq;
VirtQueue *interrupt_vq;
+ /**
+ * There are at least two steps of initialization of the
+ * vhost-user device. The first is a "connect" step and
+ * second is a "start" step. Make a separation between
+ * those initialization phases by using two fields.
+ *
+ * @connected: see vu_gpio_connect()/vu_gpio_disconnect()
+ * @started_vu: see vu_gpio_start()/vu_gpio_stop()
+ */
bool connected;
+ bool started_vu;
/*< public >*/
};
diff --git a/hw/virtio/vhost-user-gpio.c b/hw/virtio/vhost-user-gpio.c
index b38e4d4cf0..75e28bcd3b 100644
--- a/hw/virtio/vhost-user-gpio.c
+++ b/hw/virtio/vhost-user-gpio.c
@@ -86,6 +86,7 @@ static int vu_gpio_start(VirtIODevice *vdev)
error_report("Error starting vhost-user-gpio: %d", ret);
goto err_guest_notifiers;
}
+ gpio->started_vu = true;
/*
* guest_notifier_mask/pending not used yet, so just unmask
@@ -126,16 +127,12 @@ static void vu_gpio_stop(VirtIODevice *vdev)
struct vhost_dev *vhost_dev = &gpio->vhost_dev;
int ret;
- if (!k->set_guest_notifiers) {
+ if (!gpio->started_vu) {
return;
}
+ gpio->started_vu = false;
- /*
- * We can call vu_gpio_stop multiple times, for example from
- * vm_state_notify and the final object finalisation. Check we
- * aren't already stopped before doing so.
- */
- if (!vhost_dev_is_started(vhost_dev)) {
+ if (!k->set_guest_notifiers) {
return;
}
--
2.34.1
- [PATCH for 7.2-rc3 v3 0/7] fix vhost-user issues with CI, Alex Bennée, 2022/11/28
- [PATCH v3 3/7] tests/qtests: override "force-legacy" for gpio virtio-mmio tests, Alex Bennée, 2022/11/28
- [PATCH v3 6/7] hw/virtio: add started_vu status field to vhost-user-gpio,
Alex Bennée <=
- [PATCH v3 5/7] vhost: enable vrings in vhost_dev_start() for vhost-user devices, Alex Bennée, 2022/11/28
- [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Alex Bennée, 2022/11/28
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Stefan Hajnoczi, 2022/11/28
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Alex Bennée, 2022/11/28
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Michael S. Tsirkin, 2022/11/29
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Stefan Hajnoczi, 2022/11/29
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Michael S. Tsirkin, 2022/11/29
- Re: [PATCH v3 4/7] hw/virtio: ensure a valid host_feature set for virtio-user-gpio, Alex Bennée, 2022/11/29
[PATCH v3 2/7] include/hw: VM state takes precedence in virtio_device_should_start, Alex Bennée, 2022/11/28