diff options
author | Peter Xu <peterx@redhat.com> | 2025-01-14 18:07:43 -0500 |
---|---|---|
committer | Fabiano Rosas <farosas@suse.de> | 2025-01-29 11:56:41 -0300 |
commit | 455c1963d390ccf3da25f1e8fbf2c9f1ccee27e4 (patch) | |
tree | eb092ec263877e2372abb207adff9a6a3dc97f0b | |
parent | 15c2ffa0b739fb40e448892ff0d0d49f99263530 (diff) | |
download | qemu-455c1963d390ccf3da25f1e8fbf2c9f1ccee27e4.zip qemu-455c1963d390ccf3da25f1e8fbf2c9f1ccee27e4.tar.gz qemu-455c1963d390ccf3da25f1e8fbf2c9f1ccee27e4.tar.bz2 |
migration: Cleanup qemu_savevm_state_complete_precopy()
Now qemu_savevm_state_complete_precopy() is never used in postcopy, clean
it up as in_postcopy==false now unconditionally.
Signed-off-by: Peter Xu <peterx@redhat.com>
Tested-by: Jiri Denemark <jdenemar@redhat.com>
Reviewed-by: Juraj Marcin <jmarcin@redhat.com>
Link: https://lore.kernel.org/r/20250114230746.3268797-14-peterx@redhat.com
Signed-off-by: Fabiano Rosas <farosas@suse.de>
-rw-r--r-- | migration/savevm.c | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/migration/savevm.c b/migration/savevm.c index 0ddc4c8..bc375db 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -1577,25 +1577,19 @@ int qemu_savevm_state_complete_precopy_non_iterable(QEMUFile *f, int qemu_savevm_state_complete_precopy(QEMUFile *f, bool iterable_only) { int ret; - bool in_postcopy = migration_in_postcopy(); - if (!in_postcopy || iterable_only) { - ret = qemu_savevm_state_complete_precopy_iterable(f, in_postcopy); + ret = qemu_savevm_state_complete_precopy_iterable(f, false); + if (ret) { + return ret; + } + + if (!iterable_only) { + ret = qemu_savevm_state_complete_precopy_non_iterable(f, false); if (ret) { return ret; } } - if (iterable_only) { - goto flush; - } - - ret = qemu_savevm_state_complete_precopy_non_iterable(f, in_postcopy); - if (ret) { - return ret; - } - -flush: return qemu_fflush(f); } |