public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed

• * Re: Error: invalid switch -me200
         [not found]   ` <20201113190824.GA1477315@ubuntu-m3-large-x86>
  @ 2020-11-13 19:42     ` Nick Desaulniers
    2020-11-13 20:04       ` Nathan Chancellor
    0 siblings, 1 reply; 12+ 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] 12+ messages in thread

• • * Re: Error: invalid switch -me200
      2020-11-13 19:42     ` Error: invalid switch -me200 Nick Desaulniers
    @ 2020-11-13 20:04       ` Nathan Chancellor
      2020-11-13 20:14         ` Nick Desaulniers
      0 siblings, 1 reply; 12+ 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] 12+ messages in thread

  • • * Re: Error: invalid switch -me200
        2020-11-13 20:04       ` Nathan Chancellor
      @ 2020-11-13 20:14         ` Nick Desaulniers
        2020-11-14  0:20           ` Segher Boessenkool
        0 siblings, 1 reply; 12+ 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] 12+ messages in thread

    • • * Re: Error: invalid switch -me200
          2020-11-13 20:14         ` Nick Desaulniers
        @ 2020-11-14  0:20           ` Segher Boessenkool
          2020-11-14  0:37             ` Fāng-ruì Sòng
          2020-11-14  9:02             ` Christophe Leroy
          0 siblings, 2 replies; 12+ 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] 12+ 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
            2020-11-14  0:50               ` Segher Boessenkool
            2020-11-14  9:02             ` Christophe Leroy
            1 sibling, 1 reply; 12+ 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] 12+ 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
              2020-11-16 20:27                 ` Scott Wood
              2020-11-16 23:59                 ` Alan Modra
              0 siblings, 2 replies; 12+ 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] 12+ messages in thread

          • • * Re: Error: invalid switch -me200
                2020-11-14  0:50               ` Segher Boessenkool
              @ 2020-11-16 20:27                 ` Scott Wood
                2020-11-16 20:47                   ` Segher Boessenkool
                2020-11-16 23:59                 ` Alan Modra
                1 sibling, 1 reply; 12+ 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] 12+ messages in thread

            • • * Re: Error: invalid switch -me200
                  2020-11-16 20:27                 ` Scott Wood
                @ 2020-11-16 20:47                   ` Segher Boessenkool
                  0 siblings, 0 replies; 12+ 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] 12+ messages in thread

            • * Re: Error: invalid switch -me200
                2020-11-14  0:50               ` Segher Boessenkool
                2020-11-16 20:27                 ` Scott Wood
              @ 2020-11-16 23:59                 ` Alan Modra
                1 sibling, 0 replies; 12+ 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] 12+ 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
          @ 2020-11-14  9:02             ` Christophe Leroy
            2020-11-16  6:06               ` Michael Ellerman
            1 sibling, 1 reply; 12+ 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] 12+ messages in thread

        • • * Re: Error: invalid switch -me200
              2020-11-14  9:02             ` Christophe Leroy
            @ 2020-11-16  6:06               ` Michael Ellerman
              2020-11-16 15:26                 ` Christophe Leroy
              0 siblings, 1 reply; 12+ 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] 12+ messages in thread

          • • * Re: Error: invalid switch -me200
                2020-11-16  6:06               ` Michael Ellerman
              @ 2020-11-16 15:26                 ` Christophe Leroy
                0 siblings, 0 replies; 12+ 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] 12+ messages in thread

end of thread, other threads:[~2020-11-17  0:01 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <202011131146.g8dPLQDD-lkp@intel.com>
     [not found] ` <CAFP8O3LpSmxVnjHfQAN455k1ZRg3PbgZYhWr030evCq1T10k=Q@mail.gmail.com>
     [not found]   ` <20201113190824.GA1477315@ubuntu-m3-large-x86>
2020-11-13 19:42     ` Error: invalid switch -me200 Nick Desaulniers
2020-11-13 20:04       ` Nathan Chancellor
2020-11-13 20:14         ` Nick Desaulniers
2020-11-14  0:20           ` Segher Boessenkool
2020-11-14  0:37             ` Fāng-ruì Sòng
2020-11-14  0:50               ` Segher Boessenkool
2020-11-16 20:27                 ` Scott Wood
2020-11-16 20:47                   ` Segher Boessenkool
2020-11-16 23:59                 ` Alan Modra
2020-11-14  9:02             ` Christophe Leroy
2020-11-16  6:06               ` Michael Ellerman
2020-11-16 15:26                 ` Christophe Leroy

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox