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

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

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

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.


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