[f2fs-stable:linux-4.19.y 327/400] include/linux/module.h:133:6: warning: 'init_module' specifies less restrictive attribute than its target 'pkcs7_key_init': 'cold'

[kbuild test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 5143 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs-stable.git linux-4.19.y
head:   0b1c42a0110f928fde6a6d13ccdde2afce38e737
commit: 0c22f68fc44d8d7bf72b005f8ed02b1b2dc7c76a [327/400] fs-verity: support builtin file signatures
config: openrisc-randconfig-a001-20191211 (attached as .config)
compiler: or1k-linux-gcc (GCC) 9.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 0c22f68fc44d8d7bf72b005f8ed02b1b2dc7c76a
        # save the attached .config to linux build tree
        GCC_VERSION=9.2.0 make.cross ARCH=openrisc 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from crypto/asymmetric_keys/pkcs7_key_type.c:15:
>> include/linux/module.h:133:6: warning: 'init_module' specifies less restrictive attribute than its target 'pkcs7_key_init': 'cold' [-Wmissing-attributes]
     133 |  int init_module(void) __attribute__((alias(#initfn)));
         |      ^~~~~~~~~~~
>> crypto/asymmetric_keys/pkcs7_key_type.c:98:1: note: in expansion of macro 'module_init'
      98 | module_init(pkcs7_key_init);
         | ^~~~~~~~~~~
   crypto/asymmetric_keys/pkcs7_key_type.c:88:19: note: 'init_module' target declared here
      88 | static int __init pkcs7_key_init(void)
         |                   ^~~~~~~~~~~~~~
   In file included from crypto/asymmetric_keys/pkcs7_key_type.c:15:
>> include/linux/module.h:139:7: warning: 'cleanup_module' specifies less restrictive attribute than its target 'pkcs7_key_cleanup': 'cold' [-Wmissing-attributes]
     139 |  void cleanup_module(void) __attribute__((alias(#exitfn)));
         |       ^~~~~~~~~~~~~~
>> crypto/asymmetric_keys/pkcs7_key_type.c:99:1: note: in expansion of macro 'module_exit'
      99 | module_exit(pkcs7_key_cleanup);
         | ^~~~~~~~~~~
   crypto/asymmetric_keys/pkcs7_key_type.c:93:20: note: 'cleanup_module' target declared here
      93 | static void __exit pkcs7_key_cleanup(void)
         |                    ^~~~~~~~~~~~~~~~~
--
   In file included from crypto//asymmetric_keys/pkcs7_key_type.c:15:
>> include/linux/module.h:133:6: warning: 'init_module' specifies less restrictive attribute than its target 'pkcs7_key_init': 'cold' [-Wmissing-attributes]
     133 |  int init_module(void) __attribute__((alias(#initfn)));
         |      ^~~~~~~~~~~
   crypto//asymmetric_keys/pkcs7_key_type.c:98:1: note: in expansion of macro 'module_init'
      98 | module_init(pkcs7_key_init);
         | ^~~~~~~~~~~
   crypto//asymmetric_keys/pkcs7_key_type.c:88:19: note: 'init_module' target declared here
      88 | static int __init pkcs7_key_init(void)
         |                   ^~~~~~~~~~~~~~
   In file included from crypto//asymmetric_keys/pkcs7_key_type.c:15:
>> include/linux/module.h:139:7: warning: 'cleanup_module' specifies less restrictive attribute than its target 'pkcs7_key_cleanup': 'cold' [-Wmissing-attributes]
     139 |  void cleanup_module(void) __attribute__((alias(#exitfn)));
         |       ^~~~~~~~~~~~~~
   crypto//asymmetric_keys/pkcs7_key_type.c:99:1: note: in expansion of macro 'module_exit'
      99 | module_exit(pkcs7_key_cleanup);
         | ^~~~~~~~~~~
   crypto//asymmetric_keys/pkcs7_key_type.c:93:20: note: 'cleanup_module' target declared here
      93 | static void __exit pkcs7_key_cleanup(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@windriver.com>
:::::: CC: Paul Gortmaker <paul.gortmaker@windriver.com>

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 25722 bytes --]