[PATCH] ARM: multi_v7_defconfig: enable GCC plugin support and structleak plugin

[PATCH] ARM: multi_v7_defconfig: enable GCC plugin support and structleak plugin

[Ard Biesheuvel]

Enable the base GCC plugin support symbol CONFIG_GCC_PLUGINS for
multi_v7_defconfig so that builds hosted on systems that have a
supported version of GCC along with the prerequisite headers will
allow GCC plugins to be used during the build. At the moment, this
will also enable the ARM per-task stack protector plugin, since
all its remaining Kconfig dependencies are fulfilled by default.

At the same time, enable the structleak plugin. This plugin ensures
that local struct type variables are fully initialized, preventing
inadvertent leaks of kernel state if such variables are shared with
userland. This is a reasonable transformation to apply by default,
since it does not adversely affect performance, and does not affect
GCC compiler diagnostics regarding the use of uninitialized local
variables.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 arch/arm/configs/multi_v7_defconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 5bee34a7ff2e..c1e82c237ed0 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -1028,3 +1028,5 @@ CONFIG_CRYPTO_AES_ARM_CE=m
 CONFIG_CRYPTO_GHASH_ARM_CE=m
 CONFIG_CRYPTO_CRC32_ARM_CE=m
 CONFIG_CRYPTO_CHACHA20_NEON=m
+CONFIG_GCC_PLUGINS=y
+CONFIG_GCC_PLUGIN_STRUCTLEAK=y
-- 
2.20.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] ARM: multi_v7_defconfig: enable GCC plugin support and structleak plugin

[Arnd Bergmann]

On Tue, Jan 22, 2019 at 12:09 PM Ard Biesheuvel
<ard.biesheuvel@linaro.org> wrote:
>
> Enable the base GCC plugin support symbol CONFIG_GCC_PLUGINS for
> multi_v7_defconfig so that builds hosted on systems that have a
> supported version of GCC along with the prerequisite headers will
> allow GCC plugins to be used during the build. At the moment, this
> will also enable the ARM per-task stack protector plugin, since
> all its remaining Kconfig dependencies are fulfilled by default.
>
> At the same time, enable the structleak plugin. This plugin ensures
> that local struct type variables are fully initialized, preventing
> inadvertent leaks of kernel state if such variables are shared with
> userland. This is a reasonable transformation to apply by default,
> since it does not adversely affect performance, and does not affect
> GCC compiler diagnostics regarding the use of uninitialized local
> variables.
>
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
>  arch/arm/configs/multi_v7_defconfig | 2 ++
>  1 file changed, 2 insertions(+)

Applied to arm/defconfig, thanks!

      Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel