linux-kernel - Re: [PATCH] arm64: topology: Do not warn on missing AMU in cpuhp_topology

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <7f156aa7-c723-4580-aa82-b2f0fe6d3845@huawei.com>
Date: Thu, 15 Jan 2026 20:52:11 +0800
From: "zhenglifeng (A)" <zhenglifeng1@...wei.com>
To: Geert Uytterhoeven <geert+renesas@...der.be>, Beata Michalska
	<beata.michalska@....com>, Will Deacon <will@...nel.org>, Catalin Marinas
	<catalin.marinas@....com>
CC: <linux-arm-kernel@...ts.infradead.org>,
	<linux-renesas-soc@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: topology: Do not warn on missing AMU in
 cpuhp_topology_online()

On 2026/1/15 16:51, Geert Uytterhoeven wrote:
> When CONFIG_CPUMASK_OFFSTACK is not enabled, and resuming from s2ram on
> Renesas R-Car H3 (big.LITTLE 4x Cortex-A57 + 4x Cortex-A53), during
> enabling of the first little core, a warning message is printed:
> 
>     AMU: CPU[4] doesn't support AMU counters
> 
> This confuses users, as during boot amu_fie_setup() does not print such
> a message, unless debugging is enabled (freq_counters_valid() prints
> "CPU%d: counters are not supported.\n" at debug level in that case).
> 
> Hence drop the warning, freq_counters_valid() has already printed a
> debug message anyway.
> 
> Fixes: 6fd9be0b7b2e957d ("arm64: topology: Handle AMU FIE setup on CPU hotplug")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
> ---
>  arch/arm64/kernel/topology.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c
> index 539b3893518281ec..3fe1faab03620e78 100644
> --- a/arch/arm64/kernel/topology.c
> +++ b/arch/arm64/kernel/topology.c
> @@ -332,7 +332,6 @@ static int cpuhp_topology_online(unsigned int cpu)
>  	 * may use different source of the freq scale.
>  	 */
>  	if (!freq_counters_valid(cpu)) {
> -		pr_warn("CPU[%u] doesn't support AMU counters\n", cpu);
>  		topology_clear_scale_freq_source(SCALE_FREQ_SOURCE_ARCH,
>  						 policy->related_cpus);
>  		cpumask_andnot(amu_fie_cpus, amu_fie_cpus, policy->related_cpus);

Thanks!

Reviewed-by: Lifeng Zheng <zhenglifeng1@...wei.com>