diff options
author | James Clark <james.clark@linaro.org> | 2025-02-12 16:38:56 +0000 |
---|---|---|
committer | Namhyung Kim <namhyung@kernel.org> | 2025-02-12 19:34:56 -0800 |
commit | 615ec00b06f78912c370b372426190768402a5b9 (patch) | |
tree | 68614c40f54bce30b1b2e57adc09b34d310bac48 /tools/perf/util/scripting-engines/trace-event-python.c | |
parent | ee8aef2d232142e5fdfed9c16132815969a0bf81 (diff) |
perf tests: Fix Tool PMU test segfault
tool_pmu__event_to_str() now handles skipped events by returning NULL,
so it's wrong to re-check for a skip on the resulting string. Calling
tool_pmu__skip_event() with a NULL string results in a segfault so
remove the unnecessary skip to fix it:
$ perf test -vv "parsing with PMU name"
12.2: Parsing with PMU name:
...
---- unexpected signal (11) ----
12.2: Parsing with PMU name : FAILED!
Fixes: ee8aef2d2321 ("perf tools: Add skip check in tool_pmu__event_to_str()")
Signed-off-by: James Clark <james.clark@linaro.org>
Reported-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com>
Acked-by: Kan Liang <kan.liang@linux.intel.com>
Tested-by: Ian Rogers <irogers@google.com>
Link: https://lore.kernel.org/r/20250212163859.1489916-1-james.clark@linaro.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf/util/scripting-engines/trace-event-python.c')
0 files changed, 0 insertions, 0 deletions