Messages in this thread | | | Date | Fri, 6 May 2022 22:34:05 -0700 | Subject | [PATCH 0/5] Revert metric hybrid events, fix tools events | From | Ian Rogers <> |
| |
Hybrid metrics place a PMU at the end of the parse string. This is also where tool events are placed. The behavior of the parse string isn't clear and so revert the change for now.
commit 75eafc970bd9 ("perf list: Print all available tool events") had an off-by-1 error. Try to separate the tool event enum logic from evsels. Also add a for loop helper to simplify working with a variety of tool events.
Add support to metrics for more tool events than just duration_time. Make the sharing logic only look to add for sharing potentially used tool events, found by scanning the list of metrics.
Ian Rogers (5): Revert "perf stat: Support metrics with hybrid events" perf evsel: Constify a few arrays perf evsel: Add tool event helpers perf metrics: Support all tool events perf metrics: Don't add all tool events for sharing
tools/perf/tests/evsel-roundtrip-name.c | 2 +- tools/perf/util/evsel.c | 53 +++- tools/perf/util/evsel.h | 22 +- tools/perf/util/metricgroup.c | 373 +++++++----------------- tools/perf/util/parse-events.c | 2 +- tools/perf/util/stat-display.c | 8 +- tools/perf/util/stat-shadow.c | 27 +- 7 files changed, 180 insertions(+), 307 deletions(-)
-- 2.36.0.512.ge40c2bad7a-goog
| |