]> www.infradead.org Git - users/jedix/linux-maple.git/commit
perf: Remove too early and redundant CPU hotplug handling
authorFrederic Weisbecker <frederic@kernel.org>
Thu, 24 Apr 2025 16:11:27 +0000 (18:11 +0200)
committerPeter Zijlstra <peterz@infradead.org>
Thu, 8 May 2025 19:50:19 +0000 (21:50 +0200)
commitf400565faa50737ac1d550d2c75128c0dad75765
tree564f014a2b30f5d6b89c0d88a85684693927d5b5
parentd20eb2d5fe8f8818abcfdadf5ac5109938f1318e
perf: Remove too early and redundant CPU hotplug handling

The CPU hotplug handlers are called twice: at prepare and online stage.

Their role is to:

1) Enable/disable a CPU context. This is irrelevant and even buggy at
   the prepare stage because the CPU is still offline. On early
   secondary CPU up, creating an event attached to that CPU might
   silently fail because the CPU context is observed as online but the
   context installation's IPI failure is ignored.

2) Update the scope cpumasks and re-migrate the events accordingly in
   the CPU down case. This is irrelevant at the prepare stage.

3) Remove the events attached to the context of the offlining CPU. It
   even uses an (unnecessary) IPI for it. This is also irrelevant at the
   prepare stage.

Also none of the *_PREPARE and *_STARTING architecture perf related CPU
hotplug callbacks rely on CPUHP_PERF_PREPARE.

CPUHP_AP_PERF_ONLINE is enough and the right place to perform the work.

Signed-off-by: Frederic Weisbecker <frederic@kernel.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20250424161128.29176-4-frederic@kernel.org
include/linux/cpuhotplug.h
kernel/cpu.c