Commit c43c64f8 authored by Ian Rogers's avatar Ian Rogers Committed by Namhyung Kim

perf vendor events intel: Update tsx_cycles_per_elision metrics

Update tsx_cycles_per_elision as per:
https://github.com/intel/perfmon/pull/116

Prefer the el-start event rather than cycles-t for detecting whether
the metric will work as HLE may be disabled. Remove the metric from
sapphirerapids that has no el-start event.
Signed-off-by: default avatarIan Rogers <irogers@google.com>
Reviewed-by: default avatarKan Liang <kan.liang@linux.intel.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Edward Baker <edward.baker@intel.com>
Cc: Zhengjun Xing <zhengjun.xing@linux.intel.com>
Link: https://lore.kernel.org/r/20231026003149.3287633-9-irogers@google.comSigned-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
parent c44c3118
...@@ -1837,7 +1837,7 @@ ...@@ -1837,7 +1837,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1525,7 +1525,7 @@ ...@@ -1525,7 +1525,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1821,7 +1821,7 @@ ...@@ -1821,7 +1821,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1551,7 +1551,7 @@ ...@@ -1551,7 +1551,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1944,13 +1944,6 @@ ...@@ -1944,13 +1944,6 @@
"MetricName": "tsx_aborted_cycles", "MetricName": "tsx_aborted_cycles",
"ScaleUnit": "100%" "ScaleUnit": "100%"
}, },
{
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)",
"MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision"
},
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of transactions.", "BriefDescription": "Number of cycles within a transaction divided by the number of transactions.",
"MetricExpr": "(cycles\\-t / tx\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / tx\\-start if has_event(cycles\\-t) else 0)",
......
...@@ -1473,7 +1473,7 @@ ...@@ -1473,7 +1473,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1781,7 +1781,7 @@ ...@@ -1781,7 +1781,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
...@@ -1551,7 +1551,7 @@ ...@@ -1551,7 +1551,7 @@
}, },
{ {
"BriefDescription": "Number of cycles within a transaction divided by the number of elisions.", "BriefDescription": "Number of cycles within a transaction divided by the number of elisions.",
"MetricExpr": "(cycles\\-t / el\\-start if has_event(cycles\\-t) else 0)", "MetricExpr": "(cycles\\-t / el\\-start if has_event(el\\-start) else 0)",
"MetricGroup": "transaction", "MetricGroup": "transaction",
"MetricName": "tsx_cycles_per_elision", "MetricName": "tsx_cycles_per_elision",
"ScaleUnit": "1cycles / elision" "ScaleUnit": "1cycles / elision"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment