Re: [libvirt] [PATCH 5/5] [OPTIONAL] qemu: allow save and managedsave on guests marked as autodestroyable

2012-12-11 Thread Eric Blake
On 12/07/2012 06:27 AM, Peter Krempa wrote: On 12/07/12 14:19, Gao Yongwei wrote: 2012/12/7 Peter Krempa pkre...@redhat.com mailto:pkre...@redhat.com [...] I think, if a vm is marked has autodestroyed,then this means that we do not need this vm again on next reboot.? if so,what's the

[libvirt] [PATCH 5/5] [OPTIONAL] qemu: allow save and managedsave on guests marked as autodestroyable

2012-12-07 Thread Peter Krempa
I don't see any reason to forbid saving machines that should be autodestroyed. I will squash this to the previous one if this is okay. --- src/qemu/qemu_driver.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 7e19cdc..e5b6295 100644

Re: [libvirt] [PATCH 5/5] [OPTIONAL] qemu: allow save and managedsave on guests marked as autodestroyable

2012-12-07 Thread Gao Yongwei
2012/12/7 Peter Krempa pkre...@redhat.com I don't see any reason to forbid saving machines that should be autodestroyed. I will squash this to the previous one if this is okay. --- src/qemu/qemu_driver.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/src/qemu/qemu_driver.c

Re: [libvirt] [PATCH 5/5] [OPTIONAL] qemu: allow save and managedsave on guests marked as autodestroyable

2012-12-07 Thread Peter Krempa
On 12/07/12 14:19, Gao Yongwei wrote: 2012/12/7 Peter Krempa pkre...@redhat.com mailto:pkre...@redhat.com [...] I think, if a vm is marked has autodestroyed,then this means that we do not need this vm again on next reboot.? if so,what's the reason for us to save this vm? if we want to