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]

 



John Ferlan <jferlan@xxxxxxxxxx> [2018-01-12, 08:04AM -0500]:
> 
> 
> On 01/12/2018 07:44 AM, Pino Toscano wrote:
> > On Friday, 12 January 2018 12:38:02 CET 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>
> >> ---
> > 
> > LGTM, just a couple of notes that came into my mind when reading the
> > existing sysinfodata for s390.
> > 
> >> 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
> >> [...]
> >> +processor 0: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 1: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 2: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 3: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 4: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 5: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 6: version = 00,  identification = 145F07,  machine = 2964
> >> +processor 7: version = 00,  identification = 145F07,  machine = 2964
> > 
> > All the various "identification", and ...
> > 
> >> diff --git a/tests/sysinfodata/s390-freqsysinfo.data b/tests/sysinfodata/s390-freqsysinfo.data
> >> new file mode 100644
> >> index 00000000..1941ef0b
> >> --- /dev/null
> >> +++ b/tests/sysinfodata/s390-freqsysinfo.data
> >> @@ -0,0 +1,173 @@
> >> +Manufacturer:         IBM
> >> +Type:                 2964
> >> +Model:                704              NC9
> >> +Sequence Code:        0000000000085F07
> > 
> > .. this value are replaced by what look like dummy values, so maybe
> > the same should be done for this data too (in case they are sensitive
> > information)?
> > 
> 
> I could adjust Sequence Code here too if desired.  I see that
> tests/sysinfodata/s390sysinfo.data used 12345.
> 
> Let me know...

Sure, why not.

Thanks for reviewing and cleaning up and pushing.

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

Attachment: signature.asc
Description: PGP signature

--
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