Re: [PATCH] libxl: libxl_get_max_cpus returning a libxl error from 4.4 onward

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

 



On mer, 2013-12-18 at 07:39 +0100, Martin Kletzander wrote:
> On Tue, Dec 17, 2013 at 06:43:25PM +0100, Dario Faggioli wrote:
> > diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
> > index 692c3b7..a31b094 100644
> > --- a/src/libxl/libxl_driver.c
> > +++ b/src/libxl/libxl_driver.c
> > @@ -1101,9 +1101,11 @@ libxlConnectGetMaxVcpus(virConnectPtr conn, const char *type ATTRIBUTE_UNUSED)
> >
> >      cfg = libxlDriverConfigGet(driver);
> >      ret = libxl_get_max_cpus(cfg->ctx);
> > -    /* libxl_get_max_cpus() will return 0 if there were any failures,
> > -       e.g. xc_physinfo() failing */
> > -    if (ret == 0)
> > +    /* On failure, libxl_get_max_cpus() will return ERROR_FAIL from Xen 4.4
> > +     * onward, but it ever returning 0 is obviously wrong too (and it is
> > +     * what happens, on failure, on Xen 4.3 and earlier). Therefore, a 'less
> > +     * or equal' is the catchall we want. */
> > +    if (ret <= 0)
> >          ret = -1;
> >
> >      virObjectUnref(cfg);
> >
> 
> Good to change that, thanks, but the call to:
> virBitmapNew(libxl_get_max_nodes(priv->ctx)) // in the same file
> should be fixed as well, especially since virBitmapNew takes an
> unsigned argument.  Could you modify that bit as well?
> 
Right! I checkd for other uses of *_max_cpus() and forgot to do the same
for *_max_nodes()! :-/

I'll do that, thanks for looking at the patch.

Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

--
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]