[PATCH 3/3] perf vendor events amd: Remove "PerPkg" from Zen 6 memory controller metrics

[Sandipan Das]

"PerPkg" is an event attribute and has no effect on metrics. Remove it
from the Zen 6 memory controller metrics for consistency with the data
fabric metrics, which do not set it.

The "PerPkg" setting on the underlying events remains correct on Zen 6.
perf de-duplicates "PerPkg" counts by socket and die, and on AMD
processors these are the same domain. Each UMC PMU instance belongs to
a single IOD, so the CPUs it counts on always share the same socket and
die. De-duplication therefore never merges counts from different IODs,
unlike the single Data Fabric PMU, which spans both IODs in a socket.

Signed-off-by: Sandipan Das <sandipan.das@xxxxxxx>
---
 tools/perf/pmu-events/arch/x86/amdzen6/recommended.json | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/tools/perf/pmu-events/arch/x86/amdzen6/recommended.json b/tools/perf/pmu-events/arch/x86/amdzen6/recommended.json
index 853700a3a0de..1bc397f933f2 100644
--- a/tools/perf/pmu-events/arch/x86/amdzen6/recommended.json
+++ b/tools/perf/pmu-events/arch/x86/amdzen6/recommended.json
@@ -269,7 +269,6 @@
     "BriefDescription": "Memory controller data bus utilization.",
     "MetricExpr": "d_ratio(umc_data_slot_clks.all / 2, umc_mem_clk)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "100%"
   },
   {
@@ -277,7 +276,6 @@
     "BriefDescription": "Memory controller CAS command rate.",
     "MetricExpr": "d_ratio(umc_cas_cmd.all * 1000, umc_mem_clk)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1per_memclk"
   },
   {
@@ -285,7 +283,6 @@
     "BriefDescription": "Ratio of memory controller CAS commands for reads.",
     "MetricExpr": "d_ratio(umc_cas_cmd.rd, umc_cas_cmd.all)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "100%"
   },
   {
@@ -293,7 +290,6 @@
     "BriefDescription": "Ratio of memory controller CAS commands for writes.",
     "MetricExpr": "d_ratio(umc_cas_cmd.wr, umc_cas_cmd.all)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "100%"
   },
   {
@@ -301,7 +297,6 @@
     "BriefDescription": "Estimated memory read bandwidth.",
     "MetricExpr": "(umc_cas_cmd.rd * 64) / 1e6 / duration_time",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1MB/s"
   },
   {
@@ -309,7 +304,6 @@
     "BriefDescription": "Estimated memory write bandwidth.",
     "MetricExpr": "(umc_cas_cmd.wr * 64) / 1e6 / duration_time",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1MB/s"
   },
   {
@@ -317,7 +311,6 @@
     "BriefDescription": "Estimated combined memory bandwidth.",
     "MetricExpr": "(umc_cas_cmd.all * 64) / 1e6 / duration_time",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1MB/s"
   },
   {
@@ -325,7 +318,6 @@
     "BriefDescription": "Memory controller ACTIVATE command rate.",
     "MetricExpr": "d_ratio(umc_act_cmd.all * 1000, umc_mem_clk)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1per_memclk"
   },
   {
@@ -333,7 +325,6 @@
     "BriefDescription": "Memory controller PRECHARGE command rate.",
     "MetricExpr": "d_ratio(umc_pchg_cmd.all * 1000, umc_mem_clk)",
     "MetricGroup": "memory_controller",
-    "PerPkg": "1",
     "ScaleUnit": "1per_memclk"
   },
   {
-- 
2.53.0