[libvirt] [PATCH] qemu: save domain status after set the blkio parameters

[Martin Kletzander]

On Mon, Sep 29, 2014 at 01:14:58PM +0800, Shanzhi Yu wrote:
>After set the blkio parameters for running domain, save the change into
>live xml is needed to survive restarting the libvirtd
>Same story with bug 1146511
>
>Signed-off-by: Shanzhi Yu <shyu at redhat.com>
>---
> src/qemu/qemu_driver.c | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
>index 6606154..9c96dea 100644
>--- a/src/qemu/qemu_driver.c
>+++ b/src/qemu/qemu_driver.c
>@@ -8090,6 +8090,8 @@ qemuDomainSetBlkioParameters(virDomainPtr dom,
>     }
>     if (ret < 0)
>         goto cleanup;
>+    if (virDomainSaveStatus(driver->xmlopt, cfg->stateDir, vm) < 0)
>+        goto cleanup;

It this failed, the function would return with 0, so there would be no
problem indicated and it would also skip the config part.  It should
be part of the "if (flags & VIR_DOMAIN_AFFECT_LIVE)".

I also wonder how come there is no job created for the domain.  That
needs to be fixed as well.

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140929/beea6a57/attachment-0001.sig>