[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH V8 00/12] fix migration of suspended runstate
From: |
Steve Sistare |
Subject: |
[PATCH V8 00/12] fix migration of suspended runstate |
Date: |
Wed, 13 Dec 2023 07:30:09 -0800 |
Migration of a guest in the suspended runstate is broken. The incoming
migration code automatically tries to wake the guest, which is wrong;
the guest should end migration in the same runstate it started. Further,
after saving a snapshot in the suspended state and loading it, the vm_start
fails. The runstate is RUNNING, but the guest is not.
See the commit messages for the details.
Changes in V2:
* simplify "start on wakeup request"
* fix postcopy, snapshot, and background migration
* refactor fixes for each type of migration
* explicitly handled suspended events and runstate in tests
* add test for postcopy and background migration
Changes in V3:
* rebase to tip
* fix hang in new function migrate_wait_for_dirty_mem
Changes in V4:
* rebase to tip
* add patch for vm_prepare_start (thanks Peter)
* add patch to preserve cpu ticks
Changes in V5:
* rebase to tip
* added patches to completely stop vm in suspended state:
cpus: refactor vm_stop
cpus: stop vm in suspended state
* added patch to partially resume vm in suspended state:
cpus: start vm in suspended state
* modified "preserve suspended ..." patches to use the above.
* deleted patch "preserve cpu ticks if suspended". stop ticks in
vm_stop_force_state instead.
* deleted patch "add runstate function". defined new helper function
migrate_new_runstate in "preserve suspended runstate"
* Added some RB's, but removed other RB's because the patches changed.
Changes in V6:
* all vm_stop calls completely stop the suspended state
* refactored and updated the "cpus" patches
* simplified the "preserve suspended" patches
* added patch "bootfile per vm"
Changes in V7:
* rebase to tip, add RB-s
* fix backwards compatibility for global_state.vm_was_suspended
* delete vm_prepare_start state argument, and rename patch
"pass runstate to vm_prepare_start" to
"check running not RUN_STATE_RUNNING"
* drop patches:
tests/qtest: bootfile per vm
tests/qtest: background migration with suspend
* rename runstate_is_started to runstate_is_live
* move wait_for_suspend in tests
Changes in V8:
* rebase to tip
* add RB's
* add comment for runstate_is_live
* simplify global_state - the needed function, and its use of vm_was_suspended
Steve Sistare (12):
cpus: vm_was_suspended
cpus: stop vm in suspended runstate
cpus: check running not RUN_STATE_RUNNING
cpus: vm_resume
migration: propagate suspended runstate
migration: preserve suspended runstate
migration: preserve suspended for snapshot
migration: preserve suspended for bg_migration
tests/qtest: migration events
tests/qtest: option to suspend during migration
tests/qtest: precopy migration with suspend
tests/qtest: postcopy migration with suspend
backends/tpm/tpm_emulator.c | 2 +-
hw/usb/hcd-ehci.c | 2 +-
hw/usb/redirect.c | 2 +-
hw/xen/xen-hvm-common.c | 2 +-
include/migration/snapshot.h | 7 ++
include/sysemu/runstate.h | 20 ++++
migration/global_state.c | 47 +++++----
migration/migration-hmp-cmds.c | 8 +-
migration/migration.c | 15 +--
migration/savevm.c | 23 +++--
qapi/misc.json | 10 +-
system/cpus.c | 47 +++++++--
system/runstate.c | 9 ++
system/vl.c | 2 +
tests/migration/i386/Makefile | 5 +-
tests/migration/i386/a-b-bootblock.S | 50 +++++++++-
tests/migration/i386/a-b-bootblock.h | 26 +++--
tests/qtest/migration-helpers.c | 27 ++----
tests/qtest/migration-helpers.h | 11 ++-
tests/qtest/migration-test.c | 181 +++++++++++++++++++++++++----------
20 files changed, 352 insertions(+), 144 deletions(-)
--
1.8.3.1
- [PATCH V8 00/12] fix migration of suspended runstate,
Steve Sistare <=
- [PATCH V8 05/12] migration: propagate suspended runstate, Steve Sistare, 2023/12/13
- [PATCH V8 01/12] cpus: vm_was_suspended, Steve Sistare, 2023/12/13
- [PATCH V8 07/12] migration: preserve suspended for snapshot, Steve Sistare, 2023/12/13
- [PATCH V8 06/12] migration: preserve suspended runstate, Steve Sistare, 2023/12/13
- [PATCH V8 09/12] tests/qtest: migration events, Steve Sistare, 2023/12/13
- [PATCH V8 03/12] cpus: check running not RUN_STATE_RUNNING, Steve Sistare, 2023/12/13
- [PATCH V8 08/12] migration: preserve suspended for bg_migration, Steve Sistare, 2023/12/13
- [PATCH V8 02/12] cpus: stop vm in suspended runstate, Steve Sistare, 2023/12/13