GNR cstate metric update; adding scale unit and metric group.

1perrytaylor · 1perrytaylor · commit 1c91f6187f51 · 2025-01-21T14:47:54.000-07:00
diff --git a/GNR/metrics/perf/graniterapids_metrics_perf.json b/GNR/metrics/perf/graniterapids_metrics_perf.json
@@ -313,14 +313,16 @@
     {
         "BriefDescription": "Percent of time that cores are in cstate C0 as observed by the power control unit (PCU)",
         "MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C0 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",
-        "MetricGroup": "",
+        "MetricGroup": "cpu_cstate",
         "MetricName": "cpu_cstate_c0"
+		"ScaleUnit": "100%"
     },
     {
         "BriefDescription": "Percent of time that cores are in cstate C6 as observed by the power control unit (PCU)",
         "MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",
-        "MetricGroup": "",
+        "MetricGroup": "cpu_cstate",
         "MetricName": "cpu_cstate_c6"
+		"ScaleUnit": "100%"
     },
     {
         "BriefDescription": "Total pipeline cost of Branch Misprediction related bottlenecks",

Original file line number	Diff line number	Diff line change
`@@ -313,14 +313,16 @@`
`313`	`313`	`{`
`314`	`314`	`"BriefDescription": "Percent of time that cores are in cstate C0 as observed by the power control unit (PCU)",`
`315`	`315`	`"MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C0 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",`
`316`		`- "MetricGroup": "",`
	`316`	`+ "MetricGroup": "cpu_cstate",`
`317`	`317`	`"MetricName": "cpu_cstate_c0"`
	`318`	`+ "ScaleUnit": "100%"`
`318`	`319`	`},`
`319`	`320`	`{`
`320`	`321`	`"BriefDescription": "Percent of time that cores are in cstate C6 as observed by the power control unit (PCU)",`
`321`	`322`	`"MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",`
`322`		`- "MetricGroup": "",`
	`323`	`+ "MetricGroup": "cpu_cstate",`
`323`	`324`	`"MetricName": "cpu_cstate_c6"`
	`325`	`+ "ScaleUnit": "100%"`
`324`	`326`	`},`
`325`	`327`	`{`
`326`	`328`	`"BriefDescription": "Total pipeline cost of Branch Misprediction related bottlenecks",`