linux-hardening - Re: [PATCH] Documentation/Kbuild: Remove references to gcc-plugin.sh

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:   Sat, 23 Jan 2021 22:17:59 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Robert Karszniewicz <r.karszniewicz@...tec.de>
Cc:     Kees Cook <keescook@...omium.org>,
        Michal Marek <michal.lkml@...kovi.net>,
        Jonathan Corbet <corbet@....net>,
        Philipp Rudo <prudo@...ux.ibm.com>,
        linux-hardening@...r.kernel.org,
        Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        "open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
        Robert Karszniewicz <avoidr@...teo.de>
Subject: Re: [PATCH] Documentation/Kbuild: Remove references to gcc-plugin.sh

On Sat, Jan 23, 2021 at 3:19 AM Robert Karszniewicz
<r.karszniewicz@...tec.de> wrote:
>
> gcc-plugin.sh has been removed in commit
> 1e860048c53e ("gcc-plugins: simplify GCC plugin-dev capability test").
>
> Signed-off-by: Robert Karszniewicz <r.karszniewicz@...tec.de>
> ---
>  Documentation/kbuild/gcc-plugins.rst |  6 ------
>  scripts/dummy-tools/gcc              | 15 ---------------
>  2 files changed, 21 deletions(-)
>
> diff --git a/Documentation/kbuild/gcc-plugins.rst b/Documentation/kbuild/gcc-plugins.rst
> index 4b1c10f88e30..63379d0150e3 100644
> --- a/Documentation/kbuild/gcc-plugins.rst
> +++ b/Documentation/kbuild/gcc-plugins.rst
> @@ -47,12 +47,6 @@ Files
>         This is a compatibility header for GCC plugins.
>         It should be always included instead of individual gcc headers.
>
> -**$(src)/scripts/gcc-plugin.sh**
> -
> -       This script checks the availability of the included headers in
> -       gcc-common.h and chooses the proper host compiler to build the plugins
> -       (gcc-4.7 can be built by either gcc or g++).
> -
>  **$(src)/scripts/gcc-plugins/gcc-generate-gimple-pass.h,
>  $(src)/scripts/gcc-plugins/gcc-generate-ipa-pass.h,
>  $(src)/scripts/gcc-plugins/gcc-generate-simple_ipa-pass.h,
> diff --git a/scripts/dummy-tools/gcc b/scripts/dummy-tools/gcc
> index 33487e99d83e..a484de576e6f 100755
> --- a/scripts/dummy-tools/gcc
> +++ b/scripts/dummy-tools/gcc
> @@ -74,18 +74,3 @@ if arg_contain -S "$@"; then
>                 exit 0
>         fi
>  fi
> -
> -# For scripts/gcc-plugin.sh
> -if arg_contain -print-file-name=plugin "$@"; then
> -       plugin_dir=$(mktemp -d)
> -
> -       sed -n 's/.*#include "\(.*\)"/\1/p' $(dirname $0)/../gcc-plugins/gcc-common.h |
> -       while read header
> -       do
> -               mkdir -p $plugin_dir/include/$(dirname $header)
> -               touch $plugin_dir/include/$header
> -       done
> -
> -       echo $plugin_dir
> -       exit 0
> -fi


Well, it is wrong to remove this hunk entirely.
It would break the ability to enable GCC_PLUGINS.

I will simplify this part.
https://lore.kernel.org/patchwork/patch/1369968/


I took the only change to Documentation/kbuild/gcc-plugins.rst
from your patch but this document contains more stale info.

https://lore.kernel.org/patchwork/patch/1370008/


-- 
Best Regards
Masahiro Yamada