perf/core: Fix perf_pmu_register() vs. perf_init_event()
authorPeter Zijlstra <peterz@infradead.org>
Mon, 4 Nov 2024 13:39:12 +0000 (14:39 +0100)
committerIngo Molnar <mingo@kernel.org>
Sat, 1 Mar 2025 18:38:42 +0000 (19:38 +0100)
commit003659fec9f6d8c04738cb74b5384398ae8a7e88
treefc3a7217c62bae91b8a50eda27d9fcb32c28a316
parent2565e42539b120b81a68a58da961ce5d1e34eac8
perf/core: Fix perf_pmu_register() vs. perf_init_event()

There is a fairly obvious race between perf_init_event() doing
idr_find() and perf_pmu_register() doing idr_alloc() with an
incompletely initialized PMU pointer.

Avoid by doing idr_alloc() on a NULL pointer to register the id, and
swizzling the real struct pmu pointer at the end using idr_replace().

Also making sure to not set struct pmu members after publishing
the struct pmu, duh.

[ introduce idr_cmpxchg() in order to better handle the idr_replace()
  error case -- if it were to return an unexpected pointer, it will
  already have replaced the value and there is no going back. ]

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20241104135517.858805880@infradead.org
kernel/events/core.c