aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Xu <peterx@redhat.com>2025-01-14 18:07:43 -0500
committerFabiano Rosas <farosas@suse.de>2025-01-29 11:56:41 -0300
commit455c1963d390ccf3da25f1e8fbf2c9f1ccee27e4 (patch)
treeeb092ec263877e2372abb207adff9a6a3dc97f0b
parent15c2ffa0b739fb40e448892ff0d0d49f99263530 (diff)
downloadqemu-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.c20
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);
}