Messages in this thread | ![/](/images/icornerl.gif) | | Subject | Re: [PATCH 1/2] perf: arm_pmu: Only show online CPUs in device's "cpus" attribute | From | Yicong Yang <> | Date | Thu, 11 Apr 2024 16:55:35 +0800 |
| |
On 2024/4/10 23:34, Will Deacon wrote: > On Wed, Apr 10, 2024 at 05:58:32PM +0800, Yicong Yang wrote: >> From: Yicong Yang <yangyicong@hisilicon.com> >> >> When there're CPUs offline after system booting, perf will failed: >> [root@localhost ~]# /home/yang/perf stat -a -e armv8_pmuv3_0/cycles/ >> Error: >> The sys_perf_event_open() syscall returned with 19 (No such device) for event (cpu-clock). >> /bin/dmesg | grep -i perf may provide additional information. >> >> This is due to PMU's "cpus" is not updated and still contains offline >> CPUs and perf will try to open perf event on the offlined CPUs. >> >> Make "cpus" attribute only shows online CPUs and introduced a new >> "supported_cpus" where users can get the range of the CPUs this >> PMU supported monitoring. >> >> Signed-off-by: Yicong Yang <yangyicong@hisilicon.com> >> --- >> drivers/perf/arm_pmu.c | 24 +++++++++++++++++++++++- >> 1 file changed, 23 insertions(+), 1 deletion(-) > > Hmm. Is the complexity in the driver really worth it here? CPUs can be > onlined and offlined after the perf_event_open() syscall has been > executed,
Yes. So we have cpuhp callbacks to handle the cpu online/offline and migrate the perf context.
> so this feels like something userspace should be aware of and > handle on a best-effort basis anyway. >
Looks like it's a convention for a PMU device to provide a "cpus" attribute (for core PMUs) or "cpumask" attribute (for uncore PMUs) to indicates the CPUs on which the events can be opened. If no such attributes provided, all online CPUs indicated. Perf will check this and if user doesn't specify a certian range of CPUs the events will be opened on all the CPUs PMU indicated.
> Does x86 get away with this because CPU0 is never offlined? >
Checked on my x86 server there's no "cpus" or "cpumask" provided so perf will try to open the events on all the online CPUs if no CPU range specified. But for their hybrid platform there do have a "cpus" attribute[1] and it'll be updated when CPU offline[2].
The arm-cspmu also provides a "cpumask" to indicate supported online CPUs and an "associated_cpus" to indicated the CPUs related to the PMU.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/x86/events/intel/core.c?h=v6.9-rc1#n5931 [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/x86/events/intel/core.c?h=v6.9-rc1#n4949
Thanks.
| ![\](/images/icornerr.gif) |