Comment 0 for bug 1709964

Revision history for this message
bugproxy (bugproxy) wrote :

---Problem Description---
Unable to count pm_run_inst_cmpl using perf tool on boston dd2.0

root@isvbos3:~# perf stat -e r500fa,r600f4 -a sleep 1

 Performance counter stats for 'system wide':

   <not supported> r500fa
         8,162,912 r600f4

       1.001562716 seconds time elapsed

---uname output---
Linux isvbos3 4.10.0-29-generic #33~16.04.1-Ubuntu SMP Tue Jul 25 18:17:06 UTC 2017 ppc64le ppc64le ppc64le GNU/Linux

Machine Type = p9 dd2.0

---Steps to Reproduce---
 root@isvbos3:~# perf stat -e r500fa,r600f4 -a sleep 1

 Performance counter stats for 'system wide':

   <not supported> r500fa
         8,162,912 r600f4

       1.001562716 seconds time elapsed

> Also we need this upstream fix 8c218578fcbbbdb10416c8614658bf32e3bf1655 in
> the distro.

$ git log 8c218578fcbbbdb10416c8614658 -1
commit 8c218578fcbbbdb10416c8614658bf32e3bf1655
Author: Madhavan Srinivasan <email address hidden>
Date: Fri May 26 13:38:27 2017 +0530

    powerpc/perf: Fix Power9 test_adder fields

    Commit 8d911904f3ce4 ('powerpc/perf: Add restrictions to PMC5 in power9 DD1')
    was added to restrict the use of PMC5 in Power9 DD1. Intention was to disable
    the use of PMC5 using raw event code. But instead of updating the
    power9_isa207_pmu structure (used on DD1), the commit incorrectly updated the
    power9_pmu structure. Fix it.

    Fixes: 8d911904f3ce ("powerpc/perf: Add restrictions to PMC5 in power9 DD1")
    Reported-by: Shriya <email address hidden>
    Signed-off-by: Madhavan Srinivasan <email address hidden>
    Tested-by: Shriya <email address hidden>
    Signed-off-by: Michael Ellerman <email address hidden>

$ git describe --contains 8c218578fcbbbdb
v4.12-rc7~9^2~13
$

Commit 8c218578fc is available with kernel 4.12-rc7 afterwards.

I don't see this fix in the Ubuntu 16.04.3 Zesty tree (the P9 hwe kernel, 4.10 based).

Need to mirror to Canonical to request they pull in:
8c218578fcbbbdb10416c8614658bf32e3bf1655