Merge pull request #341 from edwarddavidbaker/fixup-spr-metric

1perrytaylor · web-flow · commit f904f93d13de · 2025-09-25T09:48:21.000-07:00
SPR: Trim extra @ sign from C6 metric
diff --git a/SPR/metrics/perf/sapphirerapids_metrics_perf.json b/SPR/metrics/perf/sapphirerapids_metrics_perf.json
@@ -404,7 +404,7 @@
     },
     {
         "BriefDescription": "The average number of cores are in cstate C6 as observed by the power control unit (PCU)",
-        "MetricExpr": "( UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / @UNC_P_CLOCKTICKS ) * #num_packages",
+        "MetricExpr": "( UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / UNC_P_CLOCKTICKS ) * #num_packages",
         "MetricGroup": "cpu_cstate",
         "MetricName": "cpu_cstate_c6"
     }

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,7 @@`
`404`	`404`	`},`
`405`	`405`	`{`
`406`	`406`	`"BriefDescription": "The average number of cores are in cstate C6 as observed by the power control unit (PCU)",`
`407`		`- "MetricExpr": "( UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / @UNC_P_CLOCKTICKS ) * #num_packages",`
	`407`	`+ "MetricExpr": "( UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / UNC_P_CLOCKTICKS ) * #num_packages",`
`408`	`408`	`"MetricGroup": "cpu_cstate",`
`409`	`409`	`"MetricName": "cpu_cstate_c6"`
`410`	`410`	`}`