[libvirt] [PATCH] vmx: Remove check that numvcpus has to be an even number.
Peter Krempa
pkrempa at redhat.com
Wed May 30 09:52:45 UTC 2018
On Wed, May 30, 2018 at 10:33:45 +0100, Richard W.M. Jones wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1584091
>
> It is possible to create a VMware guest with an odd number of vCPUs,
> but such guests cannot be accessed by libvirt because of this
> unnecessary check.
>
> I retained the sanity-check that numvcpus must be >= 1.
>
> Signed-off-by: Richard W.M. Jones <rjones at redhat.com>
> ---
> src/vmx/vmx.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/vmx/vmx.c b/src/vmx/vmx.c
> index bdc27b15b0..456bbaf190 100644
> --- a/src/vmx/vmx.c
> +++ b/src/vmx/vmx.c
> @@ -1506,10 +1506,10 @@ virVMXParseConfig(virVMXContext *ctx,
> if (virVMXGetConfigLong(conf, "numvcpus", &numvcpus, 1, true) < 0)
> goto cleanup;
>
> - if (numvcpus <= 0 || (numvcpus % 2 != 0 && numvcpus != 1)) {
> + if (numvcpus <= 0) {
> virReportError(VIR_ERR_INTERNAL_ERROR,
> - _("Expecting VMX entry 'numvcpus' to be an unsigned "
> - "integer (1 or a multiple of 2) but found %lld"), numvcpus);
> + _("Expecting VMX entry 'numvcpus' to be a positive "
> + "integer >= 1, but found %lld"), numvcpus);
> goto cleanup;
> }
The same check is in virVMXFormatConfig:
maxvcpus = virDomainDefGetVcpusMax(def);
if (maxvcpus == 0 || (maxvcpus % 2 != 0 && maxvcpus != 1)) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("Expecting domain XML entry 'vcpu' to be 1 or a "
"multiple of 2 but found %d"),
maxvcpus);
goto cleanup;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180530/57e9dc9a/attachment-0001.sig>
More information about the libvir-list
mailing list