Skip to content

Commit 8b99e2f

Browse files
captain5050namhyung
authored andcommitted
perf parse-events filter: Use evsel__find_pmu
Rather than manually scanning PMUs, use evsel__find_pmu that can use the PMU set during event parsing. Signed-off-by: Ian Rogers <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Namhyung Kim <[email protected]>
1 parent 189a977 commit 8b99e2f

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

tools/perf/util/parse-events.c

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2564,9 +2564,8 @@ foreach_evsel_in_last_glob(struct evlist *evlist,
25642564
static int set_filter(struct evsel *evsel, const void *arg)
25652565
{
25662566
const char *str = arg;
2567-
bool found = false;
25682567
int nr_addr_filters = 0;
2569-
struct perf_pmu *pmu = NULL;
2568+
struct perf_pmu *pmu;
25702569

25712570
if (evsel == NULL) {
25722571
fprintf(stderr,
@@ -2584,16 +2583,11 @@ static int set_filter(struct evsel *evsel, const void *arg)
25842583
return 0;
25852584
}
25862585

2587-
while ((pmu = perf_pmus__scan(pmu)) != NULL)
2588-
if (pmu->type == evsel->core.attr.type) {
2589-
found = true;
2590-
break;
2591-
}
2592-
2593-
if (found)
2586+
pmu = evsel__find_pmu(evsel);
2587+
if (pmu) {
25942588
perf_pmu__scan_file(pmu, "nr_addr_filters",
25952589
"%d", &nr_addr_filters);
2596-
2590+
}
25972591
if (!nr_addr_filters)
25982592
return perf_bpf_filter__parse(&evsel->bpf_filters, str);
25992593

0 commit comments

Comments
 (0)