diff options
| author | Wei Yang <richard.weiyang@gmail.com> | 2016-02-04 22:50:30 +0000 |
|---|---|---|
| committer | Timothy Pearson <tpearson@raptorengineering.com> | 2019-11-29 19:45:59 -0600 |
| commit | 1a14863c482ca4404d58dfeb5937c78d56cdd559 (patch) | |
| tree | f0785c583c5a04af7bd630a1900797f3b3460bdf /migration | |
| parent | 3e827f50f372e562e9639fa83eff98b0cc02d9ba (diff) | |
| download | hqemu-1a14863c482ca4404d58dfeb5937c78d56cdd559.zip hqemu-1a14863c482ca4404d58dfeb5937c78d56cdd559.tar.gz | |
migration: reorder code to make it symmetric
In qemu_savevm_state_complete_precopy(), it iterates on each device to add
a json object and transfer related status to destination, while the order
of the last two steps could be refined.
Current order:
json_start_object()
save_section_header()
vmstate_save()
json_end_object()
save_section_footer()
After the change:
json_start_object()
save_section_header()
vmstate_save()
save_section_footer()
json_end_object()
This patch reorder the code to to make it symmetric. No functional change.
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Reviewed-by: Amit Shah <amit.shah@redhat.com>
Message-Id: <1454626230-16334-1-git-send-email-richard.weiyang@gmail.com>
Signed-off-by: Amit Shah <amit.shah@redhat.com>
Diffstat (limited to 'migration')
| -rw-r--r-- | migration/savevm.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/migration/savevm.c b/migration/savevm.c index 94f2894..02e8487 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -1088,12 +1088,11 @@ void qemu_savevm_state_complete_precopy(QEMUFile *f, bool iterable_only) json_prop_int(vmdesc, "instance_id", se->instance_id); save_section_header(f, se, QEMU_VM_SECTION_FULL); - vmstate_save(f, se, vmdesc); - - json_end_object(vmdesc); trace_savevm_section_end(se->idstr, se->section_id, 0); save_section_footer(f, se); + + json_end_object(vmdesc); } if (!in_postcopy) { |
