From: Nathan Chancellor <nathan@kernel.org>
To: Arnd Bergmann <arnd@kernel.org>
Cc: Akari Tsuyukusa <akkun11.open@gmail.com>,
Nick Desaulniers <ndesaulniers@google.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Russell King <linux@armlinux.org.uk>,
Bill Wendling <morbo@google.com>,
Justin Stitt <justinstitt@google.com>,
Yingjoe Chen <yingjoe.chen@mediatek.com>,
"moderated list:ARM SUB-ARCHITECTURES"
<linux-arm-kernel@lists.infradead.org>,
"open list:ARM/Mediatek SoC support"
<linux-kernel@vger.kernel.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
"open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b"
<llvm@lists.linux.dev>,
stable@vger.kernel.org, Ard Biesheuvel <ardb@google.com>
Subject: Re: [PATCH] arm: mediatek: fix secondary CPU boot on Thumb-2 kernels with Clang
Date: Fri, 7 Aug 2026 11:52:59 -0700 [thread overview]
Message-ID: <20260807185259.GA941196@ax162> (raw)
In-Reply-To: <c4e9a566-baf7-4df6-b874-30c68a5c67f6@app.fastmail.com>
On Sun, Jul 26, 2026 at 10:00:35PM +0200, Arnd Bergmann wrote:
> Working around it in the kernel is fine, but I'd prefer to wait for
> the fix to first get merged into llvm so we can reference the
> bugfix in the kernel commit.
This is now fixed in LLVM main:
https://github.com/llvm/llvm-project/commit/261c025a03732ac6d3a26ad80e962481237de052
> A trivial workaround would be to have a dependency in
> CONFIG_THUMB2_KERNEL for !AS_IS_LLVM||CLANG_VERSION>=24
> (or whatever version), though fixing the .S code in all known
> instances is probably better if we want to ever have a chance
> to require IAS on all architectures.
I guess it is a question of how much maintenance will keeping up the .S
code take over time. Presumably this issue is not too common if we are
just now discovering it but maybe that is because CONFIG_THUMB2_KERNEL
is not too common? Is there a way to flag this at compile time? I don't
really have a strong opinion.
--
Cheers,
Nathan
prev parent reply other threads:[~2026-08-07 18:53 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-22 17:24 [PATCH] arm: mediatek: fix secondary CPU boot on Thumb-2 kernels with Clang Akari Tsuyukusa
2026-07-22 21:03 ` Nick Desaulniers
2026-07-23 4:45 ` Roman Vivchar
2026-07-23 9:15 ` Akari Tsuyukusa
2026-07-23 9:53 ` Arnd Bergmann
2026-07-26 6:08 ` Akari Tsuyukusa
2026-07-26 20:00 ` Arnd Bergmann
2026-08-07 18:52 ` Nathan Chancellor [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=20260807185259.GA941196@ax162 \
--to=nathan@kernel.org \
--cc=akkun11.open@gmail.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=ardb@google.com \
--cc=arnd@kernel.org \
--cc=justinstitt@google.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux@armlinux.org.uk \
--cc=llvm@lists.linux.dev \
--cc=matthias.bgg@gmail.com \
--cc=morbo@google.com \
--cc=ndesaulniers@google.com \
--cc=stable@vger.kernel.org \
--cc=yingjoe.chen@mediatek.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