2018-02-17 6:41 GMT+09:00 Arnd Bergmann <arnd@xxxxxxxx>: > This can easily double the time for compiling a driver but does not > provide any benefit for the compile tester, so it's better left disabled. > > In addition, any 'inline' function that is not also 'static' and that > contains an 'if' causes a warning like > > include/linux/string.h:212:2: note: in expansion of macro 'if' > if (strscpy(p, q, p_size < q_size ? p_size : q_size) < 0) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'strcpy' which is not static > > without this patch, and I could not come up with a nice fix for that. > In combination with my patch to always enable 'CONFIG_COMPILE_TEST' > during 'randconfig' builds, we can at least hide these warnings for > most users. > > Cc: Steven Rostedt <rostedt@xxxxxxxxxxx> > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> I took a look at this just in case because the subject is prefixed with "Kconfig:", but it is actually "trace:". I expect "Kconfig:" for core changes of Kconfig. Using precise patch prefix would get more attention from right people. > --- > kernel/trace/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig > index 434c840e2d82..faaf687b13b1 100644 > --- a/kernel/trace/Kconfig > +++ b/kernel/trace/Kconfig > @@ -345,6 +345,7 @@ config PROFILE_ANNOTATED_BRANCHES > config PROFILE_ALL_BRANCHES > bool "Profile all if conditionals" > select TRACE_BRANCH_PROFILING > + depends on !COMPILE_TEST > help > This tracer profiles all branch conditions. Every if () > taken in the kernel is recorded whether it hit or miss. > -- > 2.9.0 > > -- > 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 -- Best Regards Masahiro Yamada -- 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
