From 3f2d67bbfaa90d4c24c1d8c427b17dea0669a126 Mon Sep 17 00:00:00 2001 From: Maxim Nestratov Date: Tue, 10 Mar 2015 23:12:21 +0300 Subject: [PATCH] parallels: prevent domain define only if NUMA is really specified Signed-off-by: Maxim Nestratov --- src/parallels/parallels_sdk.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c index 5a3969ea6a..4ec91617f1 100644 --- a/src/parallels/parallels_sdk.c +++ b/src/parallels/parallels_sdk.c @@ -1816,7 +1816,15 @@ prlsdkCheckUnsupportedParams(PRL_HANDLE sdkdom, virDomainDefPtr def) return -1; } - if (def->numa) { + + /* + * Though we don't support NUMA configuration at the moment + * virDomainDefPtr always contain non zero NUMA configuration + * So, just make sure this configuration does't differ from auto generated. + */ + if ((virDomainNumatuneGetMode(def->numa, -1) != + VIR_DOMAIN_NUMATUNE_MEM_STRICT) || + virDomainNumatuneHasPerNodeBinding(def->numa)) { virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", _("numa parameters are not supported " "by parallels driver"));