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>
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:
> include/linux/module.h:133:6: warning: 'init_module'
specifies less restrictive attribute than its target 'init_test_ucd':
'cold' [-Wmissing-attributes]
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:
> include/linux/module.h:139:7: warning: 'cleanup_module'
specifies less restrictive attribute than its target 'exit_test_ucd':
'cold' [-Wmissing-attributes]
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>
:::::: CC: Paul Gortmaker <paul.gortmaker(a)windriver.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org