mbox series

[SRU,F,0/1] perf vendor events s390: Add new deflate counters for IBM z15

Message ID 20200722171055.15296-1-sultan.alsawaf@canonical.com
Headers show
Series perf vendor events s390: Add new deflate counters for IBM z15 | expand

Message

Sultan Alsawaf July 22, 2020, 5:10 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1888551

SRU Justification:
==================

[Impact]

* With perf from Ubuntu 20.04 on IBM z15 hardware, some counters reported with lscpumf are not usable with 'perf stat -e'.

* 'lscpumf -c' provides a list of available counters, but the following are not usable: DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR

* 'perf list' is also not offering these counters.

* On the kernel side this is fixed with the following upstream accepted commit:

[Fix]

* e7950166e40271c025e0eec348cdf5c63ac734fa "perf vendor events s390: Add new deflate counters for IBM z15"

[Test Case]

* Requires the fix/patch of the kernel (d68d5d51dc898895b4e15bea52e5668ca9e76180 "s390/cpum_cf: Add new extended counters for IBM z15")

* An IBM z15 (or LinuxONE III) system with Ubuntu 20.04 installed in LPAR is needed where the counters are enabled in the activation profile.

* Use 'perf list' to determine if the counters DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR are listed

* Use 'perf stat -e' to enable amd make use of these counters.

[Regression Potential]

* The regression can be considered as low, since:

* it's architecture specific (s390x)

* limited to IBM z15 hardware

* and only additional (s390x) hardware counters are defined and added.

[Other]

* Since these patches are upstream in kernel 5.7 they will automatically land in groovy.

Thomas Richter (1):
  perf vendor events s390: Add new deflate counters for IBM z15

 .../pmu-events/arch/s390/cf_z15/crypto6.json  |  8 ++---
 .../pmu-events/arch/s390/cf_z15/extended.json | 30 ++++++++++++++++++-
 2 files changed, 33 insertions(+), 5 deletions(-)

Comments

Stefan Bader July 23, 2020, 7:18 a.m. UTC | #1
On 22.07.20 19:10, Sultan Alsawaf wrote:
> BugLink: https://bugs.launchpad.net/bugs/1888551
> 
> SRU Justification:
> ==================
> 
> [Impact]
> 
> * With perf from Ubuntu 20.04 on IBM z15 hardware, some counters reported with lscpumf are not usable with 'perf stat -e'.
> 
> * 'lscpumf -c' provides a list of available counters, but the following are not usable: DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR
> 
> * 'perf list' is also not offering these counters.
> 
> * On the kernel side this is fixed with the following upstream accepted commit:
> 
> [Fix]
> 
> * e7950166e40271c025e0eec348cdf5c63ac734fa "perf vendor events s390: Add new deflate counters for IBM z15"
> 
> [Test Case]
>

To be checked before applying:

> * Requires the fix/patch of the kernel (d68d5d51dc898895b4e15bea52e5668ca9e76180 "s390/cpum_cf: Add new extended counters for IBM z15")

> 
> * An IBM z15 (or LinuxONE III) system with Ubuntu 20.04 installed in LPAR is needed where the counters are enabled in the activation profile.
> 
> * Use 'perf list' to determine if the counters DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR are listed
> 
> * Use 'perf stat -e' to enable amd make use of these counters.
> 
> [Regression Potential]
> 
> * The regression can be considered as low, since:
> 
> * it's architecture specific (s390x)
> 
> * limited to IBM z15 hardware
> 
> * and only additional (s390x) hardware counters are defined and added.
> 
> [Other]
> 
> * Since these patches are upstream in kernel 5.7 they will automatically land in groovy.
> 
> Thomas Richter (1):
>   perf vendor events s390: Add new deflate counters for IBM z15
> 
>  .../pmu-events/arch/s390/cf_z15/crypto6.json  |  8 ++---
>  .../pmu-events/arch/s390/cf_z15/extended.json | 30 ++++++++++++++++++-
>  2 files changed, 33 insertions(+), 5 deletions(-)
> 

For whoever applies this, make sure to check the dependency mentioned above.

Acked-by: Stefan Bader <stefan.bader@canonical.com>
Kleber Sacilotto de Souza July 27, 2020, 8:46 a.m. UTC | #2
On 22.07.20 19:10, Sultan Alsawaf wrote:
> BugLink: https://bugs.launchpad.net/bugs/1888551
> 
> SRU Justification:
> ==================
> 
> [Impact]
> 
> * With perf from Ubuntu 20.04 on IBM z15 hardware, some counters reported with lscpumf are not usable with 'perf stat -e'.
> 
> * 'lscpumf -c' provides a list of available counters, but the following are not usable: DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR
> 
> * 'perf list' is also not offering these counters.
> 
> * On the kernel side this is fixed with the following upstream accepted commit:
> 
> [Fix]
> 
> * e7950166e40271c025e0eec348cdf5c63ac734fa "perf vendor events s390: Add new deflate counters for IBM z15"
> 
> [Test Case]
> 
> * Requires the fix/patch of the kernel (d68d5d51dc898895b4e15bea52e5668ca9e76180 "s390/cpum_cf: Add new extended counters for IBM z15")
> 
> * An IBM z15 (or LinuxONE III) system with Ubuntu 20.04 installed in LPAR is needed where the counters are enabled in the activation profile.
> 
> * Use 'perf list' to determine if the counters DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR are listed
> 
> * Use 'perf stat -e' to enable amd make use of these counters.
> 
> [Regression Potential]
> 
> * The regression can be considered as low, since:
> 
> * it's architecture specific (s390x)
> 
> * limited to IBM z15 hardware
> 
> * and only additional (s390x) hardware counters are defined and added.
> 
> [Other]
> 
> * Since these patches are upstream in kernel 5.7 they will automatically land in groovy.
> 
> Thomas Richter (1):
>   perf vendor events s390: Add new deflate counters for IBM z15
> 
>  .../pmu-events/arch/s390/cf_z15/crypto6.json  |  8 ++---
>  .../pmu-events/arch/s390/cf_z15/extended.json | 30 ++++++++++++++++++-
>  2 files changed, 33 insertions(+), 5 deletions(-)
> 

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Kelsey Skunberg Aug. 6, 2020, 10:52 p.m. UTC | #3
Applied to Focal/master-next. Verified the following patch is applied to
Focal/master-next already as requested:

7da7c2da2f65 ("s390/cpum_cf: Add new extended counters for IBM z15")

-Kelsey

On 2020-07-22 10:10:54 , Sultan Alsawaf wrote:
> BugLink: https://bugs.launchpad.net/bugs/1888551
> 
> SRU Justification:
> ==================
> 
> [Impact]
> 
> * With perf from Ubuntu 20.04 on IBM z15 hardware, some counters reported with lscpumf are not usable with 'perf stat -e'.
> 
> * 'lscpumf -c' provides a list of available counters, but the following are not usable: DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR
> 
> * 'perf list' is also not offering these counters.
> 
> * On the kernel side this is fixed with the following upstream accepted commit:
> 
> [Fix]
> 
> * e7950166e40271c025e0eec348cdf5c63ac734fa "perf vendor events s390: Add new deflate counters for IBM z15"
> 
> [Test Case]
> 
> * Requires the fix/patch of the kernel (d68d5d51dc898895b4e15bea52e5668ca9e76180 "s390/cpum_cf: Add new extended counters for IBM z15")
> 
> * An IBM z15 (or LinuxONE III) system with Ubuntu 20.04 installed in LPAR is needed where the counters are enabled in the activation profile.
> 
> * Use 'perf list' to determine if the counters DFLT_ACCESS, DFLT_CYCLES, DFLT_CC, DFLT_CCERROR are listed
> 
> * Use 'perf stat -e' to enable amd make use of these counters.
> 
> [Regression Potential]
> 
> * The regression can be considered as low, since:
> 
> * it's architecture specific (s390x)
> 
> * limited to IBM z15 hardware
> 
> * and only additional (s390x) hardware counters are defined and added.
> 
> [Other]
> 
> * Since these patches are upstream in kernel 5.7 they will automatically land in groovy.
> 
> Thomas Richter (1):
>   perf vendor events s390: Add new deflate counters for IBM z15
> 
>  .../pmu-events/arch/s390/cf_z15/crypto6.json  |  8 ++---
>  .../pmu-events/arch/s390/cf_z15/extended.json | 30 ++++++++++++++++++-
>  2 files changed, 33 insertions(+), 5 deletions(-)
> 
> -- 
> 2.20.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team