Re: [PATCH v2] perf vendor events: Initial json/events list for power10 platform

2021-04-19 Thread Arnaldo Carvalho de Melo
Em Mon, Apr 19, 2021 at 10:38:46PM +1000, Michael Ellerman escreveu:
> Kajol Jain  writes:
> > Patch adds initial json/events for POWER10.
> 
> Acked-by: Michael Ellerman 

Thanks, applied.

- Arnaldo

 
> cheers
> 
> > Signed-off-by: Kajol Jain 
> > Tested-by: Paul A. Clarke 
> > Reviewed-by: Paul A. Clarke 
> > ---
> >  .../perf/pmu-events/arch/powerpc/mapfile.csv  |   1 +
> >  .../arch/powerpc/power10/cache.json   |  47 +++
> >  .../arch/powerpc/power10/floating_point.json  |   7 +
> >  .../arch/powerpc/power10/frontend.json| 217 +
> >  .../arch/powerpc/power10/locks.json   |  12 +
> >  .../arch/powerpc/power10/marked.json  | 147 +
> >  .../arch/powerpc/power10/memory.json  | 192 +++
> >  .../arch/powerpc/power10/others.json  | 297 ++
> >  .../arch/powerpc/power10/pipeline.json| 297 ++
> >  .../pmu-events/arch/powerpc/power10/pmc.json  |  22 ++
> >  .../arch/powerpc/power10/translation.json |  57 
> >  11 files changed, 1296 insertions(+)
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/cache.json
> >  create mode 100644 
> > tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/frontend.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/locks.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/marked.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/memory.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/others.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pipeline.json
> >  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pmc.json
> >  create mode 100644 
> > tools/perf/pmu-events/arch/powerpc/power10/translation.json
> >
> > ---
> > Changelog:
> > v1 -> v2
> > - Removed inconsistencies in "BriefDescription" field and make sure
> >   it will end with period without any space at the end.
> >   Suggested by : Paul A. Clarke  
> > - Added Tested-by and Reviewed-by tag.
> > ---
> > diff --git a/tools/perf/pmu-events/arch/powerpc/mapfile.csv 
> > b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> > index 229150e7ab7d..4abdfc3f9692 100644
> > --- a/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> > +++ b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> > @@ -15,3 +15,4 @@
> >  # Power8 entries
> >  004[bcd][[:xdigit:]]{4},1,power8,core
> >  004e[[:xdigit:]]{4},1,power9,core
> > +0080[[:xdigit:]]{4},1,power10,core
> > diff --git a/tools/perf/pmu-events/arch/powerpc/power10/cache.json 
> > b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
> > new file mode 100644
> > index ..95e33531fbc6
> > --- /dev/null
> > +++ b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
> > @@ -0,0 +1,47 @@
> > +[
> > +  {
> > +"EventCode": "1003C",
> > +"EventName": "PM_EXEC_STALL_DMISS_L2L3",
> > +"BriefDescription": "Cycles in which the oldest instruction in the 
> > pipeline was waiting for a load miss to resolve from either the local L2 or 
> > local L3."
> > +  },
> > +  {
> > +"EventCode": "34056",
> > +"EventName": "PM_EXEC_STALL_LOAD_FINISH",
> > +"BriefDescription": "Cycles in which the oldest instruction in the 
> > pipeline was finishing a load after its data was reloaded from a data 
> > source beyond the local L1; cycles in which the LSU was processing an 
> > L1-hit; cycles in which the NTF instruction merged with another load in the 
> > LMQ."
> > +  },
> > +  {
> > +"EventCode": "3006C",
> > +"EventName": "PM_RUN_CYC_SMT2_MODE",
> > +"BriefDescription": "Cycles when this thread's run latch is set and 
> > the core is in SMT2 mode."
> > +  },
> > +  {
> > +"EventCode": "300F4",
> > +"EventName": "PM_RUN_INST_CMPL_CONC",
> > +"BriefDescription": "PowerPC instructions completed by this thread 
> > when all threads in the core had the run-latch set."
> > +  },
> > +  {
> > +"EventCode": "4C016",
> > +"EventName": "PM_EXEC_STALL_DMISS_L2L3_CONFLICT",
> > +"BriefDescription": "Cycles in which the oldest instruction in the 
> > pipeline was waiting for a load miss to resolve from the local L2 or local 
> > L3, with a dispatch conflict."
> > +  },
> > +  {
> > +"EventCode": "4D014",
> > +"EventName": "PM_EXEC_STALL_LOAD",
> > +"BriefDescription": "Cycles in which the oldest instruction in the 
> > pipeline was a load instruction executing in the Load Store Unit."
> > +  },
> > +  {
> > +"EventCode": "4D016",
> > +"EventName": "PM_EXEC_STALL_PTESYNC",
> > +"BriefDescription": "Cycles in which the oldest instruction in the 
> > pipeline was a PTESYNC instruction executing in the Load Store Unit."
> > +  },
> > +  {
> > +"EventCode": "401EA",
> > +"EventName": "PM_THRESH_EXC_128",
> > +"BriefDescription": "Threshold counter exceeded a value of 128."
> > +  },
> > +  {
> > 

Re: [PATCH v2] perf vendor events: Initial json/events list for power10 platform

2021-04-19 Thread Michael Ellerman
Kajol Jain  writes:
> Patch adds initial json/events for POWER10.

Acked-by: Michael Ellerman 

cheers

> Signed-off-by: Kajol Jain 
> Tested-by: Paul A. Clarke 
> Reviewed-by: Paul A. Clarke 
> ---
>  .../perf/pmu-events/arch/powerpc/mapfile.csv  |   1 +
>  .../arch/powerpc/power10/cache.json   |  47 +++
>  .../arch/powerpc/power10/floating_point.json  |   7 +
>  .../arch/powerpc/power10/frontend.json| 217 +
>  .../arch/powerpc/power10/locks.json   |  12 +
>  .../arch/powerpc/power10/marked.json  | 147 +
>  .../arch/powerpc/power10/memory.json  | 192 +++
>  .../arch/powerpc/power10/others.json  | 297 ++
>  .../arch/powerpc/power10/pipeline.json| 297 ++
>  .../pmu-events/arch/powerpc/power10/pmc.json  |  22 ++
>  .../arch/powerpc/power10/translation.json |  57 
>  11 files changed, 1296 insertions(+)
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/cache.json
>  create mode 100644 
> tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/frontend.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/locks.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/marked.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/memory.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/others.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pipeline.json
>  create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pmc.json
>  create mode 100644 
> tools/perf/pmu-events/arch/powerpc/power10/translation.json
>
> ---
> Changelog:
> v1 -> v2
> - Removed inconsistencies in "BriefDescription" field and make sure
>   it will end with period without any space at the end.
>   Suggested by : Paul A. Clarke  
> - Added Tested-by and Reviewed-by tag.
> ---
> diff --git a/tools/perf/pmu-events/arch/powerpc/mapfile.csv 
> b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> index 229150e7ab7d..4abdfc3f9692 100644
> --- a/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> +++ b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> @@ -15,3 +15,4 @@
>  # Power8 entries
>  004[bcd][[:xdigit:]]{4},1,power8,core
>  004e[[:xdigit:]]{4},1,power9,core
> +0080[[:xdigit:]]{4},1,power10,core
> diff --git a/tools/perf/pmu-events/arch/powerpc/power10/cache.json 
> b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
> new file mode 100644
> index ..95e33531fbc6
> --- /dev/null
> +++ b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
> @@ -0,0 +1,47 @@
> +[
> +  {
> +"EventCode": "1003C",
> +"EventName": "PM_EXEC_STALL_DMISS_L2L3",
> +"BriefDescription": "Cycles in which the oldest instruction in the 
> pipeline was waiting for a load miss to resolve from either the local L2 or 
> local L3."
> +  },
> +  {
> +"EventCode": "34056",
> +"EventName": "PM_EXEC_STALL_LOAD_FINISH",
> +"BriefDescription": "Cycles in which the oldest instruction in the 
> pipeline was finishing a load after its data was reloaded from a data source 
> beyond the local L1; cycles in which the LSU was processing an L1-hit; cycles 
> in which the NTF instruction merged with another load in the LMQ."
> +  },
> +  {
> +"EventCode": "3006C",
> +"EventName": "PM_RUN_CYC_SMT2_MODE",
> +"BriefDescription": "Cycles when this thread's run latch is set and the 
> core is in SMT2 mode."
> +  },
> +  {
> +"EventCode": "300F4",
> +"EventName": "PM_RUN_INST_CMPL_CONC",
> +"BriefDescription": "PowerPC instructions completed by this thread when 
> all threads in the core had the run-latch set."
> +  },
> +  {
> +"EventCode": "4C016",
> +"EventName": "PM_EXEC_STALL_DMISS_L2L3_CONFLICT",
> +"BriefDescription": "Cycles in which the oldest instruction in the 
> pipeline was waiting for a load miss to resolve from the local L2 or local 
> L3, with a dispatch conflict."
> +  },
> +  {
> +"EventCode": "4D014",
> +"EventName": "PM_EXEC_STALL_LOAD",
> +"BriefDescription": "Cycles in which the oldest instruction in the 
> pipeline was a load instruction executing in the Load Store Unit."
> +  },
> +  {
> +"EventCode": "4D016",
> +"EventName": "PM_EXEC_STALL_PTESYNC",
> +"BriefDescription": "Cycles in which the oldest instruction in the 
> pipeline was a PTESYNC instruction executing in the Load Store Unit."
> +  },
> +  {
> +"EventCode": "401EA",
> +"EventName": "PM_THRESH_EXC_128",
> +"BriefDescription": "Threshold counter exceeded a value of 128."
> +  },
> +  {
> +"EventCode": "400F6",
> +"EventName": "PM_BR_MPRED_CMPL",
> +"BriefDescription": "A mispredicted branch completed. Includes direction 
> and target."
> +  }
> +]
> diff --git a/tools/perf/pmu-events/arch/powerpc/power10/floating_point.json 
> b/tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
> new file 

[PATCH v2] perf vendor events: Initial json/events list for power10 platform

2021-04-19 Thread Kajol Jain
Patch adds initial json/events for POWER10.

Signed-off-by: Kajol Jain 
Tested-by: Paul A. Clarke 
Reviewed-by: Paul A. Clarke 
---
 .../perf/pmu-events/arch/powerpc/mapfile.csv  |   1 +
 .../arch/powerpc/power10/cache.json   |  47 +++
 .../arch/powerpc/power10/floating_point.json  |   7 +
 .../arch/powerpc/power10/frontend.json| 217 +
 .../arch/powerpc/power10/locks.json   |  12 +
 .../arch/powerpc/power10/marked.json  | 147 +
 .../arch/powerpc/power10/memory.json  | 192 +++
 .../arch/powerpc/power10/others.json  | 297 ++
 .../arch/powerpc/power10/pipeline.json| 297 ++
 .../pmu-events/arch/powerpc/power10/pmc.json  |  22 ++
 .../arch/powerpc/power10/translation.json |  57 
 11 files changed, 1296 insertions(+)
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/cache.json
 create mode 100644 
tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/frontend.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/locks.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/marked.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/memory.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/others.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pipeline.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/pmc.json
 create mode 100644 tools/perf/pmu-events/arch/powerpc/power10/translation.json

---
Changelog:
v1 -> v2
- Removed inconsistencies in "BriefDescription" field and make sure
  it will end with period without any space at the end.
  Suggested by : Paul A. Clarke  
- Added Tested-by and Reviewed-by tag.
---
diff --git a/tools/perf/pmu-events/arch/powerpc/mapfile.csv 
b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
index 229150e7ab7d..4abdfc3f9692 100644
--- a/tools/perf/pmu-events/arch/powerpc/mapfile.csv
+++ b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
@@ -15,3 +15,4 @@
 # Power8 entries
 004[bcd][[:xdigit:]]{4},1,power8,core
 004e[[:xdigit:]]{4},1,power9,core
+0080[[:xdigit:]]{4},1,power10,core
diff --git a/tools/perf/pmu-events/arch/powerpc/power10/cache.json 
b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
new file mode 100644
index ..95e33531fbc6
--- /dev/null
+++ b/tools/perf/pmu-events/arch/powerpc/power10/cache.json
@@ -0,0 +1,47 @@
+[
+  {
+"EventCode": "1003C",
+"EventName": "PM_EXEC_STALL_DMISS_L2L3",
+"BriefDescription": "Cycles in which the oldest instruction in the 
pipeline was waiting for a load miss to resolve from either the local L2 or 
local L3."
+  },
+  {
+"EventCode": "34056",
+"EventName": "PM_EXEC_STALL_LOAD_FINISH",
+"BriefDescription": "Cycles in which the oldest instruction in the 
pipeline was finishing a load after its data was reloaded from a data source 
beyond the local L1; cycles in which the LSU was processing an L1-hit; cycles 
in which the NTF instruction merged with another load in the LMQ."
+  },
+  {
+"EventCode": "3006C",
+"EventName": "PM_RUN_CYC_SMT2_MODE",
+"BriefDescription": "Cycles when this thread's run latch is set and the 
core is in SMT2 mode."
+  },
+  {
+"EventCode": "300F4",
+"EventName": "PM_RUN_INST_CMPL_CONC",
+"BriefDescription": "PowerPC instructions completed by this thread when 
all threads in the core had the run-latch set."
+  },
+  {
+"EventCode": "4C016",
+"EventName": "PM_EXEC_STALL_DMISS_L2L3_CONFLICT",
+"BriefDescription": "Cycles in which the oldest instruction in the 
pipeline was waiting for a load miss to resolve from the local L2 or local L3, 
with a dispatch conflict."
+  },
+  {
+"EventCode": "4D014",
+"EventName": "PM_EXEC_STALL_LOAD",
+"BriefDescription": "Cycles in which the oldest instruction in the 
pipeline was a load instruction executing in the Load Store Unit."
+  },
+  {
+"EventCode": "4D016",
+"EventName": "PM_EXEC_STALL_PTESYNC",
+"BriefDescription": "Cycles in which the oldest instruction in the 
pipeline was a PTESYNC instruction executing in the Load Store Unit."
+  },
+  {
+"EventCode": "401EA",
+"EventName": "PM_THRESH_EXC_128",
+"BriefDescription": "Threshold counter exceeded a value of 128."
+  },
+  {
+"EventCode": "400F6",
+"EventName": "PM_BR_MPRED_CMPL",
+"BriefDescription": "A mispredicted branch completed. Includes direction 
and target."
+  }
+]
diff --git a/tools/perf/pmu-events/arch/powerpc/power10/floating_point.json 
b/tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
new file mode 100644
index ..e9b92f282d3c
--- /dev/null
+++ b/tools/perf/pmu-events/arch/powerpc/power10/floating_point.json
@@ -0,0 +1,7 @@
+[
+  {
+"EventCode": "4016E",
+"EventName": "PM_THRESH_NOT_MET",
+"BriefDescription": "Threshold counter did not meet threshold."
+  }