Re: [Libvir] [PATCH] Implement SetVcpus and DomainGetMaxVcpus for qemu

[Chris Lalancette <clalance redhat com>]

Cole Robinson wrote:
> The attached patch fills in two of the vcpu functions for the qemu driver:
> 
> virDomainSetVcpus : set the number of vcpus the domain can use
> virDomainGetMaxVcpus : max number of vcpus that can be assigned to the domain.
> 
> Code change is only in qemu_driver, as the backend stuff was already in place.
> I also edited qemudGetMaxVcpus to ignore case when checking the passed OS
> type, since it wasn't matching the returned results of qemudDomainGetOSType.

OK, but...

> +    if (qemudIsActiveVM(vm)) {
> +        qemudReportError(dom->conn, dom, NULL, VIR_ERR_INTERNAL_ERROR, "%s",
> +                         _("cannot change vcpu count of an active domain"));
> +        return -1;
> +    }

I'm fairly certain KVM is going to support this in the near future (if not now,
I can't remember).  I'm not sure what libvirt's policy is about things that
don't work now, but will potentially work in the future, is, but it seems wrong
to disable this feature for KVM.

Chris Lalancette