[libvirt] [PATCH] qemu: Fix starting domains with no cpu cgroup
Jiri Denemark
jdenemar at redhat.com
Wed Aug 29 14:17:30 UTC 2012
On Wed, Aug 29, 2012 at 21:51:58 +0800, Daniel Veillard wrote:
> On Wed, Aug 29, 2012 at 03:42:57PM +0200, Jiri Denemark wrote:
> > If cgroups are enabled in general but cpu cgroup is disabled in
> > qemu.conf or not mounted at all, libvirt would refuse to start any
> > domain even though scheduler parameters are not set in domain XML.
> >
> > This patch makes cpu cgroup mandatory only for domains that actually
> > want to use it.
> > ---
> > src/qemu/qemu_cgroup.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c
> > index be1b96d..df67ff3 100644
> > --- a/src/qemu/qemu_cgroup.c
> > +++ b/src/qemu/qemu_cgroup.c
> > @@ -549,7 +549,8 @@ int qemuSetupCgroupForVcpu(struct qemud_driver *driver, virDomainObjPtr vm)
> > if (driver->cgroup == NULL)
> > return 0; /* Not supported, so claim success */
> >
> > - if (!qemuCgroupControllerActive(driver, VIR_CGROUP_CONTROLLER_CPU)) {
> > + if ((period || quota) &&
> > + !qemuCgroupControllerActive(driver, VIR_CGROUP_CONTROLLER_CPU)) {
> > virReportError(VIR_ERR_SYSTEM_ERROR, "%s",
> > _("cgroup cpu is not active"));
> > return -1;
>
> ACK, please push :-)
Pushed, thanks.
Jirka
More information about the libvir-list
mailing list