Re: [libvirt] [PATCH 1/3] util: virhostcpu: parse frequency information on S390
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(a)linux.vnet.ibm.com>
Reviewed-by: Boris Fiuczynski <fiuczy(a)linux.vnet.ibm.com>
Signed-off-by: Bjoern Walk <bwalk(a)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",