From: Naman Jain <namjain@linux.microsoft.com>
To: Will Deacon <will@kernel.org>
Cc: Anshuman Khandual <anshuman.khandual@arm.com>,
Jinjie Ruan <ruanjinjie@huawei.com>,
Catalin Marinas <catalin.marinas@arm.com>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Marc Zyngier <maz@kernel.org>,
Thomas Huth <thuth@redhat.com>, Fuad Tabba <fuad.tabba@linux.dev>,
Thomas Gleixner <tglx@kernel.org>,
Pengjie Zhang <zhangpengjie2@huawei.com>,
mrigendrachaubey <mrigendra.chaubey@gmail.com>,
Saurabh Sengar <ssengar@linux.microsoft.com>
Subject: Re: [PATCH] arm64: smp: distinguish secondary CPUs that hang after reaching head.S
Date: Wed, 5 Aug 2026 10:12:03 +0530 [thread overview]
Message-ID: <96bf067f-b811-4a25-a7d3-ff50c13134d0@linux.microsoft.com> (raw)
In-Reply-To: <anHsh0SpRfSMIza7@willie-the-truck>
On 8/4/2026 7:13 PM, Will Deacon wrote:
> On Mon, Jul 27, 2026 at 11:08:05AM +0530, Naman Jain wrote:
>> On 7/26/2026 7:16 PM, Will Deacon wrote:
>>> On Fri, Jul 24, 2026 at 10:16:03AM +0530, Naman Jain wrote:
>>>> On 7/23/2026 11:26 AM, Anshuman Khandual wrote:
>>>>> On 23/07/26 8:19 AM, Jinjie Ruan wrote:
>>>>>> 在 2026/7/22 19:30, Naman Jain 写道:
>>>>>>> When a secondary CPU fails to come online, __cpu_up() falls back to
>>>>>>> __early_cpu_boot_status, but boot status 0x0 is ambiguous: it cannot
>>>>>>> distinguish a CPU that never executed head.S (firmware/hypervisor never
>>>>>>> dispatched it, so it never ran a single instruction) from one that
>>>>>>> entered head.S, started executing, and then got stuck somewhere in kernel
>>>>>>> bring-up. Add a change to let us tell those two cases apart, which
>>>>>>> narrows down where to look when a CPU goes missing during boot.
>>>>>> I previously encountered this issue when debugging the parallel startup
>>>>>> of ARM64 secondary cores. It is difficult for the kernel to determine
>>>>>> whether the secondary core is hung in the firmware or whether it has not
>>>>>> executed a single instruction. So I think this motive is reasonable.
>>>>>
>>>>> Why should kernel determine the difference here ? Would not the firmware
>>>>> know if it has started any secondary CPU for the kernel which must have
>>>>> come inside head.S ? If the cpu gets hung inside firmware while starting
>>>>> up then the debug responsibilities belong there instead.
>>>>>
>>>>> Still wondering what's the rationale for this change.
>>>>
>>>> Hello Anshuman,
>>>> This sounds fair to me. Let me elaborate the problem, beyond the scope of
>>>> this patch. In production, we occasionally see these crashes where one of
>>>> the CPU fails to bring up online, with 0x0 status code. Hypervisor may be
>>>> missing the telemetry, but the problem is that we don't know if the
>>>> secondary CPU ever started executing the instructions or is stuck somewhere
>>>> between the start of head.S and marking itself online at the end of
>>>> secondary_start_kernel().
>>>> There are couple of places, where we get those other status codes, but not
>>>> everywhere. If the issue is not easily reproducible, experiments on local
>>>> setups do not yield anything. That's where I am attempting to add some more
>>>> information in kernel to debug these issues.
>>>
>>> I think this is a game of diminishing returns. There's a lot of stuff
>>> that the firmware/hypervisor can get wrong here and trying to detect or
>>> handle that in Linux is going to be a real mess. For example, if it
>>> enters the kernel at the wrong address, or in the wrong mode, or with
>>> the MMU enabled etc. It sounds like you don't have much idea about
>>> what happens in the failure case, so it might not even execute the code
>>> that you're adding correctly.
>>
>> That is true. I agree. While we cannot and should not worry about adding
>> logs for each of these firmware failure points in kernel, do you see any
>> merit in adding any of this information to the kernel to at least narrow
>> down the problem?
>
> Possibly, but I'd prefer to land the parallel bringup work first and then
> see whether this still makes sense.
>
> Will
Acked.
Regards,
Naman
prev parent reply other threads:[~2026-08-05 4:42 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-22 11:30 Naman Jain
2026-07-23 2:49 ` Jinjie Ruan
2026-07-23 5:56 ` Anshuman Khandual
2026-07-24 4:46 ` Naman Jain
2026-07-24 5:23 ` Anshuman Khandual
2026-07-24 8:01 ` Naman Jain
2026-07-26 13:46 ` Will Deacon
2026-07-27 5:38 ` Naman Jain
2026-08-04 13:43 ` Will Deacon
2026-08-05 4:42 ` Naman Jain [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=96bf067f-b811-4a25-a7d3-ff50c13134d0@linux.microsoft.com \
--to=namjain@linux.microsoft.com \
--cc=anshuman.khandual@arm.com \
--cc=catalin.marinas@arm.com \
--cc=fuad.tabba@linux.dev \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maz@kernel.org \
--cc=mrigendra.chaubey@gmail.com \
--cc=ruanjinjie@huawei.com \
--cc=ssengar@linux.microsoft.com \
--cc=tglx@kernel.org \
--cc=thuth@redhat.com \
--cc=will@kernel.org \
--cc=zhangpengjie2@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome