From: Arnd Bergmann <arnd@kernel.org>
To: Nick Desaulniers <ndesaulniers@google.com>
Cc: Jian Cai <jiancai@google.com>, Ard Biesheuvel <ardb@kernel.org>,
Manoj Gupta <manojgupta@google.com>,
Luis Lozano <llozano@google.com>,
clang-built-linux <clang-built-linux@googlegroups.com>,
Russell King <linux@armlinux.org.uk>,
Nathan Chancellor <natechancellor@gmail.com>,
Linux ARM <linux-arm-kernel@lists.infradead.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler
Date: Tue, 10 Nov 2020 21:38:01 +0100 [thread overview]
Message-ID: <CAK8P3a1tU8g4HfsoYH4qa6C4Kv8QE3nv=UPSyH7824GXCYwUDQ@mail.gmail.com> (raw)
In-Reply-To: <CAKwvOdkpHdQF9Ko8FbP_SN=QfDiMq8ra5TSj_KHGRCbJdyYm6w@mail.gmail.com>
On Tue, Nov 10, 2020 at 9:11 PM 'Nick Desaulniers' via Clang Built
Linux <clang-built-linux@googlegroups.com> wrote:
>
> On Tue, Nov 10, 2020 at 12:10 PM Jian Cai <jiancai@google.com> wrote:
> >
> > I tried to verify with ixp4xx_defconfig, and I noticed it also used CONFIG_CPU_BIG_ENDIAN=y to enable big endianness as follows,
> >
> > linux$ grep ENDIAN arch/arm/configs/ixp4xx_defconfig
> > CONFIG_CPU_BIG_ENDIAN=y
> >
> > Also it appeared arch/arm/kernel/iwmmxt.o was not built with ixp4xx_defconfig. The commands I used
> >
> > linux$ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- make CC=clang ixp4xx_defconfig
> > linux$ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- make -j32 CC=clang
> > linux$ ls arch/arm/kernel/iwmmxt.o
> > ls: cannot access 'arch/arm/kernel/iwmmxt.o': No such file or directory
> >
> > Did I miss any steps?
>
> Yes, you need to manually enable CONFIG_IWMMXT in menuconfig or via
> `scripts/configs -e`.
To clarify: ixp4xx and pxa3xx were two platforms based on the XScale core.
ixp4xx was commonly used in big-endian mode but lacked iWMMXt. pxa3xx
had iWMMXt but doesn't allow enabling big-endian mode because of a Kconfig
dependency, meaning that nobody has ever tried it, and it's likely broken.
Later 'mvebu' parts (Armada 510) do have iWMMXt and allow big-endian
mode, but those are BE8, with non-reversed byteorder for the instructions.
So none of this matters in practice, but it's very satifiying to know
it is finally
all working with the integrated assembler in all those combinations, at
least in theory!
Arnd
next prev parent reply other threads:[~2020-11-10 20:38 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-09 23:27 [PATCH] ARM: do not assemble iwmmxt.S with LLVM toolchain Jian Cai
2020-04-10 0:01 ` Nick Desaulniers
2020-04-10 0:12 ` Nathan Chancellor
2020-04-10 6:38 ` Sedat Dilek
2020-04-10 7:44 ` Nathan Chancellor
2020-04-10 9:56 ` Arnd Bergmann
2020-04-10 11:15 ` Ard Biesheuvel
2020-04-10 12:33 ` Russell King - ARM Linux admin
2020-04-10 13:09 ` Ard Biesheuvel
2020-04-13 20:45 ` Nick Desaulniers
2020-04-14 8:59 ` Ard Biesheuvel
2020-04-14 18:38 ` Nick Desaulniers
2020-04-14 20:53 ` Kees Cook
2020-04-15 10:32 ` Ard Biesheuvel
2020-04-15 12:58 ` Arnd Bergmann
2020-04-15 14:44 ` Russell King - ARM Linux admin
2020-04-19 11:08 ` Stefan Agner
2020-04-15 15:44 ` Kees Cook
2020-04-17 14:12 ` Masahiro Yamada
2020-04-13 19:23 ` Nick Desaulniers
2020-04-13 21:00 ` Russell King - ARM Linux admin
2020-11-04 20:44 ` Nick Desaulniers
2020-04-13 19:20 ` Nick Desaulniers
2020-04-13 21:15 ` Andrew Lunn
2020-04-10 16:59 ` Andrew Lunn
2020-04-10 18:34 ` Russell King - ARM Linux admin
2020-04-13 19:26 ` Nick Desaulniers
2020-04-13 21:53 ` Russell King - ARM Linux admin
2020-11-07 0:10 ` [PATCH v2] Make iwmmxt.S support Clang's integrated assembler Jian Cai
2020-11-07 8:29 ` Ard Biesheuvel
2020-11-10 1:55 ` Nick Desaulniers
2020-11-10 7:35 ` Ard Biesheuvel
2020-11-10 19:26 ` Nick Desaulniers
[not found] ` <CA+SOCLJSSR62VfWhKx9R1cxT-CHXD7RN08eJBYxUH8zzyWos9Q@mail.gmail.com>
2020-11-10 20:11 ` Nick Desaulniers
2020-11-10 20:38 ` Arnd Bergmann [this message]
2020-11-10 21:35 ` Ard Biesheuvel
2020-11-12 8:42 ` Ard Biesheuvel
2020-11-12 13:06 ` Arnd Bergmann
2020-11-12 20:07 ` [PATCH v3] " Jian Cai
2020-11-17 0:37 ` [PATCH v4] " Jian Cai
2020-11-17 1:20 ` Nick Desaulniers
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='CAK8P3a1tU8g4HfsoYH4qa6C4Kv8QE3nv=UPSyH7824GXCYwUDQ@mail.gmail.com' \
--to=arnd@kernel.org \
--cc=ardb@kernel.org \
--cc=clang-built-linux@googlegroups.com \
--cc=jiancai@google.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=llozano@google.com \
--cc=manojgupta@google.com \
--cc=natechancellor@gmail.com \
--cc=ndesaulniers@google.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;
as well as URLs for NNTP newsgroup(s).