[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 01/13] vfio/migration: Move from STOP_COPY to STOP in vfio_save_cl
From: |
Cédric Le Goater |
Subject: |
[PULL 01/13] vfio/migration: Move from STOP_COPY to STOP in vfio_save_cleanup() |
Date: |
Mon, 11 Sep 2023 09:49:56 +0200 |
From: Avihai Horon <avihaih@nvidia.com>
Changing the device state from STOP_COPY to STOP can take time as the
device may need to free resources and do other operations as part of the
transition. Currently, this is done in vfio_save_complete_precopy() and
therefore it is counted in the migration downtime.
To avoid this, change the device state from STOP_COPY to STOP in
vfio_save_cleanup(), which is called after migration has completed and
thus is not part of migration downtime.
Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Tested-by: YangHang Liu <yanghliu@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
---
hw/vfio/migration.c | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git a/hw/vfio/migration.c b/hw/vfio/migration.c
index
2674f4bc472d16989910300958f697f26fefa442..8acd182a8bf3fcd0eb0368816ff3093242b103f5
100644
--- a/hw/vfio/migration.c
+++ b/hw/vfio/migration.c
@@ -383,6 +383,19 @@ static void vfio_save_cleanup(void *opaque)
VFIODevice *vbasedev = opaque;
VFIOMigration *migration = vbasedev->migration;
+ /*
+ * Changing device state from STOP_COPY to STOP can take time. Do it here,
+ * after migration has completed, so it won't increase downtime.
+ */
+ if (migration->device_state == VFIO_DEVICE_STATE_STOP_COPY) {
+ /*
+ * If setting the device in STOP state fails, the device should be
+ * reset. To do so, use ERROR state as a recover state.
+ */
+ vfio_migration_set_state(vbasedev, VFIO_DEVICE_STATE_STOP,
+ VFIO_DEVICE_STATE_ERROR);
+ }
+
g_free(migration->data_buffer);
migration->data_buffer = NULL;
migration->precopy_init_size = 0;
@@ -508,12 +521,6 @@ static int vfio_save_complete_precopy(QEMUFile *f, void
*opaque)
return ret;
}
- /*
- * If setting the device in STOP state fails, the device should be reset.
- * To do so, use ERROR state as a recover state.
- */
- ret = vfio_migration_set_state(vbasedev, VFIO_DEVICE_STATE_STOP,
- VFIO_DEVICE_STATE_ERROR);
trace_vfio_save_complete_precopy(vbasedev->name, ret);
return ret;
--
2.41.0
- [PULL 00/13] vfio queue, Cédric Le Goater, 2023/09/11
- [PULL 01/13] vfio/migration: Move from STOP_COPY to STOP in vfio_save_cleanup(),
Cédric Le Goater <=
- [PULL 02/13] sysemu: Add prepare callback to struct VMChangeStateEntry, Cédric Le Goater, 2023/09/11
- [PULL 03/13] qdev: Add qdev_add_vm_change_state_handler_full(), Cédric Le Goater, 2023/09/11
- [PULL 05/13] vfio/migration: Add P2P support for VFIO migration, Cédric Le Goater, 2023/09/11
- [PULL 08/13] vfio/migration: Fail adding device with enable-migration=on and existing blocker, Cédric Le Goater, 2023/09/11
- [PULL 06/13] vfio/migration: Allow migration of multiple P2P supporting devices, Cédric Le Goater, 2023/09/11
- [PULL 07/13] migration: Add migration prefix to functions in target.c, Cédric Le Goater, 2023/09/11
- [PULL 10/13] migration: Add .save_prepare() handler to struct SaveVMHandlers, Cédric Le Goater, 2023/09/11
- [PULL 04/13] vfio/migration: Refactor PRE_COPY and RUNNING state checks, Cédric Le Goater, 2023/09/11
- [PULL 09/13] migration: Move more initializations to migrate_init(), Cédric Le Goater, 2023/09/11
- [PULL 11/13] vfio/migration: Block VFIO migration with postcopy migration, Cédric Le Goater, 2023/09/11