On Mon, Mar 26, 2018 at 10:29 PM, Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote: > The plugin availability is checked in Kconfig, so all{yes,mod}config > will not be bothered. Remove 'depends on !COMPILE_TEST'. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> > --- > > Changes in v2: > - Remove more 'depends on' > > arch/Kconfig | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/arch/Kconfig b/arch/Kconfig > index 88cc925..b07094c 100644 > --- a/arch/Kconfig > +++ b/arch/Kconfig > @@ -414,7 +414,6 @@ menuconfig GCC_PLUGINS > bool "GCC plugins" > depends on HAVE_GCC_PLUGINS > depends on $(success $srctree/scripts/gcc-plugin.sh $HOSTCXX $CC) > - depends on !COMPILE_TEST > help > GCC plugins are loadable modules that provide extra features to the > compiler. They are useful for runtime instrumentation and static analysis. This one is nice, yes! We get coverage for anyone with plugin support. > @@ -424,7 +423,6 @@ menuconfig GCC_PLUGINS > config GCC_PLUGIN_CYC_COMPLEXITY > bool "Compute the cyclomatic complexity of a function" if EXPERT > depends on GCC_PLUGINS > - depends on !COMPILE_TEST > help > The complexity M of a function's control flow graph is defined as: > M = E - N + 2P > @@ -487,7 +485,6 @@ config GCC_PLUGIN_STRUCTLEAK_BYREF_ALL > config GCC_PLUGIN_STRUCTLEAK_VERBOSE > bool "Report forcefully initialized variables" > depends on GCC_PLUGIN_STRUCTLEAK > - depends on !COMPILE_TEST > help > This option will cause a warning to be printed each time the > structleak plugin finds a variable it thinks needs to be > @@ -527,7 +524,6 @@ config GCC_PLUGIN_RANDSTRUCT > config GCC_PLUGIN_RANDSTRUCT_PERFORMANCE > bool "Use cacheline-aware structure randomization" > depends on GCC_PLUGIN_RANDSTRUCT > - depends on !COMPILE_TEST > help > If you say Y here, the RANDSTRUCT randomization will make a > best effort at restricting randomization to cacheline-sized All three of these, though, please leave as they were. They're either needlessly noisy for COMPILE_TEST (GCC_PLUGIN_CYC_COMPLEXITY and *_VERBOSE), or reduce the COMPILE_TEST coverage (GCC_PLUGIN_RANDSTRUCT_PERFORMANCE). (And perhaps a comment is needed to explain these to avoid confusion in the future?) -Kees -- Kees Cook Pixel Security -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html