Re: [libvirt] [PATCH v2 2/2] tests: sysinfotest: add new test case for S390
On 01/12/2018 06:38 AM, Bjoern Walk wrote:
Let's add a test case for S390 with CPU frequency information
available.
Test data is sampled from an IBM z13 system running kernel 4.14 on LPAR.
Reviewed-by: Boris Fiuczynski <fiuczy(a)linux.vnet.ibm.com>
Signed-off-by: Bjoern Walk <bwalk(a)linux.vnet.ibm.com>
---
tests/sysinfodata/s390-freqcpuinfo.data | 52 +++++++++
tests/sysinfodata/s390-freqsysinfo.data | 173 ++++++++++++++++++++++++++++++
tests/sysinfodata/s390-freqsysinfo.expect | 63 +++++++++++
tests/sysinfotest.c | 1 +
4 files changed, 289 insertions(+)
create mode 100644 tests/sysinfodata/s390-freqcpuinfo.data
create mode 100644 tests/sysinfodata/s390-freqsysinfo.data
create mode 100644 tests/sysinfodata/s390-freqsysinfo.expect
diff --git a/tests/sysinfodata/s390-freqcpuinfo.data
b/tests/sysinfodata/s390-freqcpuinfo.data
new file mode 100644
index 00000000..0edc371d
--- /dev/null
+++ b/tests/sysinfodata/s390-freqcpuinfo.data
@@ -0,0 +1,52 @@
+vendor_id : IBM/S390
+# processors : 8
+bogomips per cpu: 20325.00
+max thread id : 1
+features : esan3 zarch stfle msa ldisp eimm dfp edat etf3eh highgprs te vx sie
+facilities : 0 1 2 3 4 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26
27 28 30 31 32 33 34 35 36 37 40 41 42 43 44 45 46 47 48 49 50 51 52 53 55 57 64 65 66 67
68 69 70 71 72 73 75 76 77 78 80 128 129 131 132 142 143
The end of the "features" line has an additional space which git am
complains about and of course I know from my previous push, that the git
master would stop - I've cleaned it up as well as the %u instead of %n
in patch 1 that Pino pointed out.
Will push shortly -
Reviewed-by: John Ferlan <jferlan(a)redhat.com>
John