Re: [PATCH v2 2/2] tests: sysinfotest: add new test case for S390

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Bjoern Walk <bwalk@xxxxxxxxxxxxxxxxxx>
> ---
>  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@xxxxxxxxxx>

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux