[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20b7c778-68c0-9ba3-06fb-61194ed3e04e@huawei.com>
Date: Tue, 23 Apr 2024 14:22:11 +0800
From: Hanjun Guo <guohanjun@...wei.com>
To: Jonathan Cameron <Jonathan.Cameron@...wei.com>, Thomas Gleixner
<tglx@...utronix.de>, Peter Zijlstra <peterz@...radead.org>,
<linux-pm@...r.kernel.org>, <loongarch@...ts.linux.dev>,
<linux-acpi@...r.kernel.org>, <linux-arch@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
<kvmarm@...ts.linux.dev>, <x86@...nel.org>, Russell King
<linux@...linux.org.uk>, "Rafael J . Wysocki" <rafael@...nel.org>, Miguel
Luis <miguel.luis@...cle.com>, James Morse <james.morse@....com>, Salil Mehta
<salil.mehta@...wei.com>, Jean-Philippe Brucker <jean-philippe@...aro.org>,
Catalin Marinas <catalin.marinas@....com>, Will Deacon <will@...nel.org>
CC: Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, Dave
Hansen <dave.hansen@...ux.intel.com>, <linuxarm@...wei.com>,
<justin.he@....com>, <jianyong.wu@....com>
Subject: Re: [PATCH v7 02/16] cpu: Do not warn on arch_register_cpu()
returning -EPROBE_DEFER
On 2024/4/18 21:53, Jonathan Cameron wrote:
> For arm64 the CPU registration cannot complete until the ACPI
> interpreter us up and running so in those cases the arch specific
> arch_register_cpu() will return -EPROBE_DEFER at this stage and the
> registration will be attempted later.
>
> Suggested-by: Rafael J. Wysocki <rafael@...nel.org>
> Acked-by: Rafael J. Wysocki <rafael@...nel.org>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@...wei.com>
>
> ---
> v7: Fix condition to not print the error message of success (thanks Russell!)
> ---
> drivers/base/cpu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c
> index 56fba44ba391..7b83e9c87d7c 100644
> --- a/drivers/base/cpu.c
> +++ b/drivers/base/cpu.c
> @@ -558,7 +558,7 @@ static void __init cpu_dev_register_generic(void)
>
> for_each_present_cpu(i) {
> ret = arch_register_cpu(i);
> - if (ret)
> + if (ret && ret != -EPROBE_DEFER)
> pr_warn("register_cpu %d failed (%d)\n", i, ret);
> }
> }
Reviewed-by: Hanjun Guo <guohanjun@...wei.com>
Thanks
Hanjun
Powered by blists - more mailing lists