]> git.ipfire.org Git - thirdparty/qemu.git/commit
migration/savevm: Remove extra load cleanup calls
authorFabiano Rosas <farosas@suse.de>
Tue, 17 Sep 2024 18:58:01 +0000 (15:58 -0300)
committerPeter Xu <peterx@redhat.com>
Wed, 18 Sep 2024 18:27:39 +0000 (14:27 -0400)
commit90a384d461af79c8948f1150523f868ef4862573
tree0ae27d112d95e15393e19619e2129b82c871091d
parentd8d5ca40048b04750de5a0ae0b2b9f153a391951
migration/savevm: Remove extra load cleanup calls

There are two qemu_loadvm_state_cleanup() calls that were introduced
when qemu_loadvm_state_setup() was still called before loading the
configuration section, so there was state to be cleaned up if the
header checks failed.

However, commit 9e14b84908 ("migration/savevm: load_header before
load_setup") has moved that configuration section part to
qemu_loadvm_state_header() which now happens before
qemu_loadvm_state_setup().

Remove the cleanup calls that are now misplaced.

Note that we didn't use Fixes because it's benign to cleanup() even if
setup() is not invoked.  So this patch is not needed for stable, as it
falls into cleanup category.

Reviewed-by: Peter Xu <peterx@redhat.com>
Signed-off-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/r/20240917185802.15619-2-farosas@suse.de
[peterx: added last paragraph of commit message]
Signed-off-by: Peter Xu <peterx@redhat.com>
migration/savevm.c