linux-kernel - Re: [PATCH] powerpc: Drop -me200 addition to build flags

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8b23cec8b7b490ec8ef578cbd2fe4316288d4e13.camel@buserror.net>
Date:   Mon, 16 Nov 2020 14:26:40 -0600
From:   Scott Wood <oss@...error.net>
To:     Michael Ellerman <mpe@...erman.id.au>, linuxppc-dev@...abs.org
Cc:     christophe.leroy@...roup.eu, segher@...nel.crashing.org,
        linux-kernel@...r.kernel.org, ndesaulniers@...gle.com,
        natechancellor@...il.com
Subject: Re: [PATCH] powerpc: Drop -me200 addition to build flags

On Mon, 2020-11-16 at 23:09 +1100, Michael Ellerman wrote:
> Currently a build with CONFIG_E200=y will fail with:
> 
>   Error: invalid switch -me200
>   Error: unrecognized option -me200
> 
> Upstream binutils has never supported an -me200 option. Presumably it
> was supported at some point by either a fork or Freescale internal
> binutils.
> 
> We can't support code that we can't even build test, so drop the
> addition of -me200 to the build flags, so we can at least build with
> CONFIG_E200=y.
> 
> Reported-by: Németh Márton <nm127@...email.hu>
> Reported-by: kernel test robot <lkp@...el.com>
> Signed-off-by: Michael Ellerman <mpe@...erman.id.au>
> ---
> 
> More discussion: 
> https://lore.kernel.org/lkml/202011131146.g8dPLQDD-lkp@intel.com
> ---
>  arch/powerpc/Makefile | 1 -
>  1 file changed, 1 deletion(-)

Acked-by: Scott Wood <oss@...error.net>

I'd go further and remove E200 code entirely, unless someone with the hardware
can claim that it actually works.  There doesn't appear to be any actual
platform support for an e200-based system.  It seems to be a long-abandoned
work in progress.

-Scott