[libvirt] [PATCH v2 01/11] s390: Cpu driver support for update and compare
Jiri Denemark
jdenemar at redhat.com
Thu Dec 15 11:39:35 UTC 2016
On Fri, Dec 09, 2016 at 14:38:30 -0500, Jason J. Herne wrote:
> Implement compare for s390. Required to test the guest against the host for
> guest cpu model runnability checking. We always return IDENTICAL to bypass
> Libvirt's checking. s390 will rely on Qemu to perform the runnability checking.
>
> Implement update for s390. required to support use of cpu "host-model" mode.
>
> Signed-off-by: Jason J. Herne <jjherne at linux.vnet.ibm.com>
> ---
> po/POTFILES.in | 1 +
> src/cpu/cpu_s390.c | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
> 2 files changed, 72 insertions(+), 2 deletions(-)
...
> diff --git a/src/cpu/cpu_s390.c b/src/cpu/cpu_s390.c
> index 04a6bea..3efc948 100644
> --- a/src/cpu/cpu_s390.c
> +++ b/src/cpu/cpu_s390.c
...
> +static int
> +virCPUs390Update(virCPUDefPtr guest,
> + const virCPUDef *host)
...
> + 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);
Strange indentation and inconsistent usage of an upper case letter in
the error message.
> + goto cleanup;
> + }
> +
...
ACK with the minor issue fixed.
Jirka
More information about the libvir-list
mailing list