Re: [PATCH v3 3/4] qemu: Keep numad hint after reboot

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

 



s/reboot/daemon restart/ ?

On Thu, Aug 13, 2015 at 10:18:33AM +0200, Martin Kletzander wrote:
> The numad hint stored in priv->autoNodeset is information that gets lost
> during daemon restart.  And because we would like to use that
> information in the future, we also need to save it in the status XML.
> For the sake of tests, we need to initialize nnumaCell_max to some
> value, so that the restoration doesn't fail in our test suite.  There is
> no need to fill in the actual numa cell data since the recalculating
> function virCapabilitiesGetCpusForNodemask() will not fail, it will just
> skip filling the data in the bitmap which we don't use in tests anyway.
> 
> Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> ---
>  src/qemu/qemu_domain.c  | 32 +++++++++++++++++++++++++++++++-
>  tests/qemuxml2xmltest.c |  3 ++-
>  tests/testutilsqemu.c   |  2 ++
>  3 files changed, 35 insertions(+), 2 deletions(-)

> @@ -805,15 +817,33 @@ qemuDomainObjPrivateXMLParse(xmlXPathContextPtr ctxt,
>      }
>      VIR_FREE(nodes);
> 
> +    if (!(caps = virQEMUDriverGetCapabilities(driver, false)))
> +        goto error;
> +
> +    if ((tmp = virXPathString("string(./numad/@nodeset)", ctxt))) {
> +        if (virBitmapParse(tmp, 0, &priv->autoNodeset,
> +                           caps->host.nnumaCell_max) < 0)
> +            goto error;
> +
> +        if (!(priv->autoCpuset = virCapabilitiesGetCpusForNodemask(caps,
> +                                                                   priv->autoNodeset)))
> +            goto error;
> +    }
> +    virObjectUnref(caps);
> +    caps = NULL;

This assignment seems redundant.

Jan

> +    VIR_FREE(tmp);
> +
>      return 0;
> 

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]