niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

perf/core: Remove pmu linear searching code

Browse source 
Searching for the right pmu by iterating over all pmus is no longer
required since all pmus now *must* be present in the 'pmu_idr' list.
So, remove linear searching code.

Signed-off-by: Ravi Bangoria <ravi.bangoria@amd.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20230504110003.2548-4-ravi.bangoria@amd.com
This commit is contained in:
Ravi Bangoria 2023-05-04 16:30:02 +05:30 • committed by Peter Zijlstra
parent 2fad201fe3
commit 9551fbb64d
1 changed files with 13 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
kernel/events/core.c
View file

@ -11630,10 +11630,13 @@ static struct pmu *perf_init_event(struct perf_event *event)
} }
again: again:
ret = -ENOENT;
rcu_read_lock(); rcu_read_lock();
pmu = idr_find(&pmu_idr, type); pmu = idr_find(&pmu_idr, type);
rcu_read_unlock(); rcu_read_unlock();
if (pmu) { if (!pmu)
goto fail;
if (event->attr.type != type && type != PERF_TYPE_RAW && if (event->attr.type != type && type != PERF_TYPE_RAW &&
!(pmu->capabilities & PERF_PMU_CAP_EXTENDED_HW_TYPE)) !(pmu->capabilities & PERF_PMU_CAP_EXTENDED_HW_TYPE))
goto fail; goto fail;
@ -11644,24 +11647,10 @@ again:
goto again; goto again;
} }
fail:
if (ret) if (ret)
pmu = ERR_PTR(ret); pmu = ERR_PTR(ret);
goto unlock;
}
list_for_each_entry_rcu(pmu, &pmus, entry, lockdep_is_held(&pmus_srcu)) {
ret = perf_try_init_event(pmu, event);
if (!ret)
goto unlock;
if (ret != -ENOENT) {
pmu = ERR_PTR(ret);
goto unlock;
}
}
fail:
pmu = ERR_PTR(-ENOENT);
unlock: unlock:
srcu_read_unlock(&pmus_srcu, idx); srcu_read_unlock(&pmus_srcu, idx);