[PATCH] arm64: Restrict CPU_BIG_ENDIAN to GNU as or LLVM IAS 15.x or newer

Catalin Marinas catalin.marinas at arm.com
Thu Oct 26 08:33:15 PDT 2023


On Thu, Oct 26, 2023 at 04:31:59PM +0100, Mark Rutland wrote:
> On Thu, Oct 26, 2023 at 04:28:45PM +0100, Catalin Marinas wrote:
> > On Wed, Oct 25, 2023 at 11:31:14AM -0700, Nathan Chancellor wrote:
> > > On Wed, Oct 25, 2023 at 07:01:53PM +0100, Mark Rutland wrote:
> > > > On Wed, Oct 25, 2023 at 10:21:28AM -0700, Nathan Chancellor wrote:
> > > > > A recent refactoring in the arm64 tree exposed an assembler bug in LLVM
> > > > > with regards to the generation of NOPs for arm64 big endian, resulting
> > > > > in near-immediate crashes on boot in QEMU.
> > > > 
> > > > Could we please put a bit more detail into the commit message about what
> > > > exactly went wrong and how this was detected? I know that can be found
> > > > from the github links below, but having to go chase that is a bit of a
> > > > pain.
> > > 
> > > Sure, sorry for leaving that out of the initial revision.
> > > 
> > > > Would you be happy with the below? I've also added a Cc stable, since
> > > > this is a potential state corruption issue.
> > > 
> > > That text looks much better to me, especially since it explains exactly
> > > what goes wrong here (which I was unsure of, this helps). Thanks a lot!
> > > 
> > > Will / Catalin, would you like a v2 with that text or could it just be
> > > copied and pasted from Mark's mail during application time? I am happy
> > > to do whatever.
> > 
> > I'll copy/paste Mark's text, no need for v2. Thanks.
> 
> If you do, could you fix my typo in the first line? I accidentally wrote:
> 
> 	LLVM's integrated assemble
> 
> When that should have been:
> 
> 	LLVM's integrated assembler
> 
> If you're already picked the patch it's not worth worrying about.

I can still fix it, hasn't pushed it out yet.

-- 
Catalin



More information about the linux-arm-kernel mailing list