[libvirt] [PATCH] qemu: Mark domains as having managed state image only on managed save
Michal Privoznik
mprivozn at redhat.com
Wed Jul 4 09:01:15 UTC 2012
On 04.07.2012 10:26, Peter Krempa wrote:
> QEmu domains were marked as having managed save image even if they were
It's QEMU, Qemu, or qemu actually.
> saved using the regular save. With this patch, domains are marked so
> only when using managed save API.
> ---
> src/qemu/qemu_driver.c | 7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 2f93404..3410535 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -2739,7 +2739,6 @@ qemuDomainSaveInternal(struct qemud_driver *driver, virDomainPtr dom,
> }
>
> ret = 0;
> - vm->hasManagedSave = true;
>
> /* Shut it down */
> qemuProcessStop(driver, vm, VIR_DOMAIN_SHUTOFF_SAVED, 0);
> @@ -2916,8 +2915,10 @@ qemuDomainManagedSave(virDomainPtr dom, unsigned int flags)
> VIR_INFO("Saving state to %s", name);
>
> compressed = QEMUD_SAVE_FORMAT_RAW;
> - ret = qemuDomainSaveInternal(driver, dom, vm, name, compressed,
> - NULL, flags);
> + if ((ret = qemuDomainSaveInternal(driver, dom, vm, name, compressed,
> + NULL, flags)) == 0)
> + vm->hasManagedSave = true;
> +
> vm = NULL;
>
> cleanup:
>
ACK with capitalization fixed.
Michal
More information about the libvir-list
mailing list