[libvirt] [PATCH 1/3] util: virhostcpu: parse frequency information on S390

[John Ferlan]

On 12/19/2017 05:08 AM, Bjoern Walk wrote:
> Since kernel version 4.7, processor frequency information is available
> on S390. Let's adjust the parser so this information shows up for virsh
> nodeinfo:
> 
>     # virsh nodeinfo
>     CPU model:           s390x
>     CPU(s):              8
>     CPU frequency:       5000 MHz
>     CPU socket(s):       1
>     Core(s) per socket:  8
>     Thread(s) per core:  1
>     NUMA cell(s):        1
>     Memory size:         16273908 KiB
> 
> Reviewed-by: Marc Hartmayer <mhartmay at linux.vnet.ibm.com>
> Reviewed-by: Boris Fiuczynski <fiuczy at linux.vnet.ibm.com>
> Signed-off-by: Bjoern Walk <bwalk at linux.vnet.ibm.com>
> ---
>  src/util/virhostcpu.c | 2 ++
>  1 file changed, 2 insertions(+)
> 

This one doesn't apply for me... seems we're missing something like
virHostCPUParseFrequency. Probably makes the next one moot too.
Although they both look reasonable even though I know nothing about S390
(other than perhaps one spelling variation ;-)).

John

> diff --git a/src/util/virhostcpu.c b/src/util/virhostcpu.c
> index 1c9ee592..24f0d936 100644
> --- a/src/util/virhostcpu.c
> +++ b/src/util/virhostcpu.c
> @@ -598,6 +598,8 @@ virHostCPUParseFrequency(FILE *cpuinfo,
>          prefix = "cpu MHz";
>      else if (ARCH_IS_PPC(arch))
>          prefix = "clock";
> +    else if (ARCH_IS_S390(arch))
> +        prefix = "cpu MHz dynamic";
>  
>      if (!prefix) {
>          VIR_WARN("%s is not supported by the %s parser",
>