External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-30 13:37:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

cpu_s390: Implement virCPUValidateFeatures

Browse Source 
Only feature policy is checked on s390, which was previously done in
virCPUUpdate, but that's not the correct place for the check once we
have virCPUValidateFeatures.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
This commit is contained in:
Jiri Denemark 2017-09-14 16:09:35 +02:00
parent 3f0193f7e5
commit 20edbad776
1 changed files with 21 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/cpu/cpu_s390.c
View File

@ -78,14 +78,6 @@ virCPUs390Update(virCPUDefPtr guest,
goto cleanup;
for (i = 0; i < guest->nfeatures; i++) {
if (guest->features[i].policy == VIR_CPU_FEATURE_OPTIONAL) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
_("only cpu feature policies 'require' and "
"'disable' are supported for %s"),
guest->features[i].name);
goto cleanup;
}
if (virCPUDefUpdateFeature(updated,
guest->features[i].name,
guest->features[i].policy) < 0)
@ -102,6 +94,26 @@ virCPUs390Update(virCPUDefPtr guest,
return ret;
}
static int
virCPUs390ValidateFeatures(virCPUDefPtr cpu)
{
size_t i;
for (i = 0; i < cpu->nfeatures; i++) {
if (cpu->features[i].policy == VIR_CPU_FEATURE_OPTIONAL) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
_("only cpu feature policies 'require' and "
"'disable' are supported for %s"),
cpu->features[i].name);
return -1;
}
}
return 0;
}
struct cpuArchDriver cpuDriverS390 = {
.name = "s390",
.arch = archs,
@ -111,4 +123,5 @@ struct cpuArchDriver cpuDriverS390 = {
.encode = NULL,
.baseline = NULL,
.update = virCPUs390Update,
.validateFeatures = virCPUs390ValidateFeatures,
};