Skip to content

Commit e8bd238

Browse files
authored
Merge pull request #275 from intel/cstate_metric_update
GNR cstate metric update
2 parents 61052f1 + 23d3e1a commit e8bd238

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

GNR/metrics/perf/graniterapids_metrics_perf.json

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -311,18 +311,16 @@
311311
"ScaleUnit": "1MB/s"
312312
},
313313
{
314-
"BriefDescription": "Percent of time that cores are in cstate C0 as observed by the power control unit (PCU)",
315-
"MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C0 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",
314+
"BriefDescription": "The average number of cores that are in cstate C0 as observed by the power control unit (PCU)",
315+
"MetricExpr": "(UNC_P_POWER_STATE_OCCUPANCY_CORES_C0 / UNC_P_CLOCKTICKS[0]) * #num_packages",
316316
"MetricGroup": "cpu_cstate",
317-
"MetricName": "cpu_cstate_c0",
318-
"ScaleUnit": "100%"
317+
"MetricName": "cpu_cstate_c0"
319318
},
320319
{
321-
"BriefDescription": "Percent of time that cores are in cstate C6 as observed by the power control unit (PCU)",
322-
"MetricExpr": "100 * ((UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / (pcu_0@UNC_P_CLOCKTICKS@ / #num_packages ) ) / ( #num_cores / #num_packages * #num_packages ) )",
320+
"BriefDescription": "The average number of cores are in cstate C6 as observed by the power control unit (PCU)",
321+
"MetricExpr": "(UNC_P_POWER_STATE_OCCUPANCY_CORES_C6 / UNC_P_CLOCKTICKS[0]) * #num_packages",
323322
"MetricGroup": "cpu_cstate",
324-
"MetricName": "cpu_cstate_c6",
325-
"ScaleUnit": "100%"
323+
"MetricName": "cpu_cstate_c6"
326324
},
327325
{
328326
"BriefDescription": "Total pipeline cost of Branch Misprediction related bottlenecks",

0 commit comments

Comments
 (0)