linux-kernel - Re: [PATCH] scripts: remove obsolete gcc-ld script

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]

Date:   Mon, 29 Aug 2022 09:07:55 +0200
From:   Jiri Slaby <jirislaby@...nel.org>
To:     Nick Desaulniers <ndesaulniers@...gle.com>,
        Lukas Bulwahn <lukas.bulwahn@...il.com>
Cc:     Andi Kleen <ak@...ux.intel.com>,
        "H . Peter Anvin" <hpa@...ux.intel.com>,
        linux-kbuild@...r.kernel.org,
        Masahiro Yamada <masahiroy@...nel.org>,
        Michal Marek <michal.lkml@...kovi.net>,
        kernel-janitors@...r.kernel.org, linux-kernel@...r.kernel.org,
        Martin Liska <martin.liska@...e.com>
Subject: Re: [PATCH] scripts: remove obsolete gcc-ld script

On 25. 08. 22, 19:19, Nick Desaulniers wrote:
> + Jiri in case this needs to be carried downstream.

Thanks.

> On Thu, Aug 25, 2022 at 2:15 AM Lukas Bulwahn <lukas.bulwahn@...il.com> wrote:
>>
>> Since commit 8564ed2b3888 ("Kbuild, lto: Add a gcc-ld script to let run gcc
>> as ld") in 2014, there was not specific work on this the gcc-ld script
>> other than treewide clean-ups.
>>
>> There are no users within the kernel tree, and probably no out-of-tree
>> users either, and there is no dedicated maintainer in MAINTAINERS.

There are out-of-tree users.

>> Delete this obsolete gcc-ld script.
>>
>> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@...il.com>
> 
> No callers in-tree; happy to bring it back though should there later
> be. Thanks for the patch.

I agree to have this downstream-only for the time being. We have updates 
for it queued, so we'd only start tracking the full content now...

BTW the script is not nice at all. How do the clang people cope with the 
issue? (Running gcc-ld instead of ld with proper arguments when linking 
using (full) LTO. For example "-z now" -> "-Wl,-z,now".)

> Reviewed-by: Nick Desaulniers <ndesaulniers@...gle.com>
> 
>> ---
>> If there are no objections, I would like to get this patch included
>> through the kbuild tree.
>>
>> Masahiro-san, please pick this patch.
>>
>>   scripts/gcc-ld | 30 ------------------------------
>>   1 file changed, 30 deletions(-)
>>   delete mode 100755 scripts/gcc-ld
>>
>> diff --git a/scripts/gcc-ld b/scripts/gcc-ld
>> deleted file mode 100755
>> index 997b818c3962..000000000000
>> --- a/scripts/gcc-ld
>> +++ /dev/null
>> @@ -1,30 +0,0 @@
>> -#!/bin/sh
>> -# SPDX-License-Identifier: GPL-2.0
>> -# run gcc with ld options
>> -# used as a wrapper to execute link time optimizations
>> -# yes virginia, this is not pretty
>> -
>> -ARGS="-nostdlib"
>> -
>> -while [ "$1" != "" ] ; do
>> -       case "$1" in
>> -       -save-temps|-m32|-m64) N="$1" ;;
>> -       -r) N="$1" ;;
>> -       -[Wg]*) N="$1" ;;
>> -       -[olv]|-[Ofd]*|-nostdlib) N="$1" ;;
>> -       --end-group|--start-group)
>> -                N="-Wl,$1" ;;
>> -       -[RTFGhIezcbyYu]*|\
>> ---script|--defsym|-init|-Map|--oformat|-rpath|\
>> --rpath-link|--sort-section|--section-start|-Tbss|-Tdata|-Ttext|\
>> ---version-script|--dynamic-list|--version-exports-symbol|--wrap|-m)
>> -               A="$1" ; shift ; N="-Wl,$A,$1" ;;
>> -       -[m]*) N="$1" ;;
>> -       -*) N="-Wl,$1" ;;
>> -       *)  N="$1" ;;
>> -       esac
>> -       ARGS="$ARGS $N"
>> -       shift
>> -done
>> -
>> -exec $CC $ARGS


-- 
js
suse labs