* Error: invalid switch -me200
@ 2020-11-13 3:21 ` kernel test robot
0 siblings, 0 replies; 43+ messages in thread
From: kernel test robot @ 2020-11-13 3:21 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 2082 bytes --]
Hi Fangrui,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba
commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation
date: 4 months ago
config: powerpc-randconfig-r031-20201113 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
Assembler messages:
>> Error: invalid switch -me200
>> Error: unrecognized option -me200
clang-12: error: assembler command failed with exit code 1 (use -v to see invocation)
make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1174: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 24128 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread* Error: invalid switch -me200 @ 2020-11-13 3:21 ` kernel test robot 0 siblings, 0 replies; 43+ messages in thread From: kernel test robot @ 2020-11-13 3:21 UTC (permalink / raw) To: Fangrui Song Cc: kbuild-all, clang-built-linux, linux-kernel, Masahiro Yamada, Nathan Chancellor, Nick Desaulniers [-- Attachment #1: Type: text/plain, Size: 2039 bytes --] Hi Fangrui, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation date: 4 months ago config: powerpc-randconfig-r031-20201113 (attached as .config) compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install powerpc cross compiling tool for clang build # apt-get install binutils-powerpc-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): Assembler messages: >> Error: invalid switch -me200 >> Error: unrecognized option -me200 clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 make[2]: Target '__build' not remade because of errors. make[1]: *** [Makefile:1174: prepare0] Error 2 make[1]: Target 'prepare' not remade because of errors. make: *** [Makefile:185: __sub-make] Error 2 make: Target 'prepare' not remade because of errors. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 24128 bytes --] ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 3:21 ` kernel test robot @ 2020-11-13 17:28 ` Fāng-ruì Sòng -1 siblings, 0 replies; 43+ messages in thread From: Fāng-ruì Sòng @ 2020-11-13 17:28 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 2803 bytes --] On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > Hi Fangrui, > > FYI, the error/warning still remains. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > date: 4 months ago > config: powerpc-randconfig-r031-20201113 (attached as .config) > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > reproduce (this is a W=1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # install powerpc cross compiling tool for clang build > # apt-get install binutils-powerpc-linux-gnu > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > git fetch --no-tags linus master > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): > > Assembler messages: > >> Error: invalid switch -me200 > >> Error: unrecognized option -me200 > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > make[2]: Target '__build' not remade because of errors. > make[1]: *** [Makefile:1174: prepare0] Error 2 > make[1]: Target 'prepare' not remade because of errors. > make: *** [Makefile:185: __sub-make] Error 2 > make: Target 'prepare' not remade because of errors. > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org This can be ignored. The LLVM integrated assembler does not recognize -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m option is similar to .arch or .machine and controls what instructions are recognized. The integrated assembler tends to support all instructions (conditional supporting some instructions has some challenges; in the end I have patched parsing but ignoring `.arch` for x86-64 and ignoring `.machine ppc64` for ppc64) (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit support may get less attention in LLVM.) ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 17:28 ` Fāng-ruì Sòng 0 siblings, 0 replies; 43+ messages in thread From: Fāng-ruì Sòng @ 2020-11-13 17:28 UTC (permalink / raw) To: kernel test robot Cc: kbuild-all, clang-built-linux, LKML, Masahiro Yamada, Nathan Chancellor, Nick Desaulniers On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > Hi Fangrui, > > FYI, the error/warning still remains. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > date: 4 months ago > config: powerpc-randconfig-r031-20201113 (attached as .config) > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > reproduce (this is a W=1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # install powerpc cross compiling tool for clang build > # apt-get install binutils-powerpc-linux-gnu > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > git fetch --no-tags linus master > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): > > Assembler messages: > >> Error: invalid switch -me200 > >> Error: unrecognized option -me200 > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > make[2]: Target '__build' not remade because of errors. > make[1]: *** [Makefile:1174: prepare0] Error 2 > make[1]: Target 'prepare' not remade because of errors. > make: *** [Makefile:185: __sub-make] Error 2 > make: Target 'prepare' not remade because of errors. > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org This can be ignored. The LLVM integrated assembler does not recognize -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m option is similar to .arch or .machine and controls what instructions are recognized. The integrated assembler tends to support all instructions (conditional supporting some instructions has some challenges; in the end I have patched parsing but ignoring `.arch` for x86-64 and ignoring `.machine ppc64` for ppc64) (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit support may get less attention in LLVM.) ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 17:28 ` Fāng-ruì Sòng @ 2020-11-13 19:08 ` Nathan Chancellor -1 siblings, 0 replies; 43+ messages in thread From: Nathan Chancellor @ 2020-11-13 19:08 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 3261 bytes --] On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > Hi Fangrui, > > > > FYI, the error/warning still remains. > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > date: 4 months ago > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > reproduce (this is a W=1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # install powerpc cross compiling tool for clang build > > # apt-get install binutils-powerpc-linux-gnu > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > git fetch --no-tags linus master > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > # save the attached .config to linux build tree > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot <lkp@intel.com> > > > > All errors (new ones prefixed by >>): > > > > Assembler messages: > > >> Error: invalid switch -me200 > > >> Error: unrecognized option -me200 > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > make[2]: Target '__build' not remade because of errors. > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > make[1]: Target 'prepare' not remade because of errors. > > make: *** [Makefile:185: __sub-make] Error 2 > > make: Target 'prepare' not remade because of errors. > > > > --- > > 0-DAY CI Kernel Test Service, Intel Corporation > > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org > > This can be ignored. The LLVM integrated assembler does not recognize > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > option is similar to .arch or .machine and controls what instructions > are recognized. The integrated assembler tends to support all > instructions (conditional supporting some instructions has some > challenges; in the end I have patched parsing but ignoring `.arch` for > x86-64 and ignoring `.machine ppc64` for ppc64) > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > support may get less attention in LLVM.) This is also not a clang specific issue, I see the exact same error with GCC 10.2.0 and binutils 2.35. $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux ... Error: invalid switch -me200 Error: unrecognized option -me200 ... Cheers, Nathan ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 19:08 ` Nathan Chancellor 0 siblings, 0 replies; 43+ messages in thread From: Nathan Chancellor @ 2020-11-13 19:08 UTC (permalink / raw) To: Fāng-ruì Sòng Cc: kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, Nick Desaulniers On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > Hi Fangrui, > > > > FYI, the error/warning still remains. > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > date: 4 months ago > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > reproduce (this is a W=1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # install powerpc cross compiling tool for clang build > > # apt-get install binutils-powerpc-linux-gnu > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > git fetch --no-tags linus master > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > # save the attached .config to linux build tree > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot <lkp@intel.com> > > > > All errors (new ones prefixed by >>): > > > > Assembler messages: > > >> Error: invalid switch -me200 > > >> Error: unrecognized option -me200 > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > make[2]: Target '__build' not remade because of errors. > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > make[1]: Target 'prepare' not remade because of errors. > > make: *** [Makefile:185: __sub-make] Error 2 > > make: Target 'prepare' not remade because of errors. > > > > --- > > 0-DAY CI Kernel Test Service, Intel Corporation > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > This can be ignored. The LLVM integrated assembler does not recognize > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > option is similar to .arch or .machine and controls what instructions > are recognized. The integrated assembler tends to support all > instructions (conditional supporting some instructions has some > challenges; in the end I have patched parsing but ignoring `.arch` for > x86-64 and ignoring `.machine ppc64` for ppc64) > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > support may get less attention in LLVM.) This is also not a clang specific issue, I see the exact same error with GCC 10.2.0 and binutils 2.35. $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux ... Error: invalid switch -me200 Error: unrecognized option -me200 ... Cheers, Nathan ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 19:08 ` Nathan Chancellor (?) @ 2020-11-13 19:42 ` Nick Desaulniers -1 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 19:42 UTC (permalink / raw) To: Nathan Chancellor, Michael Ellerman Cc: kbuild-all, kernel test robot, Fāng-ruì Sòng, Masahiro Yamada, LKML, clang-built-linux, linuxppc-dev + MPE, PPC On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > Hi Fangrui, > > > > > > FYI, the error/warning still remains. > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > date: 4 months ago > > > config: powerpc-randconfig-r031-20201113 (attached as .config) ^ randconfig > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # install powerpc cross compiling tool for clang build > > > # apt-get install binutils-powerpc-linux-gnu > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > git fetch --no-tags linus master > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > All errors (new ones prefixed by >>): > > > > > > Assembler messages: > > > >> Error: invalid switch -me200 > > > >> Error: unrecognized option -me200 > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > make[2]: Target '__build' not remade because of errors. > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > make[1]: Target 'prepare' not remade because of errors. > > > make: *** [Makefile:185: __sub-make] Error 2 > > > make: Target 'prepare' not remade because of errors. > > > > > > --- > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > This can be ignored. The LLVM integrated assembler does not recognize > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > option is similar to .arch or .machine and controls what instructions > > are recognized. The integrated assembler tends to support all > > instructions (conditional supporting some instructions has some > > challenges; in the end I have patched parsing but ignoring `.arch` for > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > support may get less attention in LLVM.) > > This is also not a clang specific issue, I see the exact same error > with GCC 10.2.0 and binutils 2.35. > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux Does using a non 64b triple produce the same failure? > ... > Error: invalid switch -me200 > Error: unrecognized option -me200 There's a block in arch/powerpc/Makefile: 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 250 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 Are those all broken configs, or is Kconfig messed up such that randconfig can select these when it should not? -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 19:42 ` Nick Desaulniers 0 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 19:42 UTC (permalink / raw) To: Nathan Chancellor, Michael Ellerman Cc: Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev + MPE, PPC On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > Hi Fangrui, > > > > > > FYI, the error/warning still remains. > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > date: 4 months ago > > > config: powerpc-randconfig-r031-20201113 (attached as .config) ^ randconfig > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # install powerpc cross compiling tool for clang build > > > # apt-get install binutils-powerpc-linux-gnu > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > git fetch --no-tags linus master > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > All errors (new ones prefixed by >>): > > > > > > Assembler messages: > > > >> Error: invalid switch -me200 > > > >> Error: unrecognized option -me200 > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > make[2]: Target '__build' not remade because of errors. > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > make[1]: Target 'prepare' not remade because of errors. > > > make: *** [Makefile:185: __sub-make] Error 2 > > > make: Target 'prepare' not remade because of errors. > > > > > > --- > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > This can be ignored. The LLVM integrated assembler does not recognize > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > option is similar to .arch or .machine and controls what instructions > > are recognized. The integrated assembler tends to support all > > instructions (conditional supporting some instructions has some > > challenges; in the end I have patched parsing but ignoring `.arch` for > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > support may get less attention in LLVM.) > > This is also not a clang specific issue, I see the exact same error > with GCC 10.2.0 and binutils 2.35. > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux Does using a non 64b triple produce the same failure? > ... > Error: invalid switch -me200 > Error: unrecognized option -me200 There's a block in arch/powerpc/Makefile: 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 250 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 Are those all broken configs, or is Kconfig messed up such that randconfig can select these when it should not? -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 19:42 ` Nick Desaulniers 0 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 19:42 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 3979 bytes --] + MPE, PPC On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > Hi Fangrui, > > > > > > FYI, the error/warning still remains. > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > date: 4 months ago > > > config: powerpc-randconfig-r031-20201113 (attached as .config) ^ randconfig > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # install powerpc cross compiling tool for clang build > > > # apt-get install binutils-powerpc-linux-gnu > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > git fetch --no-tags linus master > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > All errors (new ones prefixed by >>): > > > > > > Assembler messages: > > > >> Error: invalid switch -me200 > > > >> Error: unrecognized option -me200 > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > make[2]: Target '__build' not remade because of errors. > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > make[1]: Target 'prepare' not remade because of errors. > > > make: *** [Makefile:185: __sub-make] Error 2 > > > make: Target 'prepare' not remade because of errors. > > > > > > --- > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org > > > > This can be ignored. The LLVM integrated assembler does not recognize > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > option is similar to .arch or .machine and controls what instructions > > are recognized. The integrated assembler tends to support all > > instructions (conditional supporting some instructions has some > > challenges; in the end I have patched parsing but ignoring `.arch` for > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > support may get less attention in LLVM.) > > This is also not a clang specific issue, I see the exact same error > with GCC 10.2.0 and binutils 2.35. > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux Does using a non 64b triple produce the same failure? > ... > Error: invalid switch -me200 > Error: unrecognized option -me200 There's a block in arch/powerpc/Makefile: 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 250 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 Are those all broken configs, or is Kconfig messed up such that randconfig can select these when it should not? -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 19:42 ` Nick Desaulniers (?) @ 2020-11-13 20:04 ` Nathan Chancellor -1 siblings, 0 replies; 43+ messages in thread From: Nathan Chancellor @ 2020-11-13 20:04 UTC (permalink / raw) To: Nick Desaulniers Cc: kbuild-all, kernel test robot, Fāng-ruì Sòng, Masahiro Yamada, LKML, clang-built-linux, linuxppc-dev On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > + MPE, PPC > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > <natechancellor@gmail.com> wrote: > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > Hi Fangrui, > > > > > > > > FYI, the error/warning still remains. > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > date: 4 months ago > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > ^ randconfig > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # install powerpc cross compiling tool for clang build > > > > # apt-get install binutils-powerpc-linux-gnu > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > git fetch --no-tags linus master > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > # save the attached .config to linux build tree > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > Assembler messages: > > > > >> Error: invalid switch -me200 > > > > >> Error: unrecognized option -me200 > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > make[2]: Target '__build' not remade because of errors. > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > make[1]: Target 'prepare' not remade because of errors. > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > --- > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > option is similar to .arch or .machine and controls what instructions > > > are recognized. The integrated assembler tends to support all > > > instructions (conditional supporting some instructions has some > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > support may get less attention in LLVM.) > > > > This is also not a clang specific issue, I see the exact same error > > with GCC 10.2.0 and binutils 2.35. > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > Does using a non 64b triple produce the same failure? Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > ... > > Error: invalid switch -me200 > > Error: unrecognized option -me200 > > There's a block in arch/powerpc/Makefile: > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > as-option,-Wa$(comma)-maltivec) > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > Are those all broken configs, or is Kconfig messed up such that > randconfig can select these when it should not? Hmmm, looks like this flag does not exist in mainline binutils? There is a thread in 2010 about this that Segher commented on: https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ Guess this config should be eliminated? Cheers, Nathan ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 20:04 ` Nathan Chancellor 0 siblings, 0 replies; 43+ messages in thread From: Nathan Chancellor @ 2020-11-13 20:04 UTC (permalink / raw) To: Nick Desaulniers Cc: Michael Ellerman, Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > + MPE, PPC > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > <natechancellor@gmail.com> wrote: > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > Hi Fangrui, > > > > > > > > FYI, the error/warning still remains. > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > date: 4 months ago > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > ^ randconfig > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # install powerpc cross compiling tool for clang build > > > > # apt-get install binutils-powerpc-linux-gnu > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > git fetch --no-tags linus master > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > # save the attached .config to linux build tree > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > Assembler messages: > > > > >> Error: invalid switch -me200 > > > > >> Error: unrecognized option -me200 > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > make[2]: Target '__build' not remade because of errors. > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > make[1]: Target 'prepare' not remade because of errors. > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > --- > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > option is similar to .arch or .machine and controls what instructions > > > are recognized. The integrated assembler tends to support all > > > instructions (conditional supporting some instructions has some > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > support may get less attention in LLVM.) > > > > This is also not a clang specific issue, I see the exact same error > > with GCC 10.2.0 and binutils 2.35. > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > Does using a non 64b triple produce the same failure? Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > ... > > Error: invalid switch -me200 > > Error: unrecognized option -me200 > > There's a block in arch/powerpc/Makefile: > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > as-option,-Wa$(comma)-maltivec) > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > Are those all broken configs, or is Kconfig messed up such that > randconfig can select these when it should not? Hmmm, looks like this flag does not exist in mainline binutils? There is a thread in 2010 about this that Segher commented on: https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ Guess this config should be eliminated? Cheers, Nathan ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 20:04 ` Nathan Chancellor 0 siblings, 0 replies; 43+ messages in thread From: Nathan Chancellor @ 2020-11-13 20:04 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 4538 bytes --] On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > + MPE, PPC > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > <natechancellor@gmail.com> wrote: > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > Hi Fangrui, > > > > > > > > FYI, the error/warning still remains. > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > date: 4 months ago > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > ^ randconfig > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # install powerpc cross compiling tool for clang build > > > > # apt-get install binutils-powerpc-linux-gnu > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > git fetch --no-tags linus master > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > # save the attached .config to linux build tree > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > Assembler messages: > > > > >> Error: invalid switch -me200 > > > > >> Error: unrecognized option -me200 > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > make[2]: Target '__build' not remade because of errors. > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > make[1]: Target 'prepare' not remade because of errors. > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > --- > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > option is similar to .arch or .machine and controls what instructions > > > are recognized. The integrated assembler tends to support all > > > instructions (conditional supporting some instructions has some > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > support may get less attention in LLVM.) > > > > This is also not a clang specific issue, I see the exact same error > > with GCC 10.2.0 and binutils 2.35. > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > Does using a non 64b triple produce the same failure? Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > ... > > Error: invalid switch -me200 > > Error: unrecognized option -me200 > > There's a block in arch/powerpc/Makefile: > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > as-option,-Wa$(comma)-maltivec) > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > Are those all broken configs, or is Kconfig messed up such that > randconfig can select these when it should not? Hmmm, looks like this flag does not exist in mainline binutils? There is a thread in 2010 about this that Segher commented on: https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ Guess this config should be eliminated? Cheers, Nathan ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 20:04 ` Nathan Chancellor (?) @ 2020-11-13 20:14 ` Nick Desaulniers -1 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 20:14 UTC (permalink / raw) To: Nathan Chancellor, Michael Ellerman, Segher Boessenkool, Linus Torvalds, Arnd Bergmann, Brian Cain Cc: kbuild-all, kernel test robot, Fāng-ruì Sòng, Masahiro Yamada, LKML, clang-built-linux, linuxppc-dev On Fri, Nov 13, 2020 at 12:04 PM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > > + MPE, PPC > > > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > > <natechancellor@gmail.com> wrote: > > > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > > > Hi Fangrui, > > > > > > > > > > FYI, the error/warning still remains. > > > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > > date: 4 months ago > > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > > > ^ randconfig > > > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > > reproduce (this is a W=1 build): > > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > > chmod +x ~/bin/make.cross > > > > > # install powerpc cross compiling tool for clang build > > > > > # apt-get install binutils-powerpc-linux-gnu > > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > > git fetch --no-tags linus master > > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > # save the attached .config to linux build tree > > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > > > Assembler messages: > > > > > >> Error: invalid switch -me200 > > > > > >> Error: unrecognized option -me200 > > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > > make[2]: Target '__build' not remade because of errors. > > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > > make[1]: Target 'prepare' not remade because of errors. > > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > > > --- > > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > > option is similar to .arch or .machine and controls what instructions > > > > are recognized. The integrated assembler tends to support all > > > > instructions (conditional supporting some instructions has some > > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > > support may get less attention in LLVM.) > > > > > > This is also not a clang specific issue, I see the exact same error > > > with GCC 10.2.0 and binutils 2.35. > > > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > > > Does using a non 64b triple produce the same failure? > > Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > > > ... > > > Error: invalid switch -me200 > > > Error: unrecognized option -me200 > > > > There's a block in arch/powerpc/Makefile: > > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > > as-option,-Wa$(comma)-maltivec) > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > > > Are those all broken configs, or is Kconfig messed up such that > > randconfig can select these when it should not? > > Hmmm, looks like this flag does not exist in mainline binutils? There is > a thread in 2010 about this that Segher commented on: > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > Guess this config should be eliminated? If we're going to get pestered by 0day bot randconfigs over code that's not possible to build, I'm all for deleting it. I doubt we'll be seeing patches from anyone to binutils for supporting these. What has the kernel's policy been for code in tree that other folks can't build (without proprietary tools)? (ARCH=hexagon is pretty close to not toeing the line here, not sure ICC actually works either). -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 20:14 ` Nick Desaulniers 0 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 20:14 UTC (permalink / raw) To: Nathan Chancellor, Michael Ellerman, Segher Boessenkool, Linus Torvalds, Arnd Bergmann, Brian Cain Cc: Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev On Fri, Nov 13, 2020 at 12:04 PM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > > + MPE, PPC > > > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > > <natechancellor@gmail.com> wrote: > > > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > > > Hi Fangrui, > > > > > > > > > > FYI, the error/warning still remains. > > > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > > date: 4 months ago > > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > > > ^ randconfig > > > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > > reproduce (this is a W=1 build): > > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > > chmod +x ~/bin/make.cross > > > > > # install powerpc cross compiling tool for clang build > > > > > # apt-get install binutils-powerpc-linux-gnu > > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > > git fetch --no-tags linus master > > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > # save the attached .config to linux build tree > > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > > > Assembler messages: > > > > > >> Error: invalid switch -me200 > > > > > >> Error: unrecognized option -me200 > > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > > make[2]: Target '__build' not remade because of errors. > > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > > make[1]: Target 'prepare' not remade because of errors. > > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > > > --- > > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > > option is similar to .arch or .machine and controls what instructions > > > > are recognized. The integrated assembler tends to support all > > > > instructions (conditional supporting some instructions has some > > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > > support may get less attention in LLVM.) > > > > > > This is also not a clang specific issue, I see the exact same error > > > with GCC 10.2.0 and binutils 2.35. > > > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > > > Does using a non 64b triple produce the same failure? > > Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > > > ... > > > Error: invalid switch -me200 > > > Error: unrecognized option -me200 > > > > There's a block in arch/powerpc/Makefile: > > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > > as-option,-Wa$(comma)-maltivec) > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > > > Are those all broken configs, or is Kconfig messed up such that > > randconfig can select these when it should not? > > Hmmm, looks like this flag does not exist in mainline binutils? There is > a thread in 2010 about this that Segher commented on: > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > Guess this config should be eliminated? If we're going to get pestered by 0day bot randconfigs over code that's not possible to build, I'm all for deleting it. I doubt we'll be seeing patches from anyone to binutils for supporting these. What has the kernel's policy been for code in tree that other folks can't build (without proprietary tools)? (ARCH=hexagon is pretty close to not toeing the line here, not sure ICC actually works either). -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-13 20:14 ` Nick Desaulniers 0 siblings, 0 replies; 43+ messages in thread From: Nick Desaulniers @ 2020-11-13 20:14 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 5235 bytes --] On Fri, Nov 13, 2020 at 12:04 PM Nathan Chancellor <natechancellor@gmail.com> wrote: > > On Fri, Nov 13, 2020 at 11:42:03AM -0800, Nick Desaulniers wrote: > > + MPE, PPC > > > > On Fri, Nov 13, 2020 at 11:08 AM Nathan Chancellor > > <natechancellor@gmail.com> wrote: > > > > > > On Fri, Nov 13, 2020 at 09:28:03AM -0800, Fāng-ruì Sòng wrote: > > > > On Thu, Nov 12, 2020 at 7:22 PM kernel test robot <lkp@intel.com> wrote: > > > > > > > > > > Hi Fangrui, > > > > > > > > > > FYI, the error/warning still remains. > > > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > > > > head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba > > > > > commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation > > > > > date: 4 months ago > > > > > config: powerpc-randconfig-r031-20201113 (attached as .config) > > > > ^ randconfig > > > > > > > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 9e0c35655b6e8186baef8840b26ba4090503b554) > > > > > reproduce (this is a W=1 build): > > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > > chmod +x ~/bin/make.cross > > > > > # install powerpc cross compiling tool for clang build > > > > > # apt-get install binutils-powerpc-linux-gnu > > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > > git fetch --no-tags linus master > > > > > git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 > > > > > # save the attached .config to linux build tree > > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc > > > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > > > Assembler messages: > > > > > >> Error: invalid switch -me200 > > > > > >> Error: unrecognized option -me200 > > > > > clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) > > > > > make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 > > > > > make[2]: Target '__build' not remade because of errors. > > > > > make[1]: *** [Makefile:1174: prepare0] Error 2 > > > > > make[1]: Target 'prepare' not remade because of errors. > > > > > make: *** [Makefile:185: __sub-make] Error 2 > > > > > make: Target 'prepare' not remade because of errors. > > > > > > > > > > --- > > > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > > > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org > > > > > > > > This can be ignored. The LLVM integrated assembler does not recognize > > > > -me200 (-Wa,-me200 in arch/powerpc/Makefile). I guess the GNU as -m > > > > option is similar to .arch or .machine and controls what instructions > > > > are recognized. The integrated assembler tends to support all > > > > instructions (conditional supporting some instructions has some > > > > challenges; in the end I have patched parsing but ignoring `.arch` for > > > > x86-64 and ignoring `.machine ppc64` for ppc64) > > > > > > > > (In addition, e200 is a 32-bit Power ISA microprocessor. 32-bit > > > > support may get less attention in LLVM.) > > > > > > This is also not a clang specific issue, I see the exact same error > > > with GCC 10.2.0 and binutils 2.35. > > > > > > $ make -skj64 ARCH=powerpc CROSS_COMPILE=powerpc64-linux- olddefconfig vmlinux > > > > Does using a non 64b triple produce the same failure? > > Yes, CROSS_COMPILE=powerpc-linux- produces the same failure. > > > > ... > > > Error: invalid switch -me200 > > > Error: unrecognized option -me200 > > > > There's a block in arch/powerpc/Makefile: > > 248 cpu-as-$(CONFIG_40x) += -Wa,-m405 > > 249 cpu-as-$(CONFIG_44x) += -Wa,-m440 > > 250 cpu-as-$(CONFIG_ALTIVEC) += $(call > > as-option,-Wa$(comma)-maltivec) > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > 252 cpu-as-$(CONFIG_E500) += -Wa,-me500 > > > > Are those all broken configs, or is Kconfig messed up such that > > randconfig can select these when it should not? > > Hmmm, looks like this flag does not exist in mainline binutils? There is > a thread in 2010 about this that Segher commented on: > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ > > Guess this config should be eliminated? If we're going to get pestered by 0day bot randconfigs over code that's not possible to build, I'm all for deleting it. I doubt we'll be seeing patches from anyone to binutils for supporting these. What has the kernel's policy been for code in tree that other folks can't build (without proprietary tools)? (ARCH=hexagon is pretty close to not toeing the line here, not sure ICC actually works either). -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-13 20:14 ` Nick Desaulniers @ 2020-11-14 0:20 ` Segher Boessenkool -1 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-14 0:20 UTC (permalink / raw) To: Nick Desaulniers Cc: Arnd Bergmann, kbuild-all, Brian Cain, Fāng-ruì Sòng, Masahiro Yamada, linuxppc-dev, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds, kernel test robot On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > Error: invalid switch -me200 > > > > Error: unrecognized option -me200 > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > randconfig can select these when it should not? > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > a thread in 2010 about this that Segher commented on: > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > Guess this config should be eliminated? The help text for this config options says that e200 is used in 55xx, and there *is* an -me5500 GAS flag (which probably does this same thing, too). But is any of this tested, or useful, or wanted? Maybe Christophe knows, cc:ed. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 0:20 ` Segher Boessenkool 0 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-14 0:20 UTC (permalink / raw) To: Nick Desaulniers Cc: Nathan Chancellor, Michael Ellerman, Linus Torvalds, Arnd Bergmann, Brian Cain, Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev, Christophe Leroy On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > Error: invalid switch -me200 > > > > Error: unrecognized option -me200 > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > randconfig can select these when it should not? > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > a thread in 2010 about this that Segher commented on: > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > Guess this config should be eliminated? The help text for this config options says that e200 is used in 55xx, and there *is* an -me5500 GAS flag (which probably does this same thing, too). But is any of this tested, or useful, or wanted? Maybe Christophe knows, cc:ed. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 0:20 ` Segher Boessenkool (?) @ 2020-11-14 0:37 ` Fāng-ruì Sòng -1 siblings, 0 replies; 43+ messages in thread From: Fāng-ruì Sòng @ 2020-11-14 0:37 UTC (permalink / raw) To: Alan Modra Cc: Arnd Bergmann, kbuild-all, Brian Cain, Masahiro Yamada, linuxppc-dev, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds, kernel test robot On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool <segher@kernel.crashing.org> wrote: > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > Error: invalid switch -me200 > > > > > Error: unrecognized option -me200 > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > randconfig can select these when it should not? > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > a thread in 2010 about this that Segher commented on: > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > > > Segher CC Alan Modra, a binutils global maintainer. Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? The topic started at http://lore.kernel.org/r/202011131146.g8dPLQDD-lkp@intel.com and people would like to get rid of some options (if possible). ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 0:37 ` Fāng-ruì Sòng 0 siblings, 0 replies; 43+ messages in thread From: Fāng-ruì Sòng @ 2020-11-14 0:37 UTC (permalink / raw) To: Alan Modra Cc: Nick Desaulniers, Segher Boessenkool, Nathan Chancellor, Michael Ellerman, Linus Torvalds, Arnd Bergmann, Brian Cain, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev, Christophe Leroy On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool <segher@kernel.crashing.org> wrote: > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > Error: invalid switch -me200 > > > > > Error: unrecognized option -me200 > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > randconfig can select these when it should not? > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > a thread in 2010 about this that Segher commented on: > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > > > Segher CC Alan Modra, a binutils global maintainer. Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? The topic started at http://lore.kernel.org/r/202011131146.g8dPLQDD-lkp@intel.com and people would like to get rid of some options (if possible). ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 0:37 ` Fāng-ruì Sòng 0 siblings, 0 replies; 43+ messages in thread From: Fāng-ruì Sòng @ 2020-11-14 0:37 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 1293 bytes --] On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool <segher@kernel.crashing.org> wrote: > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > Error: invalid switch -me200 > > > > > Error: unrecognized option -me200 > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > randconfig can select these when it should not? > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > a thread in 2010 about this that Segher commented on: > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ > > > > > > Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > > > Segher CC Alan Modra, a binutils global maintainer. Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? The topic started at http://lore.kernel.org/r/202011131146.g8dPLQDD-lkp(a)intel.com and people would like to get rid of some options (if possible). ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 0:37 ` Fāng-ruì Sòng @ 2020-11-14 0:50 ` Segher Boessenkool -1 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-14 0:50 UTC (permalink / raw) To: Fāng-ruì Sòng Cc: Arnd Bergmann, kbuild-all, kernel test robot, Brian Cain, Alan Modra, Masahiro Yamada, linuxppc-dev, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > <segher@kernel.crashing.org> wrote: > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > Error: invalid switch -me200 > > > > > > Error: unrecognized option -me200 > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > randconfig can select these when it should not? > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > Guess this config should be eliminated? > > > > The help text for this config options says that e200 is used in 55xx, > > and there *is* an -me5500 GAS flag (which probably does this same > > thing, too). But is any of this tested, or useful, or wanted? > > > > Maybe Christophe knows, cc:ed. > > CC Alan Modra, a binutils global maintainer. > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? All the others work fine (and are needed afaics), it is only -me200 that doesn't exist (in mainline binutils). Perhaps -me5500 will work for it instead. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 0:50 ` Segher Boessenkool 0 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-14 0:50 UTC (permalink / raw) To: Fāng-ruì Sòng Cc: Alan Modra, Nick Desaulniers, Nathan Chancellor, Michael Ellerman, Linus Torvalds, Arnd Bergmann, Brian Cain, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev, Christophe Leroy On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > <segher@kernel.crashing.org> wrote: > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > Error: invalid switch -me200 > > > > > > Error: unrecognized option -me200 > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > randconfig can select these when it should not? > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > Guess this config should be eliminated? > > > > The help text for this config options says that e200 is used in 55xx, > > and there *is* an -me5500 GAS flag (which probably does this same > > thing, too). But is any of this tested, or useful, or wanted? > > > > Maybe Christophe knows, cc:ed. > > CC Alan Modra, a binutils global maintainer. > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? All the others work fine (and are needed afaics), it is only -me200 that doesn't exist (in mainline binutils). Perhaps -me5500 will work for it instead. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 0:50 ` Segher Boessenkool (?) @ 2020-11-16 20:27 ` Scott Wood -1 siblings, 0 replies; 43+ messages in thread From: Scott Wood @ 2020-11-16 20:27 UTC (permalink / raw) To: Segher Boessenkool, Fāng-ruì Sòng Cc: Arnd Bergmann, kbuild-all, kernel test robot, Brian Cain, Alan Modra, Linus Torvalds, Masahiro Yamada, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, linuxppc-dev On Fri, 2020-11-13 at 18:50 -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? > > > > > There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Perhaps -me5500 will work for it > instead. According to Wikipedia e200 is from mpc55xx (for which I don't see any platform support having ever been added). e5500 is completely different (64- bit version of e500mc). -Scott ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 20:27 ` Scott Wood 0 siblings, 0 replies; 43+ messages in thread From: Scott Wood @ 2020-11-16 20:27 UTC (permalink / raw) To: Segher Boessenkool, Fāng-ruì Sòng Cc: Arnd Bergmann, kbuild-all, kernel test robot, Brian Cain, Alan Modra, Masahiro Yamada, linuxppc-dev, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds On Fri, 2020-11-13 at 18:50 -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? > > > > > There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Perhaps -me5500 will work for it > instead. According to Wikipedia e200 is from mpc55xx (for which I don't see any platform support having ever been added). e5500 is completely different (64- bit version of e500mc). -Scott ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 20:27 ` Scott Wood 0 siblings, 0 replies; 43+ messages in thread From: Scott Wood @ 2020-11-16 20:27 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 1741 bytes --] On Fri, 2020-11-13 at 18:50 -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? > > > > > There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Perhaps -me5500 will work for it > instead. According to Wikipedia e200 is from mpc55xx (for which I don't see any platform support having ever been added). e5500 is completely different (64- bit version of e500mc). -Scott ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-16 20:27 ` Scott Wood @ 2020-11-16 20:47 ` Segher Boessenkool -1 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-16 20:47 UTC (permalink / raw) To: Scott Wood Cc: Arnd Bergmann, kbuild-all, kernel test robot, Brian Cain, Alan Modra, Linus Torvalds, Masahiro Yamada, Nick Desaulniers, LKML, clang-built-linux, Fāng-ruì Sòng, Nathan Chancellor, linuxppc-dev On Mon, Nov 16, 2020 at 02:27:12PM -0600, Scott Wood wrote: > On Fri, 2020-11-13 at 18:50 -0600, Segher Boessenkool wrote: > > All the others work fine (and are needed afaics), it is only -me200 that > > doesn't exist (in mainline binutils). Perhaps -me5500 will work for it > > instead. > > According to Wikipedia e200 is from mpc55xx (for which I don't see any > platform support having ever been added). e5500 is completely different (64- > bit version of e500mc). Ah yes, confusing processor numbers :-( That explains, sorry. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 20:47 ` Segher Boessenkool 0 siblings, 0 replies; 43+ messages in thread From: Segher Boessenkool @ 2020-11-16 20:47 UTC (permalink / raw) To: Scott Wood Cc: Fāng-ruì Sòng, Arnd Bergmann, kbuild-all, kernel test robot, Brian Cain, Alan Modra, Masahiro Yamada, linuxppc-dev, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds On Mon, Nov 16, 2020 at 02:27:12PM -0600, Scott Wood wrote: > On Fri, 2020-11-13 at 18:50 -0600, Segher Boessenkool wrote: > > All the others work fine (and are needed afaics), it is only -me200 that > > doesn't exist (in mainline binutils). Perhaps -me5500 will work for it > > instead. > > According to Wikipedia e200 is from mpc55xx (for which I don't see any > platform support having ever been added). e5500 is completely different (64- > bit version of e500mc). Ah yes, confusing processor numbers :-( That explains, sorry. Segher ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 0:50 ` Segher Boessenkool (?) @ 2020-11-16 23:59 ` Alan Modra -1 siblings, 0 replies; 43+ messages in thread From: Alan Modra @ 2020-11-16 23:59 UTC (permalink / raw) To: Segher Boessenkool Cc: Arnd Bergmann, kbuild-all, Brian Cain, Fāng-ruì Sòng, Masahiro Yamada, linuxppc-dev, Nick Desaulniers, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds, kernel test robot On Fri, Nov 13, 2020 at 06:50:15PM -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Right, and a quick check says it never existed. There is e200z4, added to binutils with dfdaec14b0d, 2016-08-01, but the kernel -me200 was added in 2005. I suspect the toolchain support only existed inside Freescale and pushing it upstream was too difficult. -- Alan Modra Australia Development Lab, IBM ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 23:59 ` Alan Modra 0 siblings, 0 replies; 43+ messages in thread From: Alan Modra @ 2020-11-16 23:59 UTC (permalink / raw) To: Segher Boessenkool Cc: Fāng-ruì Sòng, Nick Desaulniers, Nathan Chancellor, Michael Ellerman, Linus Torvalds, Arnd Bergmann, Brian Cain, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev, Christophe Leroy On Fri, Nov 13, 2020 at 06:50:15PM -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Right, and a quick check says it never existed. There is e200z4, added to binutils with dfdaec14b0d, 2016-08-01, but the kernel -me200 was added in 2005. I suspect the toolchain support only existed inside Freescale and pushing it upstream was too difficult. -- Alan Modra Australia Development Lab, IBM ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 23:59 ` Alan Modra 0 siblings, 0 replies; 43+ messages in thread From: Alan Modra @ 2020-11-16 23:59 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 1808 bytes --] On Fri, Nov 13, 2020 at 06:50:15PM -0600, Segher Boessenkool wrote: > On Fri, Nov 13, 2020 at 04:37:38PM -0800, Fāng-ruì Sòng wrote: > > On Fri, Nov 13, 2020 at 4:23 PM Segher Boessenkool > > <segher@kernel.crashing.org> wrote: > > > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: > > > > > > > Error: invalid switch -me200 > > > > > > > Error: unrecognized option -me200 > > > > > > > > > > > > 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 > > > > > > > > > > > > Are those all broken configs, or is Kconfig messed up such that > > > > > > randconfig can select these when it should not? > > > > > > > > > > Hmmm, looks like this flag does not exist in mainline binutils? There is > > > > > a thread in 2010 about this that Segher commented on: > > > > > > > > > > https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ > > > > > > > > > > Guess this config should be eliminated? > > > > > > The help text for this config options says that e200 is used in 55xx, > > > and there *is* an -me5500 GAS flag (which probably does this same > > > thing, too). But is any of this tested, or useful, or wanted? > > > > > > Maybe Christophe knows, cc:ed. > > > > CC Alan Modra, a binutils global maintainer. > > > > Alan, can the few -Wa,-m* options deleted from arch/powerpc/Makefile ? > > All the others work fine (and are needed afaics), it is only -me200 that > doesn't exist (in mainline binutils). Right, and a quick check says it never existed. There is e200z4, added to binutils with dfdaec14b0d, 2016-08-01, but the kernel -me200 was added in 2005. I suspect the toolchain support only existed inside Freescale and pushing it upstream was too difficult. -- Alan Modra Australia Development Lab, IBM ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 0:20 ` Segher Boessenkool (?) @ 2020-11-14 9:02 ` Christophe Leroy -1 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-14 9:02 UTC (permalink / raw) To: Segher Boessenkool, Nick Desaulniers Cc: Arnd Bergmann, kbuild-all, Brian Cain, Fāng-ruì Sòng, Masahiro Yamada, linuxppc-dev, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds, kernel test robot Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>> Error: invalid switch -me200 >>>>> Error: unrecognized option -me200 >>>> >>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>> >>>> Are those all broken configs, or is Kconfig messed up such that >>>> randconfig can select these when it should not? >>> >>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>> a thread in 2010 about this that Segher commented on: >>> >>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>> >>> Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > I don't have much clue on this. But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 e200z3 e200z4 e200z6 e200z7): valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 Apparently based on binutils 2.28 https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf But that's not exactly -me200 though. Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel at all ? Christophe ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 9:02 ` Christophe Leroy 0 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-14 9:02 UTC (permalink / raw) To: Segher Boessenkool, Nick Desaulniers Cc: Nathan Chancellor, Michael Ellerman, Linus Torvalds, Arnd Bergmann, Brian Cain, Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>> Error: invalid switch -me200 >>>>> Error: unrecognized option -me200 >>>> >>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>> >>>> Are those all broken configs, or is Kconfig messed up such that >>>> randconfig can select these when it should not? >>> >>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>> a thread in 2010 about this that Segher commented on: >>> >>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>> >>> Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > I don't have much clue on this. But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 e200z3 e200z4 e200z6 e200z7): valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 Apparently based on binutils 2.28 https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf But that's not exactly -me200 though. Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel at all ? Christophe ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-14 9:02 ` Christophe Leroy 0 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-14 9:02 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 1973 bytes --] Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : > On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>> Error: invalid switch -me200 >>>>> Error: unrecognized option -me200 >>>> >>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>> >>>> Are those all broken configs, or is Kconfig messed up such that >>>> randconfig can select these when it should not? >>> >>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>> a thread in 2010 about this that Segher commented on: >>> >>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ >>> >>> Guess this config should be eliminated? > > The help text for this config options says that e200 is used in 55xx, > and there *is* an -me5500 GAS flag (which probably does this same > thing, too). But is any of this tested, or useful, or wanted? > > Maybe Christophe knows, cc:ed. > I don't have much clue on this. But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 e200z3 e200z4 e200z6 e200z7): valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 Apparently based on binutils 2.28 https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf But that's not exactly -me200 though. Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel at all ? Christophe ^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-14 9:02 ` Christophe Leroy (?) @ 2020-11-16 6:06 ` Michael Ellerman -1 siblings, 0 replies; 43+ messages in thread From: Michael Ellerman @ 2020-11-16 6:06 UTC (permalink / raw) To: Christophe Leroy, Segher Boessenkool, Nick Desaulniers, mihai.caraman Cc: Arnd Bergmann, kbuild-all, Brian Cain, Fāng-ruì Sòng, linuxppc-dev, Masahiro Yamada, LKML, clang-built-linux, Nathan Chancellor, Linus Torvalds, kernel test robot Christophe Leroy <christophe.leroy@csgroup.eu> writes: > Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>> Error: invalid switch -me200 >>>>>> Error: unrecognized option -me200 >>>>> >>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>> >>>>> Are those all broken configs, or is Kconfig messed up such that >>>>> randconfig can select these when it should not? >>>> >>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>> a thread in 2010 about this that Segher commented on: >>>> >>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>>> >>>> Guess this config should be eliminated? >> >> The help text for this config options says that e200 is used in 55xx, >> and there *is* an -me5500 GAS flag (which probably does this same >> thing, too). But is any of this tested, or useful, or wanted? >> >> Maybe Christophe knows, cc:ed. >> > > I don't have much clue on this. Me either. > But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) > > What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7): > > valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 > 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 > power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " > > https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 > > Apparently based on binutils 2.28 > > https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf > > But that's not exactly -me200 though. > > Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel > at all ? There was a commit in 2014 that suggests it worked at least to some extent then: 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to e200/e500 cores") Presumably there was a non-upstream toolchain where it was supported? AFAICS the kernel builds OK with just the cpu-as modification removed: diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile index a4d56f0a41d9..16b8336f91dd 100644 --- a/arch/powerpc/Makefile +++ b/arch/powerpc/Makefile @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call cc-option,-mno-string) cpu-as-$(CONFIG_40x) += -Wa,-m405 cpu-as-$(CONFIG_44x) += -Wa,-m440 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) -cpu-as-$(CONFIG_E200) += -Wa,-me200 cpu-as-$(CONFIG_E500) += -Wa,-me500 # When using '-many -mpower4' gas will first try and find a matching power4 So that seems like the obvious fix for now. I tried booting the resulting kernel in qemu, but I get: $ qemu-system-ppc -M none -cpu e200 -kernel build\~/vmlinux Error: Trying to register SPR 574 (23e) twice ! Which is not related AFAIK and indicates the qemu support is broken. Unless we hear from someone that they're using mainline on an e200 then it seems like it's a candidate for removal. cheers ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 6:06 ` Michael Ellerman 0 siblings, 0 replies; 43+ messages in thread From: Michael Ellerman @ 2020-11-16 6:06 UTC (permalink / raw) To: Christophe Leroy, Segher Boessenkool, Nick Desaulniers, mihai.caraman Cc: Nathan Chancellor, Linus Torvalds, Arnd Bergmann, Brian Cain, Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev Christophe Leroy <christophe.leroy@csgroup.eu> writes: > Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>> Error: invalid switch -me200 >>>>>> Error: unrecognized option -me200 >>>>> >>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>> >>>>> Are those all broken configs, or is Kconfig messed up such that >>>>> randconfig can select these when it should not? >>>> >>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>> a thread in 2010 about this that Segher commented on: >>>> >>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>>> >>>> Guess this config should be eliminated? >> >> The help text for this config options says that e200 is used in 55xx, >> and there *is* an -me5500 GAS flag (which probably does this same >> thing, too). But is any of this tested, or useful, or wanted? >> >> Maybe Christophe knows, cc:ed. >> > > I don't have much clue on this. Me either. > But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) > > What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7): > > valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 > 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 > power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " > > https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 > > Apparently based on binutils 2.28 > > https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf > > But that's not exactly -me200 though. > > Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel > at all ? There was a commit in 2014 that suggests it worked at least to some extent then: 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to e200/e500 cores") Presumably there was a non-upstream toolchain where it was supported? AFAICS the kernel builds OK with just the cpu-as modification removed: diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile index a4d56f0a41d9..16b8336f91dd 100644 --- a/arch/powerpc/Makefile +++ b/arch/powerpc/Makefile @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call cc-option,-mno-string) cpu-as-$(CONFIG_40x) += -Wa,-m405 cpu-as-$(CONFIG_44x) += -Wa,-m440 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) -cpu-as-$(CONFIG_E200) += -Wa,-me200 cpu-as-$(CONFIG_E500) += -Wa,-me500 # When using '-many -mpower4' gas will first try and find a matching power4 So that seems like the obvious fix for now. I tried booting the resulting kernel in qemu, but I get: $ qemu-system-ppc -M none -cpu e200 -kernel build\~/vmlinux Error: Trying to register SPR 574 (23e) twice ! Which is not related AFAIK and indicates the qemu support is broken. Unless we hear from someone that they're using mainline on an e200 then it seems like it's a candidate for removal. cheers ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 6:06 ` Michael Ellerman 0 siblings, 0 replies; 43+ messages in thread From: Michael Ellerman @ 2020-11-16 6:06 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 3420 bytes --] Christophe Leroy <christophe.leroy@csgroup.eu> writes: > Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>> Error: invalid switch -me200 >>>>>> Error: unrecognized option -me200 >>>>> >>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>> >>>>> Are those all broken configs, or is Kconfig messed up such that >>>>> randconfig can select these when it should not? >>>> >>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>> a thread in 2010 about this that Segher commented on: >>>> >>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ >>>> >>>> Guess this config should be eliminated? >> >> The help text for this config options says that e200 is used in 55xx, >> and there *is* an -me5500 GAS flag (which probably does this same >> thing, too). But is any of this tested, or useful, or wanted? >> >> Maybe Christophe knows, cc:ed. >> > > I don't have much clue on this. Me either. > But I see on wikipedia that e5500 is a 64 bits powerpc (https://en.wikipedia.org/wiki/PowerPC_e5500) > > What I see is that NXP seems to provide a GCC version that includes aditionnal cpu (e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7): > > valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 464fp 476 476fp 505 601 602 603 > 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 970 G3 G4 G5 a2 cell e200z0 e200z2 > e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 e6500 ec603e native power3 power4 > power5 power5+ power6 power6x power7 power8 powerpc powerpc64 powerpc64le rs64 titan " > > https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 > > Apparently based on binutils 2.28 > > https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf > > But that's not exactly -me200 though. > > Now, I can't see any defconfig that selects CONFIG_E200, so is that worth keeping it in the kernel > at all ? There was a commit in 2014 that suggests it worked at least to some extent then: 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to e200/e500 cores") Presumably there was a non-upstream toolchain where it was supported? AFAICS the kernel builds OK with just the cpu-as modification removed: diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile index a4d56f0a41d9..16b8336f91dd 100644 --- a/arch/powerpc/Makefile +++ b/arch/powerpc/Makefile @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call cc-option,-mno-string) cpu-as-$(CONFIG_40x) += -Wa,-m405 cpu-as-$(CONFIG_44x) += -Wa,-m440 cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) -cpu-as-$(CONFIG_E200) += -Wa,-me200 cpu-as-$(CONFIG_E500) += -Wa,-me500 # When using '-many -mpower4' gas will first try and find a matching power4 So that seems like the obvious fix for now. I tried booting the resulting kernel in qemu, but I get: $ qemu-system-ppc -M none -cpu e200 -kernel build\~/vmlinux Error: Trying to register SPR 574 (23e) twice ! Which is not related AFAIK and indicates the qemu support is broken. Unless we hear from someone that they're using mainline on an e200 then it seems like it's a candidate for removal. cheers ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 2020-11-16 6:06 ` Michael Ellerman (?) @ 2020-11-16 15:26 ` Christophe Leroy -1 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-16 15:26 UTC (permalink / raw) To: Michael Ellerman Cc: Arnd Bergmann, kbuild-all, Brian Cain, linuxppc-dev, Masahiro Yamada, Nick Desaulniers, LKML, clang-built-linux, Fāng-ruì Sòng, mihai.caraman, Nathan Chancellor, Linus Torvalds, kernel test robot Quoting Michael Ellerman <mpe@ellerman.id.au>: > Christophe Leroy <christophe.leroy@csgroup.eu> writes: >> Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >>> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>>> Error: invalid switch -me200 >>>>>>> Error: unrecognized option -me200 >>>>>> >>>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>>> >>>>>> Are those all broken configs, or is Kconfig messed up such that >>>>>> randconfig can select these when it should not? >>>>> >>>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>>> a thread in 2010 about this that Segher commented on: >>>>> >>>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>>>> >>>>> Guess this config should be eliminated? >>> >>> The help text for this config options says that e200 is used in 55xx, >>> and there *is* an -me5500 GAS flag (which probably does this same >>> thing, too). But is any of this tested, or useful, or wanted? >>> >>> Maybe Christophe knows, cc:ed. >>> >> >> I don't have much clue on this. > > Me either. > >> But I see on wikipedia that e5500 is a 64 bits powerpc >> (https://en.wikipedia.org/wiki/PowerPC_e5500) >> >> What I see is that NXP seems to provide a GCC version that includes >> aditionnal cpu (e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7): >> >> valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 >> 464fp 476 476fp 505 601 602 603 >> 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 >> 970 G3 G4 G5 a2 cell e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 >> e6500 ec603e native power3 power4 >> power5 power5+ power6 power6x power7 power8 powerpc powerpc64 >> powerpc64le rs64 titan " >> >> https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 >> >> Apparently based on binutils 2.28 >> >> https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf >> >> But that's not exactly -me200 though. >> >> Now, I can't see any defconfig that selects CONFIG_E200, so is that >> worth keeping it in the kernel >> at all ? > > There was a commit in 2014 that suggests it worked at least to some > extent then: > > 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to > e200/e500 cores") Not sure, that patch seems to be focussed on the new e500mc > > > Presumably there was a non-upstream toolchain where it was supported? > > AFAICS the kernel builds OK with just the cpu-as modification removed: > > diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile > index a4d56f0a41d9..16b8336f91dd 100644 > --- a/arch/powerpc/Makefile > +++ b/arch/powerpc/Makefile > @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call > cc-option,-mno-string) > cpu-as-$(CONFIG_40x) += -Wa,-m405 > cpu-as-$(CONFIG_44x) += -Wa,-m440 > cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) > -cpu-as-$(CONFIG_E200) += -Wa,-me200 > cpu-as-$(CONFIG_E500) += -Wa,-me500 > > # When using '-many -mpower4' gas will first try and find a matching power4 > > > So that seems like the obvious fix for now. Or we could do diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype index c194c4ae8bc7..a11cf9431e1e 100644 --- a/arch/powerpc/platforms/Kconfig.cputype +++ b/arch/powerpc/platforms/Kconfig.cputype @@ -67,6 +67,7 @@ config 44x select PHYS_64BIT config E200 + depends on $(cc-option,-me200) bool "Freescale e200" endchoice --- Christophe ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 15:26 ` Christophe Leroy 0 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-16 15:26 UTC (permalink / raw) To: Michael Ellerman Cc: Segher Boessenkool, Nick Desaulniers, mihai.caraman, Nathan Chancellor, Linus Torvalds, Arnd Bergmann, Brian Cain, Fāng-ruì Sòng, kernel test robot, kbuild-all, clang-built-linux, LKML, Masahiro Yamada, linuxppc-dev Quoting Michael Ellerman <mpe@ellerman.id.au>: > Christophe Leroy <christophe.leroy@csgroup.eu> writes: >> Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >>> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>>> Error: invalid switch -me200 >>>>>>> Error: unrecognized option -me200 >>>>>> >>>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>>> >>>>>> Are those all broken configs, or is Kconfig messed up such that >>>>>> randconfig can select these when it should not? >>>>> >>>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>>> a thread in 2010 about this that Segher commented on: >>>>> >>>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E@kernel.crashing.org/ >>>>> >>>>> Guess this config should be eliminated? >>> >>> The help text for this config options says that e200 is used in 55xx, >>> and there *is* an -me5500 GAS flag (which probably does this same >>> thing, too). But is any of this tested, or useful, or wanted? >>> >>> Maybe Christophe knows, cc:ed. >>> >> >> I don't have much clue on this. > > Me either. > >> But I see on wikipedia that e5500 is a 64 bits powerpc >> (https://en.wikipedia.org/wiki/PowerPC_e5500) >> >> What I see is that NXP seems to provide a GCC version that includes >> aditionnal cpu (e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7): >> >> valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 >> 464fp 476 476fp 505 601 602 603 >> 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 >> 970 G3 G4 G5 a2 cell e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 >> e6500 ec603e native power3 power4 >> power5 power5+ power6 power6x power7 power8 powerpc powerpc64 >> powerpc64le rs64 titan " >> >> https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 >> >> Apparently based on binutils 2.28 >> >> https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf >> >> But that's not exactly -me200 though. >> >> Now, I can't see any defconfig that selects CONFIG_E200, so is that >> worth keeping it in the kernel >> at all ? > > There was a commit in 2014 that suggests it worked at least to some > extent then: > > 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to > e200/e500 cores") Not sure, that patch seems to be focussed on the new e500mc > > > Presumably there was a non-upstream toolchain where it was supported? > > AFAICS the kernel builds OK with just the cpu-as modification removed: > > diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile > index a4d56f0a41d9..16b8336f91dd 100644 > --- a/arch/powerpc/Makefile > +++ b/arch/powerpc/Makefile > @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call > cc-option,-mno-string) > cpu-as-$(CONFIG_40x) += -Wa,-m405 > cpu-as-$(CONFIG_44x) += -Wa,-m440 > cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) > -cpu-as-$(CONFIG_E200) += -Wa,-me200 > cpu-as-$(CONFIG_E500) += -Wa,-me500 > > # When using '-many -mpower4' gas will first try and find a matching power4 > > > So that seems like the obvious fix for now. Or we could do diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype index c194c4ae8bc7..a11cf9431e1e 100644 --- a/arch/powerpc/platforms/Kconfig.cputype +++ b/arch/powerpc/platforms/Kconfig.cputype @@ -67,6 +67,7 @@ config 44x select PHYS_64BIT config E200 + depends on $(cc-option,-me200) bool "Freescale e200" endchoice --- Christophe ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: Error: invalid switch -me200 @ 2020-11-16 15:26 ` Christophe Leroy 0 siblings, 0 replies; 43+ messages in thread From: Christophe Leroy @ 2020-11-16 15:26 UTC (permalink / raw) To: kbuild-all [-- Attachment #1: Type: text/plain, Size: 3708 bytes --] Quoting Michael Ellerman <mpe@ellerman.id.au>: > Christophe Leroy <christophe.leroy@csgroup.eu> writes: >> Le 14/11/2020 à 01:20, Segher Boessenkool a écrit : >>> On Fri, Nov 13, 2020 at 12:14:18PM -0800, Nick Desaulniers wrote: >>>>>>> Error: invalid switch -me200 >>>>>>> Error: unrecognized option -me200 >>>>>> >>>>>> 251 cpu-as-$(CONFIG_E200) += -Wa,-me200 >>>>>> >>>>>> Are those all broken configs, or is Kconfig messed up such that >>>>>> randconfig can select these when it should not? >>>>> >>>>> Hmmm, looks like this flag does not exist in mainline binutils? There is >>>>> a thread in 2010 about this that Segher commented on: >>>>> >>>>> https://lore.kernel.org/linuxppc-dev/9859E645-954D-4D07-8003-FFCD2391AB6E(a)kernel.crashing.org/ >>>>> >>>>> Guess this config should be eliminated? >>> >>> The help text for this config options says that e200 is used in 55xx, >>> and there *is* an -me5500 GAS flag (which probably does this same >>> thing, too). But is any of this tested, or useful, or wanted? >>> >>> Maybe Christophe knows, cc:ed. >>> >> >> I don't have much clue on this. > > Me either. > >> But I see on wikipedia that e5500 is a 64 bits powerpc >> (https://en.wikipedia.org/wiki/PowerPC_e5500) >> >> What I see is that NXP seems to provide a GCC version that includes >> aditionnal cpu (e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7): >> >> valid arguments to '-mcpu=' are: 401 403 405 405fp 440 440fp 464 >> 464fp 476 476fp 505 601 602 603 >> 603e 604 604e 620 630 740 7400 7450 750 801 821 823 8540 8548 860 >> 970 G3 G4 G5 a2 cell e200z0 e200z2 >> e200z3 e200z4 e200z6 e200z7 e300c2 e300c3 e500mc e500mc64 e5500 >> e6500 ec603e native power3 power4 >> power5 power5+ power6 power6x power7 power8 powerpc powerpc64 >> powerpc64le rs64 titan " >> >> https://community.nxp.com/t5/MPC5xxx/GCC-generating-not-implemented-instructions/m-p/845049 >> >> Apparently based on binutils 2.28 >> >> https://www.nxp.com/docs/en/release-note/S32DS-POWER-v1-2-RN.pdf >> >> But that's not exactly -me200 though. >> >> Now, I can't see any defconfig that selects CONFIG_E200, so is that >> worth keeping it in the kernel >> at all ? > > There was a commit in 2014 that suggests it worked at least to some > extent then: > > 3477e71d5319 ("powerpc/booke: Restrict SPE exception handlers to > e200/e500 cores") Not sure, that patch seems to be focussed on the new e500mc > > > Presumably there was a non-upstream toolchain where it was supported? > > AFAICS the kernel builds OK with just the cpu-as modification removed: > > diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile > index a4d56f0a41d9..16b8336f91dd 100644 > --- a/arch/powerpc/Makefile > +++ b/arch/powerpc/Makefile > @@ -248,7 +248,6 @@ KBUILD_CFLAGS += $(call > cc-option,-mno-string) > cpu-as-$(CONFIG_40x) += -Wa,-m405 > cpu-as-$(CONFIG_44x) += -Wa,-m440 > cpu-as-$(CONFIG_ALTIVEC) += $(call as-option,-Wa$(comma)-maltivec) > -cpu-as-$(CONFIG_E200) += -Wa,-me200 > cpu-as-$(CONFIG_E500) += -Wa,-me500 > > # When using '-many -mpower4' gas will first try and find a matching power4 > > > So that seems like the obvious fix for now. Or we could do diff --git a/arch/powerpc/platforms/Kconfig.cputype b/arch/powerpc/platforms/Kconfig.cputype index c194c4ae8bc7..a11cf9431e1e 100644 --- a/arch/powerpc/platforms/Kconfig.cputype +++ b/arch/powerpc/platforms/Kconfig.cputype @@ -67,6 +67,7 @@ config 44x select PHYS_64BIT config E200 + depends on $(cc-option,-me200) bool "Freescale e200" endchoice --- Christophe ^ permalink raw reply related [flat|nested] 43+ messages in thread
* Error: invalid switch -me200
@ 2020-10-02 8:48 ` kernel test robot
0 siblings, 0 replies; 43+ messages in thread
From: kernel test robot @ 2020-10-02 8:48 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 2090 bytes --]
Hi Fangrui,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 472e5b056f000a778abb41f1e443de58eb259783
commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation
date: 2 months ago
config: powerpc64-randconfig-r022-20201002 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc64 cross compiling tool for clang build
# apt-get install binutils-powerpc64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
Assembler messages:
>> Error: invalid switch -me200
>> Error: unrecognized option -me200
clang-12: error: assembler command failed with exit code 1 (use -v to see invocation)
make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1174: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 27188 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread* Error: invalid switch -me200 @ 2020-10-02 8:48 ` kernel test robot 0 siblings, 0 replies; 43+ messages in thread From: kernel test robot @ 2020-10-02 8:48 UTC (permalink / raw) To: Fangrui Song Cc: kbuild-all, clang-built-linux, linux-kernel, Masahiro Yamada, Nathan Chancellor, Nick Desaulniers [-- Attachment #1: Type: text/plain, Size: 2047 bytes --] Hi Fangrui, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 472e5b056f000a778abb41f1e443de58eb259783 commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation date: 2 months ago config: powerpc64-randconfig-r022-20201002 (attached as .config) compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install powerpc64 cross compiling tool for clang build # apt-get install binutils-powerpc64-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): Assembler messages: >> Error: invalid switch -me200 >> Error: unrecognized option -me200 clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 make[2]: Target '__build' not remade because of errors. make[1]: *** [Makefile:1174: prepare0] Error 2 make[1]: Target 'prepare' not remade because of errors. make: *** [Makefile:185: __sub-make] Error 2 make: Target 'prepare' not remade because of errors. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 27188 bytes --] ^ permalink raw reply [flat|nested] 43+ messages in thread
* Error: invalid switch -me200
@ 2020-07-29 16:50 ` kernel test robot
0 siblings, 0 replies; 43+ messages in thread
From: kernel test robot @ 2020-07-29 16:50 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 1755 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 6ba1b005ffc388c2aeaddae20da29e4810dea298
commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation
date: 7 days ago
config: powerpc-randconfig-r014-20200729 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 417d3d495f1cfb0a2f7b60d00829925126fdcfd9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
Assembler messages:
>> Error: invalid switch -me200
>> Error: unrecognized option -me200
clang-12: error: assembler command failed with exit code 1 (use -v to see invocation)
make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1174: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 26260 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread* Error: invalid switch -me200 @ 2020-07-29 16:50 ` kernel test robot 0 siblings, 0 replies; 43+ messages in thread From: kernel test robot @ 2020-07-29 16:50 UTC (permalink / raw) To: Fangrui Song Cc: kbuild-all, clang-built-linux, linux-kernel, Masahiro Yamada, Nathan Chancellor, Nick Desaulniers [-- Attachment #1: Type: text/plain, Size: 1719 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 6ba1b005ffc388c2aeaddae20da29e4810dea298 commit: ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross compilation date: 7 days ago config: powerpc-randconfig-r014-20200729 (attached as .config) compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 417d3d495f1cfb0a2f7b60d00829925126fdcfd9) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install powerpc cross compiling tool for clang build # apt-get install binutils-powerpc-linux-gnu git checkout ca9b31f6bb9c6aa9b4e5f0792f39a97bbffb8c51 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): Assembler messages: >> Error: invalid switch -me200 >> Error: unrecognized option -me200 clang-12: error: assembler command failed with exit code 1 (use -v to see invocation) make[2]: *** [scripts/Makefile.build:281: scripts/mod/empty.o] Error 1 make[2]: Target '__build' not remade because of errors. make[1]: *** [Makefile:1174: prepare0] Error 2 make[1]: Target 'prepare' not remade because of errors. make: *** [Makefile:185: __sub-make] Error 2 make: Target 'prepare' not remade because of errors. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 26260 bytes --] ^ permalink raw reply [flat|nested] 43+ messages in thread
end of thread, other threads:[~2020-11-17 0:01 UTC | newest] Thread overview: 43+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-11-13 3:21 Error: invalid switch -me200 kernel test robot 2020-11-13 3:21 ` kernel test robot 2020-11-13 17:28 ` Fāng-ruì Sòng 2020-11-13 17:28 ` Fāng-ruì Sòng 2020-11-13 19:08 ` Nathan Chancellor 2020-11-13 19:08 ` Nathan Chancellor 2020-11-13 19:42 ` Nick Desaulniers 2020-11-13 19:42 ` Nick Desaulniers 2020-11-13 19:42 ` Nick Desaulniers 2020-11-13 20:04 ` Nathan Chancellor 2020-11-13 20:04 ` Nathan Chancellor 2020-11-13 20:04 ` Nathan Chancellor 2020-11-13 20:14 ` Nick Desaulniers 2020-11-13 20:14 ` Nick Desaulniers 2020-11-13 20:14 ` Nick Desaulniers 2020-11-14 0:20 ` Segher Boessenkool 2020-11-14 0:20 ` Segher Boessenkool 2020-11-14 0:37 ` Fāng-ruì Sòng 2020-11-14 0:37 ` Fāng-ruì Sòng 2020-11-14 0:37 ` Fāng-ruì Sòng 2020-11-14 0:50 ` Segher Boessenkool 2020-11-14 0:50 ` Segher Boessenkool 2020-11-16 20:27 ` Scott Wood 2020-11-16 20:27 ` Scott Wood 2020-11-16 20:27 ` Scott Wood 2020-11-16 20:47 ` Segher Boessenkool 2020-11-16 20:47 ` Segher Boessenkool 2020-11-16 23:59 ` Alan Modra 2020-11-16 23:59 ` Alan Modra 2020-11-16 23:59 ` Alan Modra 2020-11-14 9:02 ` Christophe Leroy 2020-11-14 9:02 ` Christophe Leroy 2020-11-14 9:02 ` Christophe Leroy 2020-11-16 6:06 ` Michael Ellerman 2020-11-16 6:06 ` Michael Ellerman 2020-11-16 6:06 ` Michael Ellerman 2020-11-16 15:26 ` Christophe Leroy 2020-11-16 15:26 ` Christophe Leroy 2020-11-16 15:26 ` Christophe Leroy -- strict thread matches above, loose matches on Subject: below -- 2020-10-02 8:48 kernel test robot 2020-10-02 8:48 ` kernel test robot 2020-07-29 16:50 kernel test robot 2020-07-29 16:50 ` kernel test robot
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.