Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1571688 > unrolled thread

[PATCH] gcc-plugins: make GCC_PLUGIN_INITIFY_VERBOSE depend on !COMPILE_TEST

Started byArnd Bergmann <arnd@arndb.de>
First post2017-02-01 17:10 +0100
Last post2017-02-01 17:30 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] gcc-plugins: make GCC_PLUGIN_INITIFY_VERBOSE depend on !COMPILE_TEST Arnd Bergmann <arnd@arndb.de> - 2017-02-01 17:10 +0100
    Re: [PATCH] gcc-plugins: make GCC_PLUGIN_INITIFY_VERBOSE depend on !COMPILE_TEST Kees Cook <keescook@chromium.org> - 2017-02-01 17:30 +0100

#1571688 — [PATCH] gcc-plugins: make GCC_PLUGIN_INITIFY_VERBOSE depend on !COMPILE_TEST

FromArnd Bergmann <arnd@arndb.de>
Date2017-02-01 17:10 +0100
Subject[PATCH] gcc-plugins: make GCC_PLUGIN_INITIFY_VERBOSE depend on !COMPILE_TEST
Message-ID<t65DQ-86Y-23@gated-at.bofh.it>
Just like GCC_PLUGIN_STRUCTLEAK_VERBOSE, we should make this one depend on !COMPILE_TEST,
otherwise we get lots of unnecessary output for randconfig and possibly allmodconfig
builds.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/Kconfig b/arch/Kconfig
index 48f395ed5207..b5eafefbe2bd 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -453,6 +453,7 @@ config GCC_PLUGIN_INITIFY
 config GCC_PLUGIN_INITIFY_VERBOSE
 	bool "Report initification"
 	depends on GCC_PLUGIN_INITIFY
+	depends on !COMPILE_TEST
 	help
 	  Print all initified strings and all functions which should be
 	  __init/__exit.
-- 
2.9.0

[toc] | [next] | [standalone]


#1571710

FromKees Cook <keescook@chromium.org>
Date2017-02-01 17:30 +0100
Message-ID<t65Xd-8e7-29@gated-at.bofh.it>
In reply to#1571688
On Wed, Feb 1, 2017 at 8:07 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> Just like GCC_PLUGIN_STRUCTLEAK_VERBOSE, we should make this one depend on !COMPILE_TEST,
> otherwise we get lots of unnecessary output for randconfig and possibly allmodconfig
> builds.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Thanks! And sorry, I thought I already had that. :P I'll add this to my tree.

-Kees

> ---
>  arch/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/Kconfig b/arch/Kconfig
> index 48f395ed5207..b5eafefbe2bd 100644
> --- a/arch/Kconfig
> +++ b/arch/Kconfig
> @@ -453,6 +453,7 @@ config GCC_PLUGIN_INITIFY
>  config GCC_PLUGIN_INITIFY_VERBOSE
>         bool "Report initification"
>         depends on GCC_PLUGIN_INITIFY
> +       depends on !COMPILE_TEST
>         help
>           Print all initified strings and all functions which should be
>           __init/__exit.
> --
> 2.9.0
>



-- 
Kees Cook
Pixel Security

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web