Re: [PATCH 11/21] drivers: base: Print a warning instead of panic() when register_cpu() fails

Gavin Shan <[email protected]> Fri, 1 Dec 2023 14:49:03 +1100
Newsgroups org.kernel.vger.linux-ia64,dev.linux.lists.kvmarm,dev.linux.lists.loongarch,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-riscv,org.kernel.vger.linux-acpi,org.kernel.vger.linux-arch,org.kernel.vger.linux-csky,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel,org.kernel.vger.linux-parisc,org.kernel.vger.linux-pm
Message-ID <[email protected]>
On 11/22/23 00:44, Russell King (Oracle) wrote:
> From: James Morse <[email protected]>
> 
> loongarch, mips, parisc, riscv and sh all print a warning if
> register_cpu() returns an error. Architectures that use
> GENERIC_CPU_DEVICES call panic() instead.
> 
> Errors in this path indicate something is wrong with the firmware
> description of the platform, but the kernel is able to keep running.
> 
> Downgrade this to a warning to make it easier to debug this issue.
> 
> This will allow architectures that switching over to GENERIC_CPU_DEVICES
> to drop their warning, but keep the existing behaviour.
> 
> Signed-off-by: James Morse <[email protected]>
> Reviewed-by: Russell King (Oracle) <[email protected]>
> Reviewed-by: Shaoqin Huang <[email protected]>
> Signed-off-by: Russell King (Oracle) <[email protected]>
> ---
>   drivers/base/cpu.c | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 

Reviewed-by: Gavin Shan <[email protected]>