Re: [PATCH] perf/stat: Handle different PMU names with common prefix

2018-10-29 Thread Sebastien Boisvert
On 2018-10-29 8:18 a.m., Thomas-Mich Richter wrote: > Yes, mistake on my side... > but it is the same type of error , regardless if 6 or 7 characters are > compared: > > strncmp("cpmu_cf", "cpum_cf_diag", 7) > > returns 0 when it should not. The device names of both PMUs are different.

Re: [PATCH] perf/stat: Handle different PMU names with common prefix

2018-10-29 Thread Thomas-Mich Richter
On 10/26/2018 05:04 PM, Sébastien Boisvert wrote: > On 2018-10-23 11:16 a.m., Thomas Richter wrote: >> On s390 the CPU Measurement Facility for counters now supports >> 2 PMUs named cpum_cf (CPU Measurement Facility for counters) and >> cpum_cf_diag (CPU Measurement Facility for diagnostic counters

Re: [PATCH] perf/stat: Handle different PMU names with common prefix

2018-10-26 Thread Sébastien Boisvert
On 2018-10-23 11:16 a.m., Thomas Richter wrote: > On s390 the CPU Measurement Facility for counters now supports > 2 PMUs named cpum_cf (CPU Measurement Facility for counters) and > cpum_cf_diag (CPU Measurement Facility for diagnostic counters) > for one and the same CPU. > > Running command > >

[PATCH] perf/stat: Handle different PMU names with common prefix

2018-10-23 Thread Thomas Richter
On s390 the CPU Measurement Facility for counters now supports 2 PMUs named cpum_cf (CPU Measurement Facility for counters) and cpum_cf_diag (CPU Measurement Facility for diagnostic counters) for one and the same CPU. Running command [root@s35lp76 perf]# ./perf stat -e tx_c_tend \ -- ~/