Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Will Deacon <will@kernel.org>
To: Kiryl Shutsemau <kas@kernel.org>
Cc: Catalin Marinas <catalin.marinas@arm.com>,
	James Morse <james.morse@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Marc Zyngier <maz@kernel.org>,
	Doug Anderson <dianders@chromium.org>,
	Petr Mladek <pmladek@suse.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Baoquan He <bhe@redhat.com>, Puranjay Mohan <puranjay@kernel.org>,
	Usama Arif <usama.arif@linux.dev>,
	Breno Leitao <leitao@debian.org>,
	Julien Thierry <julien.thierry.kdev@gmail.com>,
	Lecopzer Chen <lecopzer@gmail.com>,
	Sumit Garg <sumit.garg@kernel.org>,
	kernel-team@meta.com, kexec@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 0/4] arm64: cross-CPU NMI via SDEI
Date: Tue, 28 Jul 2026 11:25:37 +0100	[thread overview]
Message-ID: <amiDoZdszC1pB6Z2@willie-the-truck> (raw)
In-Reply-To: <amZRtS2Lu1epc0S4@thinkstation>

On Sun, Jul 26, 2026 at 07:35:20PM +0100, Kiryl Shutsemau wrote:
> On Sun, Jul 26, 2026 at 02:55:49PM +0100, Will Deacon wrote:
> > On Thu, Jul 09, 2026 at 05:16:37PM +0100, Kiryl Shutsemau wrote:
> > > On Mon, Jun 29, 2026 at 04:07:14PM +0100, Kiryl Shutsemau wrote:
> > > > From: "Kiryl Shutsemau (Meta)" <kas@kernel.org>
> > > > 
> > > > A class of debug/observability features needs to interrupt a CPU that has
> > > > its interrupts locally masked: the all-CPU backtrace behind sysrq-l /
> > > > RCU-stall / hung-task / hard-lockup dumps, and crash_smp_send_stop()
> > > > capturing a stuck CPU's state into the vmcore. On arm64 these need a
> > > > mechanism that reaches a CPU spinning with DAIF masked, which a normal IPI
> > > > cannot.
> > > 
> > > Gentle ping. Any feedback?
> > > 
> > > I'm looking forward to finding an upstreamable solution to the problem.
> > 
> > This is all pretty small, self-contained and it's useful to you, so I'm
> > inclined to merge it.
> 
> Thanks!
> 
> > However, the one vague concern I have is about the direction of SDEI in
> > the future. AFAIK, the TF-A implementation is known to have issues, it's
> > not supported at all by R-FA and I worry that the spec is going to fall
> > behind the architecture, particularly as FEAT_NMI becomes available.
> > 
> > So it would be good to understand what we're signing up to maintain here.
> > Kiryl, do you see an (eventual) migration over to FEAT_NMI, giving us a
> > path to deprecating SDEI altogether, or do you think the two will live
> > alongside each other for the forseeable future?
> 
> Realistically, the foreseeable future.
> 
> FEAT_NMI is the endgame and a machine that has it needs none of this,
> but non-FEAT_NMI parts stay in fleets for many years, so I can't
> honestly give you a date for dropping SDEI.

That's fair, I was really just trying to understand the direction. It
sounds like you will *eventually* move to FEAT_NMI, even if it's many
years away.

> While this change is pretty self-contained, I understand that SDEI infra
> maintenance can be a burden.
> 
> Is there anything on the wider SDEI infrastructure that would lower the
> maintenance burden or keep it more out of your way -- consolidation,
> tighter isolation, docs, help carrying it? If there's work there that
> makes SDEI more palatable to keep while non-FEAT_NMI parts are in
> service, I'm glad to take it on.

I'm somewhat trying to predict the future here, but I wouldn't be
shocked if some whizz-bang new architecture feature interacted badly
with SDEI, either due to outdated specs or because of a fundamental
incompatability. In this case, I would like the option of being able to
make new hardware support mutually exclusive with SDEI rather than try
to hack around the mess. It sounds like that would be ok for you?

Will


  reply	other threads:[~2026-07-28 10:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-29 15:07 [PATCH v5 0/4] arm64: cross-CPU NMI via SDEI Kiryl Shutsemau
2026-06-29 15:07 ` [PATCH v5 1/4] firmware: arm_sdei: add sdei_is_present() Kiryl Shutsemau
2026-06-30 10:47   ` Usama Arif
2026-06-30 10:57     ` Kiryl Shutsemau
2026-06-29 15:07 ` [PATCH v5 2/4] firmware: arm_sdei: add SDEI_EVENT_SIGNAL support Kiryl Shutsemau
2026-06-30 10:51   ` Usama Arif
2026-06-30 10:58     ` Kiryl Shutsemau
2026-06-29 15:07 ` [PATCH v5 3/4] drivers/firmware: add SDEI cross-CPU NMI service for arm64 Kiryl Shutsemau
2026-06-29 15:07 ` [PATCH v5 4/4] arm64: escalate smp_send_stop() to an SDEI NMI as a last resort Kiryl Shutsemau
2026-07-03  6:01 ` [PATCH v5 0/4] arm64: cross-CPU NMI via SDEI YinFengwei
2026-07-09 16:16 ` Kiryl Shutsemau
2026-07-26 13:55   ` Will Deacon
2026-07-26 18:35     ` Kiryl Shutsemau
2026-07-28 10:25       ` Will Deacon [this message]
2026-07-28 12:17         ` Kiryl Shutsemau

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=amiDoZdszC1pB6Z2@willie-the-truck \
    --to=will@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=bhe@redhat.com \
    --cc=catalin.marinas@arm.com \
    --cc=dianders@chromium.org \
    --cc=james.morse@arm.com \
    --cc=julien.thierry.kdev@gmail.com \
    --cc=kas@kernel.org \
    --cc=kernel-team@meta.com \
    --cc=kexec@lists.infradead.org \
    --cc=lecopzer@gmail.com \
    --cc=leitao@debian.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=maz@kernel.org \
    --cc=pmladek@suse.com \
    --cc=puranjay@kernel.org \
    --cc=sumit.garg@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=usama.arif@linux.dev \
    /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