Re: [PATCH] qemu: snapshot: Collect 'query-named-block-nodes' prior to memory migration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/31/20 9:13 AM, Peter Krempa wrote:
When doing an external snapshot we migrate memory to a file as a form of
taking the memory state. This creates a problem as qemu deactivates all
active bitmaps after a successful migration. This means that calling
'query-named-block-nodes' will return an empty list of bitmaps for
devices. We use the bitmap list to propagate the active bitmaps into the
overlay files being created which is required for backups to work after
a snapshot. Since we wouldn't propagate anythign a subsequent backup

anything

will fail with:

invalid argument: missing or broken bitmap 'testchck' for disk 'vda'

To fix this, we can simply collect the bitmap list prior to the
migration.

https://bugzilla.redhat.com/show_bug.cgi?id=1862472

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---

Note that with current qemu the above steps will still fail as qemu
fails to 'cont' after a migration if backing images contain bitmaps
which is the case here.

See: https://lists.nongnu.org/archive/html/qemu-block/2020-07/msg01833.html

But that is on track to be fixed in 5.1-rc3, so that shouldn't be a show-stopper.


  src/qemu/qemu_driver.c | 17 +++++++++++------
  1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 0ad6359102..b655df8c98 100644
--- a/src/qemu/qemu_driver.c

Reviewed-by: Eric Blake <eblake@xxxxxxxxxx>

I'll leave it up to you to decide if this is a mandatory fix during freeze.

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux