[kbuild-all] [android-common:upstream-f2fs-stable-linux-4.19.y 303/856] include/linux/module.h:133:6: warning: 'init_module' specifies less restrictive attribute than its target 'init_test_ucd': 'cold'

Hi Gabriel, FYI, the error/warning still remains. tree: https://android.googlesource.com/kernel/common upstream-f2fs-stable-linux-4.19.y head: ab44caf36dac040440ee3e8f0e4dc58a7d2717fc commit: 56427c9cf549342906953cfd1d180d014b78c97b [303/856] unicode: introduce test module for normalized utf8 implementation config: x86_64-allmodconfig (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): git remote add android-common https://android.googlesource.com/kernel/common git fetch --no-tags android-common upstream-f2fs-stable-linux-4.19.y git checkout 56427c9cf549342906953cfd1d180d014b78c97b # save the attached .config to linux build tree make W=1 W=1 ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): fs/unicode/utf8-selftest.c:46:1: warning: 'static' is not at beginning of declaration [-Wold-style-declaration] 46 | const static struct { | ^~~~~ fs/unicode/utf8-selftest.c:100:1: warning: 'static' is not at beginning of declaration [-Wold-style-declaration] 100 | const static struct { | ^~~~~ In file included from fs/unicode/utf8-selftest.c:18: 133 | int init_module(void) __attribute__((alias(#initfn))); | ^~~~~~~~~~~ fs/unicode/utf8-selftest.c:316:1: note: in expansion of macro 'module_init' 316 | module_init(init_test_ucd); | ^~~~~~~~~~~ fs/unicode/utf8-selftest.c:294:19: note: 'init_module' target declared here 294 | static int __init init_test_ucd(void) | ^~~~~~~~~~~~~ In file included from fs/unicode/utf8-selftest.c:18: 139 | void cleanup_module(void) __attribute__((alias(#exitfn))); | ^~~~~~~~~~~~~~ fs/unicode/utf8-selftest.c:317:1: note: in expansion of macro 'module_exit' 317 | module_exit(exit_test_ucd); | ^~~~~~~~~~~ fs/unicode/utf8-selftest.c:312:20: note: 'cleanup_module' target declared here 312 | static void __exit exit_test_ucd(void) | ^~~~~~~~~~~~~ vim +133 include/linux/module.h 0fd972a7d91d6e Paul Gortmaker 2015-05-01 128 0fd972a7d91d6e Paul Gortmaker 2015-05-01 129 /* Each module must use one module_init(). */ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 130 #define module_init(initfn) \ 1f318a8bafcfba Arnd Bergmann 2017-02-01 131 static inline initcall_t __maybe_unused __inittest(void) \ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 132 { return initfn; } \ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 @133 int init_module(void) __attribute__((alias(#initfn))); 0fd972a7d91d6e Paul Gortmaker 2015-05-01 134 0fd972a7d91d6e Paul Gortmaker 2015-05-01 135 /* This is only required if you want to be unloadable. */ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 136 #define module_exit(exitfn) \ 1f318a8bafcfba Arnd Bergmann 2017-02-01 137 static inline exitcall_t __maybe_unused __exittest(void) \ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 138 { return exitfn; } \ 0fd972a7d91d6e Paul Gortmaker 2015-05-01 @139 void cleanup_module(void) __attribute__((alias(#exitfn))); 0fd972a7d91d6e Paul Gortmaker 2015-05-01 140 :::::: The code at line 133 was first introduced by commit :::::: 0fd972a7d91d6e15393c449492a04d94c0b89351 module: relocate module_init from init.h to module.h :::::: TO: Paul Gortmaker <paul.gortmaker(a)windriver.com&gt; :::::: CC: Paul Gortmaker <paul.gortmaker(a)windriver.com&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org