Re: [PATCH 3/4] backup: Add new qemu monitor bitmap

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

 



On Wed, Jun 05, 2019 at 22:01:09 -0500, Eric Blake wrote:
> The upcoming virDomainBackup() API needs to take advantage of various
> qcow2 bitmap manipulations as the basis to virDomainCheckpoints and
> incremental backups.  Add four functions to expose
> block-dirty-bitmap-{add,enable,disable,merge} (this is the
> recently-added QEMU_CAPS_BITMAP_MERGE capability).
> 
> Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>
> ---
>  src/qemu/qemu_monitor.h      |  19 ++++++
>  src/qemu/qemu_monitor_json.h |  17 +++++
>  src/qemu/qemu_monitor.c      |  51 +++++++++++++++
>  src/qemu/qemu_monitor_json.c | 119 +++++++++++++++++++++++++++++++++++
>  4 files changed, 206 insertions(+)

I'd suggest you add GEN_TEST_FUNC/DO_TEST_GEN tests for those. It's
simple to add and gives you checking of the arguments against the QAPI
schema for free.

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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