在 2022/11/8 16:46, Stefano Garzarella 写道:
On Tue, Nov 08, 2022 at 08:41:56AM +0800, Longpeng(Mike) wrote:
From: Longpeng <longpe...@huawei.com>

The generic vDPA device doesn't support migration currently, so
mark it as unmigratable temporarily.

Signed-off-by: Longpeng <longpe...@huawei.com>
---
hw/virtio/vdpa-dev.c | 1 +
1 file changed, 1 insertion(+)

Is there a particular reason why we don't squash this change in the second patch of this series where we add the hw/virtio/vdpa-dev.c file?

No, just want to make it noticeable.

Anyway, this change LGTM:

Reviewed-by: Stefano Garzarella <sgarz...@redhat.com>

Thanks.


diff --git a/hw/virtio/vdpa-dev.c b/hw/virtio/vdpa-dev.c
index 2885d06cbe..62d83d3423 100644
--- a/hw/virtio/vdpa-dev.c
+++ b/hw/virtio/vdpa-dev.c
@@ -327,6 +327,7 @@ static Property vhost_vdpa_device_properties[] = {

static const VMStateDescription vmstate_vhost_vdpa_device = {
    .name = "vhost-vdpa-device",
+    .unmigratable = 1,
    .minimum_version_id = 1,
    .version_id = 1,
    .fields = (VMStateField[]) {
--
2.23.0


.

Reply via email to