[PATCH v2 16/19] qemuDomainBlockCommit: Handle bitmaps on start of commit

Eric Blake eblake at redhat.com
Wed Mar 11 21:37:42 UTC 2020


On 3/11/20 7:56 AM, Peter Krempa wrote:
> On start of the commit job, we need to disable any active bitmap in the
> base. Use qemuBlockBitmapsHandleCommitStart to calculate which and call
> the appropriate QMP APIs. We use blockdev-reopen to make the 'base'
> writable to disable the bitmaps.
> 
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
>   src/qemu/qemu_driver.c | 43 +++++++++++++++++++++++++++++++++++++++++-
>   1 file changed, 42 insertions(+), 1 deletion(-)
> 

Someday, it would be nice if qemu let us do this without requiring us to 
do the reopen step ourselves, but in the meantime, this looks like it 
works.  Design-wise, I'm still not convinced why we have to disable the 
bitmaps in the base prior to the commit (as the only thing a bitmap is 
good for is for cumulative merging in determining how much of an image 
to expose over a future differential backup, but that differential is 
the same whether we write bits in one or multiple bitmaps as part of the 
commit operation).  But code wise, this looks accurate.

Reviewed-by: Eric Blake <eblake at redhat.com>

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




More information about the libvir-list mailing list