motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-01 14:53:54 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

block: Inactivate external snapshot overlays when necessary

Browse source 
Putting an active block node on top of an inactive one is strictly
speaking an invalid configuration and the next patch will turn it into a
hard error.

However, taking a snapshot while disk images are inactive after
completing migration has an important use case: After migrating to a
file, taking an external snapshot is what is needed to take a full VM
snapshot.

In order for this to keep working after the later patches, change
creating a snapshot such that it automatically inactivates an overlay
that is added on top of an already inactive node.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Acked-by: Fabiano Rosas <farosas@suse.de>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-ID: <20250204211407.381505-4-kwolf@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
Kevin Wolf 2025-02-04 22:13:54 +01:00
parent a6490ec9d5
commit e80210ffb2
1 changed files with 16 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

16
blockdev.c
View file

@ -1497,6 +1497,22 @@ static void external_snapshot_action(TransactionAction *action,
return;
}
/*
* Older QEMU versions have allowed adding an active parent node to an
* inactive child node. This is unsafe in the general case, but there is an
* important use case, which is taking a VM snapshot with migration to file
* and then adding an external snapshot while the VM is still stopped and
* images are inactive. Requiring the user to explicitly create the overlay
* as inactive would break compatibility, so just do it automatically here
* to keep this working.
*/
if (bdrv_is_inactive(state->old_bs) && !bdrv_is_inactive(state->new_bs)) {
ret = bdrv_inactivate(state->new_bs, errp);
if (ret < 0) {
return;
}
}
ret = bdrv_append(state->new_bs, state->old_bs, errp);
if (ret < 0) {
return;