[libvirt] [PATCH 01/15] conf: Make error reporting in virCPUDefIsEqual optional
Jiri Denemark
jdenemar at redhat.com
Mon Jun 5 09:26:49 UTC 2017
The function will be used in paths where mismatching CPU defs are not an
error.
Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
---
src/conf/cpu_conf.c | 119 ++++++++++++++++++++++++++++++-------------------
src/conf/cpu_conf.h | 3 +-
src/conf/domain_conf.c | 2 +-
3 files changed, 77 insertions(+), 47 deletions(-)
diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c
index a4be5742e..79ed5e646 100644
--- a/src/conf/cpu_conf.c
+++ b/src/conf/cpu_conf.c
@@ -811,7 +811,8 @@ virCPUDefAddFeature(virCPUDefPtr def,
bool
virCPUDefIsEqual(virCPUDefPtr src,
- virCPUDefPtr dst)
+ virCPUDefPtr dst,
+ bool reportError)
{
bool identical = false;
size_t i;
@@ -820,97 +821,123 @@ virCPUDefIsEqual(virCPUDefPtr src,
return true;
if ((src && !dst) || (!src && dst)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
- _("Target CPU does not match source"));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
+ _("Target CPU does not match source"));
+ }
goto cleanup;
}
if (src->type != dst->type) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU type %s does not match source %s"),
- virCPUTypeToString(dst->type),
- virCPUTypeToString(src->type));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU type %s does not match source %s"),
+ virCPUTypeToString(dst->type),
+ virCPUTypeToString(src->type));
+ }
goto cleanup;
}
if (src->mode != dst->mode) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU mode %s does not match source %s"),
- virCPUModeTypeToString(dst->mode),
- virCPUModeTypeToString(src->mode));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU mode %s does not match source %s"),
+ virCPUModeTypeToString(dst->mode),
+ virCPUModeTypeToString(src->mode));
+ }
goto cleanup;
}
if (src->arch != dst->arch) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU arch %s does not match source %s"),
- virArchToString(dst->arch),
- virArchToString(src->arch));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU arch %s does not match source %s"),
+ virArchToString(dst->arch),
+ virArchToString(src->arch));
+ }
goto cleanup;
}
if (STRNEQ_NULLABLE(src->model, dst->model)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU model %s does not match source %s"),
- NULLSTR(dst->model), NULLSTR(src->model));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU model %s does not match source %s"),
+ NULLSTR(dst->model), NULLSTR(src->model));
+ }
goto cleanup;
}
if (STRNEQ_NULLABLE(src->vendor, dst->vendor)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU vendor %s does not match source %s"),
- NULLSTR(dst->vendor), NULLSTR(src->vendor));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU vendor %s does not match source %s"),
+ NULLSTR(dst->vendor), NULLSTR(src->vendor));
+ }
goto cleanup;
}
if (STRNEQ_NULLABLE(src->vendor_id, dst->vendor_id)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU vendor id %s does not match source %s"),
- NULLSTR(dst->vendor_id), NULLSTR(src->vendor_id));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU vendor id %s does not match source %s"),
+ NULLSTR(dst->vendor_id), NULLSTR(src->vendor_id));
+ }
goto cleanup;
}
if (src->sockets != dst->sockets) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU sockets %d does not match source %d"),
- dst->sockets, src->sockets);
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU sockets %d does not match source %d"),
+ dst->sockets, src->sockets);
+ }
goto cleanup;
}
if (src->cores != dst->cores) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU cores %d does not match source %d"),
- dst->cores, src->cores);
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU cores %d does not match source %d"),
+ dst->cores, src->cores);
+ }
goto cleanup;
}
if (src->threads != dst->threads) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU threads %d does not match source %d"),
- dst->threads, src->threads);
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU threads %d does not match source %d"),
+ dst->threads, src->threads);
+ }
goto cleanup;
}
if (src->nfeatures != dst->nfeatures) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU feature count %zu does not match source %zu"),
- dst->nfeatures, src->nfeatures);
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU feature count %zu does not match source %zu"),
+ dst->nfeatures, src->nfeatures);
+ }
goto cleanup;
}
for (i = 0; i < src->nfeatures; i++) {
if (STRNEQ(src->features[i].name, dst->features[i].name)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU feature %s does not match source %s"),
- dst->features[i].name, src->features[i].name);
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU feature %s does not match source %s"),
+ dst->features[i].name, src->features[i].name);
+ }
goto cleanup;
}
if (src->features[i].policy != dst->features[i].policy) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target CPU feature policy %s does not match source %s"),
- virCPUFeaturePolicyTypeToString(dst->features[i].policy),
- virCPUFeaturePolicyTypeToString(src->features[i].policy));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Target CPU feature policy %s does not match source %s"),
+ virCPUFeaturePolicyTypeToString(dst->features[i].policy),
+ virCPUFeaturePolicyTypeToString(src->features[i].policy));
+ }
goto cleanup;
}
}
@@ -920,8 +947,10 @@ virCPUDefIsEqual(virCPUDefPtr src,
(src->cache && dst->cache &&
(src->cache->level != dst->cache->level ||
src->cache->mode != dst->cache->mode))) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
- _("Target CPU cache does not match source"));
+ if (reportError) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
+ _("Target CPU cache does not match source"));
+ }
goto cleanup;
}
diff --git a/src/conf/cpu_conf.h b/src/conf/cpu_conf.h
index 09438b68b..b0d891552 100644
--- a/src/conf/cpu_conf.h
+++ b/src/conf/cpu_conf.h
@@ -189,7 +189,8 @@ virCPUDefParseXML(xmlNodePtr node,
bool
virCPUDefIsEqual(virCPUDefPtr src,
- virCPUDefPtr dst);
+ virCPUDefPtr dst,
+ bool reportError);
char *
virCPUDefFormat(virCPUDefPtr def,
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 9cd39edf2..a61d293c9 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -20090,7 +20090,7 @@ virDomainDefCheckABIStabilityFlags(virDomainDefPtr src,
goto error;
}
- if (!virCPUDefIsEqual(src->cpu, dst->cpu))
+ if (!virCPUDefIsEqual(src->cpu, dst->cpu, true))
goto error;
if (!virSysinfoIsEqual(src->sysinfo, dst->sysinfo))
--
2.13.0
More information about the libvir-list
mailing list