Re: [libvirt] [Patch 1/1] enable --checkpoint option in 'virsh save'

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

 



Matt McCowan wrote:
> diff -ur libvirt.orig/include/libvirt/libvirt.h.in libvirt/include/libvirt/libvirt.h.in
> --- libvirt.orig/include/libvirt/libvirt.h.in	2009-01-20 21:14:03.000000000 +0900
> +++ libvirt/include/libvirt/libvirt.h.in	2009-02-24 13:09:07.000000000 +0900
> @@ -463,10 +463,16 @@
>  int                     virDomainSuspend        (virDomainPtr domain);
>  int                     virDomainResume         (virDomainPtr domain);
>  
> +/* Domain save flags. */
> +typedef enum {
> +  VIR_SAVE_CHECKPOINT                  = 1, /* checkpoint save */
> +} virDomainSaveFlags;
> +
>  /*
>   * Domain save/restore
>   */
>  int                     virDomainSave           (virDomainPtr domain,
> +                                                 unsigned long flags,
>                                                   const char *to);

Unfortunately, you cannot do this.  virDomainSave() is part of the public API in
libvirt, and that is guaranteed to never change.  So we can't add more fields to
it.  Your best bet at this point might be to add another call,
"virDomainCheckpoint", with the arguments you need.  Then factor out the
internal implementations of virDomainSave so that both virDomainSave and
virDomainCheckpoint share most of the same code, with just the slight
differences for checkpointing.

-- 
Chris Lalancette

--
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]