[PATCH] cpu_s390: Implement getVendorForModel for IBM Z
Boris Fiuczynski
fiuczy at linux.ibm.com
Thu Nov 24 16:34:16 UTC 2022
Hi Thomas,
thanks for catching this. I agree with the changes but your patch is
missing the required changes in tests.
I have a patch added as attachment which you can use for squashing the
required changes into your patch.
With the tests fixed
Reviewed-by: Boris Fiuczynski <fiuczy at linux.ibm.com>
On 11/24/22 3:07 PM, Thomas Huth wrote:
> When running "virsh domcapabilities" on a s390x host, all the CPU
> models show up with vendor='unknown' - which sounds kind of weird
> since the vendor of these mainframe CPUs is well known: IBM.
> All CPUs starting with either "z" or "gen" match a real mainframe
> CPU by IBM, so let's return the string "IBM" for those now.
> The only remaining ones are now the artifical "qemu" and "max"
> models from QEMU itself, so it should be OK to get an "unknown"
> vendor for those two.
>
> Signed-off-by: Thomas Huth <thuth at redhat.com>
> ---
> src/cpu/cpu_s390.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/src/cpu/cpu_s390.c b/src/cpu/cpu_s390.c
> index d908a83928..7416ec6dc5 100644
> --- a/src/cpu/cpu_s390.c
> +++ b/src/cpu/cpu_s390.c
> @@ -109,6 +109,16 @@ virCPUs390ValidateFeatures(virCPUDef *cpu)
> }
>
>
> +static const char *
> +virCPUs390GetVendorForModel(const char *modelName)
> +{
> + if (modelName[0] == 'z' || STREQLEN(modelName, "gen", 3))
> + return "IBM";
> +
> + return NULL;
> +}
> +
> +
> struct cpuArchDriver cpuDriverS390 = {
> .name = "s390",
> .arch = archs,
> @@ -119,4 +129,5 @@ struct cpuArchDriver cpuDriverS390 = {
> .baseline = NULL,
> .update = virCPUs390Update,
> .validateFeatures = virCPUs390ValidateFeatures,
> + .getVendorForModel = virCPUs390GetVendorForModel,
> };
>
--
Mit freundlichen Grüßen/Kind regards
Boris Fiuczynski
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Gregor Pillen
Geschäftsführung: David Faller
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-To-be-squashed-into-patch.patch
Type: text/x-patch
Size: 27397 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20221124/ed882dc2/attachment-0001.bin>
More information about the libvir-list
mailing list