Re: [PATCH] make virDomainGetMaxVcpus work on inactive domains

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

 



At Thu, 20 Feb 2014 12:19:35 -0700,
Jim Fehlig wrote:
> 
> Claudio Bley wrote:
> > Signed-off-by: Claudio Bley <cbley@xxxxxxxxxx>
> > ---
> > OK, how about this patch?
> >
> > While at it, should I convert the VIR_DOMAIN_VCPU_* instances to
> > VIR_DOMAIN_AFFECT_* instances for consistency?
> >
> >  src/esx/esx_driver.c       |    2 +-
> >  src/openvz/openvz_driver.c |    2 +-
> >  src/phyp/phyp_driver.c     |    2 +-
> >  src/qemu/qemu_driver.c     |    2 +-
> >  src/test/test_driver.c     |    2 +-
> >  src/vbox/vbox_tmpl.c       |    2 +-
> >  src/xen/xen_driver.c       |    2 +-
> >  src/xenapi/xenapi_driver.c |    2 +-
> >   
> 
> I think the libxl driver should be included here too.

Seems the libxl driver does not implement the virDomainGetMaxVcpus
function.

Claudio
-- 
BSc (Comp) Claudio Bley - Principal Software Engineer
AV-TEST GmbH, Klewitzstr. 7, 39112 Magdeburg, Germany
Phone: +49 391 6075460, Fax: +49 391 6075469
Web: <http://www.av-test.org>

* https://twitter.com/avtestorg * https://facebook.com/avtestorg *
* https://plus.google.com/100383867141221115206/ *

Eingetragen am / Registered at: Amtsgericht Stendal (HRB 114076)
Geschaeftsfuehrer (CEO): Andreas Marx, Guido Habicht, Maik Morgenstern

Our services shall be effected on the basis of the General Terms
and Conditions of AV-TEST GmbH, which are accessible under
<http://www.av-test.org/en/av-test/terms-and-conditions/> or
obtainable upon request.

Unsere Leistungen erfolgen auf der Grundlage der Allgemeinen
Geschäftsbedingungen der AV-TEST GmbH, die unter
<http://www.av-test.org/av-test/agb/> abrufbar sind oder auf
Anfrage übersandt werden.

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