Ubuntu17.10 - perf: Update Power9 PMU event JSON files

Bug #1708630 reported by bugproxy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Ubuntu-power-systems project
Fix Released
Medium
Canonical Kernel Team
linux (Ubuntu)
Fix Released
Medium
Seth Forshee

Bug Description

== Comment: #0 - Sukadev Bhattiprolu <email address hidden> - 2017-08-03 20:36:26 ==
+++ This bug was initially created as a clone of Bug #157304 +++

---Problem Description---
BZ 150738 added Power9 PMU event lists to the Linux kernel tree.
We need to update the event lists to include several more events
as well as clean up the event descriptions of some events.

The patches for these updates were posted to LKML
     https://lkml.org/lkml/2017/8/2/699
with:
    Subject: Re: [GIT PULL] Please pull JSON files for Power9 PMU events

and were merged into Arnaldo's "perf-core" tree as git commits:

https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=perf/core&id=864c572433dedfb5c36db79c4ceb6dfb80b3344b
("perf vendor events powerpc: remove suffix in mapfile")

and

https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=perf/core&id=b547e94b919c84de22052935a77de9c8a97d3418
("perf vendor events powerpc: Update POWER9 events")

We will need to include those updates into

Contact Information = <email address hidden>

---uname output---
na

Machine Type = Powre9

---Debugger---
A debugger is not configured

---Steps to Reproduce---
 'perf list pmu' , 'perf stat' and 'perf record' are unable to list/use some Power9 PMU events as they are missing from the kernel source tree.

Stack trace output:
 no

Oops output:
 no

Userspace tool common name: perf

Userspace rpm: linux-tools

The userspace tool has the following bit modes: 64-bit

System Dump Info:
  The system is not configured to capture a system dump.

Userspace tool obtained from project website: na

*Additional Instructions for <email address hidden>:
-Attach sysctl -a output output to the bug.
-Attach ltrace and strace of userspace application.

bugproxy (bugproxy)
tags: added: architecture-ppc64le bugnameltc-157305 severity-medium targetmilestone-inin1710
Changed in ubuntu:
assignee: nobody → Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage)
affects: ubuntu → linux (Ubuntu)
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
assignee: nobody → Canonical Kernel Team (canonical-kernel-team)
importance: Undecided → Medium
Changed in linux (Ubuntu):
importance: Undecided → Medium
tags: added: arty kernel-da-key
tags: added: artful
removed: arty
Seth Forshee (sforshee)
Changed in linux (Ubuntu):
assignee: Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage) → Seth Forshee (sforshee)
status: New → In Progress
Revision history for this message
Seth Forshee (sforshee) wrote :

I tried cherry picking these onto 4.13, "perf vendor events powerpc: remove suffix in mapfile" I could do with a little backporting but "perf vendor events powerpc: Update POWER9 events" is trying to update files which don't exist in 4.13. So we're obviously missing some prerequisites, can you provide a full list of commits needed for 4.12/4.13?

Changed in linux (Ubuntu):
status: In Progress → Incomplete
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
status: New → Incomplete
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2017-08-14 14:57 EDT-------
I have added a dependency on the earlier bug, 150738, that added
the commits that are pre-requisite. Those commits are not in 4.13.

Specifically, from: comment #7 from the above bug:

---
Submitted a pull request for the P9 PMU event lists:

https://lkml.org/lkml/2017/7/12/415

which reads "[GIT PULL] Please pull JSON files for Power9 PMU events"
and which consists of these 2 patches:

Sukadev Bhattiprolu (2):
perf vendor events: Add POWER9 PMU events
perf vendor events: Add POWER9 PVRs to mapfile
---

Revision history for this message
Seth Forshee (sforshee) wrote :

Thanks. In addition to those I went ahead and applied "perf pmu-events: Support additional POWER8+ PVR in mapfile" for completeness and to make everything a clean cherry pick.

Applied to artful 4.12 and 4.13 branches.

Changed in linux (Ubuntu):
status: Incomplete → Fix Committed
Manoj Iyer (manjo)
Changed in ubuntu-power-systems:
status: Incomplete → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (13.1 KiB)

This bug was fixed in the package linux - 4.12.0-12.13

---------------
linux (4.12.0-12.13) artful; urgency=low

  * Artful update to v4.12.8 stable release (LP: #1711368)
    - mm: ratelimit PFNs busy info message
    - mm: fix list corruptions on shmem shrinklist
    - futex: Remove unnecessary warning from get_futex_key
    - xtensa: fix cache aliasing handling code for WT cache
    - xtensa: mm/cache: add missing EXPORT_SYMBOLs
    - xtensa: don't limit csum_partial export by CONFIG_NET
    - xfs: Fix leak of discard bio
    - pinctrl: armada-37xx: Fix number of pin in south bridge
    - mtd: nand: atmel: Fix DT backward compatibility in pmecc.c
    - mtd: nand: Fix timing setup for NANDs that do not support SET FEATURES
    - mtd: nand: Declare tBERS, tR and tPROG as u64 to avoid integer overflow
    - iscsi-target: fix memory leak in iscsit_setup_text_cmd()
    - iscsi-target: Fix iscsi_np reset hung task during parallel delete
    - usb-storage: fix deadlock involving host lock and scsi_done
    - target: Fix node_acl demo-mode + uncached dynamic shutdown regression
    - fuse: initialize the flock flag in fuse_file on allocation
    - i2c: designware: Some broken DSTDs use 1MiHz instead of 1MHz
    - nand: fix wrong default oob layout for small pages using soft ecc
    - mmc: mmc: correct the logic for setting HS400ES signal voltage
    - nfs/flexfiles: fix leak of nfs4_ff_ds_version arrays
    - drm/bridge: tc358767: fix probe without attached output node
    - drm/etnaviv: Fix off-by-one error in reloc checking
    - drm/i915: Fix out-of-bounds array access in bdw_load_gamma_lut
    - USB: serial: option: add D-Link DWM-222 device ID
    - USB: serial: cp210x: add support for Qivicon USB ZigBee dongle
    - USB: serial: pl2303: add new ATEN device id
    - usb: musb: fix tx fifo flush handling again
    - USB: hcd: Mark secondary HCD as dead if the primary one died
    - staging:iio:resolver:ad2s1210 fix negative IIO_ANGL_VEL read
    - iio: aspeed-adc: wait for initial sequence.
    - iio: accel: st_accel: add SPI-3wire support
    - iio: accel: bmc150: Always restore device to normal mode after suspend-
      resume
    - iio: pressure: st_pressure_core: disable multiread by default for LPS22HB
    - iio: light: tsl2563: use correct event code
    - iio: adc: Revert "axp288: Drop bogus AXP288_ADC_TS_PIN_CTRL register
      modifications"
    - staging: comedi: comedi_fops: do not call blocking ops when !TASK_RUNNING
    - uas: Add US_FL_IGNORE_RESIDUE for Initio Corporation INIC-3069
    - firmware: fix batched requests - wake all waiters
    - firmware: fix batched requests - send wake up on failure on direct lookups
    - firmware: avoid invalid fallback aborts by using killable wait
    - block: Make blk_mq_delay_kick_requeue_list() rerun the queue at a quiet time
    - usb: gadget: udc: renesas_usb3: Fix usb_gadget_giveback_request() calling
    - usb: renesas_usbhs: Fix UGCTRL2 value for R-Car Gen3
    - USB: Check for dropped connection before switching to full speed
    - usb: core: unlink urbs from the tail of the endpoint's urb_list
    - usb: quirks: Add no-lpm quirk for Moshi USB to Ethernet Adapter
    - usb:xhci:Add ...

Changed in linux (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

The patch mentioned in bug 1714571 may be needed on top of the SAUCE patch:
UBUNTU: SAUCE: perf vendor events powerpc: Update POWER9 events

Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

Attaching patch mentioned in bug 1714571 that I formatted from perf-tip.

Changed in ubuntu-power-systems:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.