[libvirt] [PATCHv3 4/5] snapshot: Add flag to allow hypervisor restart when reverting snapshots

Peter Krempa pkrempa at redhat.com
Tue Nov 13 15:28:06 UTC 2012


On 11/12/12 18:49, Peter Krempa wrote:
> Some hypervisors require a respawn of the hypervisor to allow reverting
> to some snapshot states. This patch adds flag to remove the default
> safe approach to not allow this. When this flag is specified the
> hypervisor driver should re-emit events to allow management apps to
> reconnect.
>
> This flag is meant as a lesser way to enforce the restart of the
> hypervisor, that is a fairly common possibility compared to other
> meanings that the existing force flag has.
> ---
> New in series.
> Please see the next patch on actual use semantics. I was considering how to
> split the very powerful meaning of VIR_DOMAIN_SNAPSHOT_REVERT_FORCE to something
> less powerful but still useful. (I could add a flag that would allow invalidating of
> snapshot/image but that wouldn't be used that much)
> ---
>   include/libvirt/libvirt.h.in | 2 ++
>   src/libvirt.c                | 5 +++++
>   tools/virsh-snapshot.c       | 4 ++++
>   tools/virsh.pod              | 6 +++++-
>   4 files changed, 16 insertions(+), 1 deletion(-)
>

I should have added code to libvirt.c that will set the _RESPAWN flag 
when force is specified to save hassle. I will do it in the next version 
if this idea doesn't get NACKed to oblivion.

Peter




More information about the libvir-list mailing list