linux-kernel - Re: [PATCH 4.14] kbuild: allow to use GCC toolchain not in Clang search path

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] [day] [month] [year] [list]

Date:   Thu, 29 Nov 2018 11:27:19 +0100
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Nathan Chancellor <natechancellor@...il.com>
Cc:     Sasha Levin <sashal@...nel.org>, stable@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        Stefan Agner <stefan@...er.ch>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>
Subject: Re: [PATCH 4.14] kbuild: allow to use GCC toolchain not in Clang
 search path

On Wed, Nov 28, 2018 at 10:35:12PM -0700, Nathan Chancellor wrote:
> From: Stefan Agner <stefan@...er.ch>
> 
> commit ef8c4ed9db80261f397f0c0bf723684601ae3b52 upstream.
> 
> When using a GCC cross toolchain which is not in a compiled in
> Clang search path, Clang reverts to the system assembler and
> linker. This leads to assembler or linker errors, depending on
> which tool is first used for a given architecture.
> 
> It seems that Clang is not searching $PATH for a matching
> assembler or linker.
> 
> Make sure that Clang picks up the correct assembler or linker by
> passing the cross compilers bin directory as search path.
> 
> This allows to use Clang provided by distributions with GCC
> toolchains not in /usr/bin.
> 
> Link: https://github.com/ClangBuiltLinux/linux/issues/78
> Signed-off-by: Stefan Agner <stefan@...er.ch>
> Reviewed-and-tested-by: Nick Desaulniers <ndesaulniers@...gle.com>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
> [nc: Adjust context]
> Signed-off-by: Nathan Chancellor <natechancellor@...il.com>
> ---
>  Makefile | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)

We also need this for 4.9, right?

thanks,

greg k-h