Re: [RFC PATCH v2 2/3] fs: define a firmware security filesystem named fwsecurityfs

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

Hi Nayna,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on powerpc/next]
[also build test ERROR on linus/master v5.19-rc3 next-20220623]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/intel-lab-lkp/linux/commits/Nayna-Jain/powerpc-pseries-add-support-for-local-secure-storage-called-Platform-KeyStore-PKS/20220623-060048
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220624/202206240601.5oMCfW72-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 46be5faaf03466c3751f8a2882bef5a217e15926)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/02d7d8ac9272e2e56bac334a16100784450e8902
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Nayna-Jain/powerpc-pseries-add-support-for-local-secure-storage-called-Platform-KeyStore-PKS/20220623-060048
        git checkout 02d7d8ac9272e2e56bac334a16100784450e8902
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

>> fs/fwsecurityfs/super.c:152:20: error: expected parameter declarator
   MODULE_DESCRIPTION("Firmware Security Filesystem");
                      ^
>> fs/fwsecurityfs/super.c:152:20: error: expected ')'
   fs/fwsecurityfs/super.c:152:19: note: to match this '('
   MODULE_DESCRIPTION("Firmware Security Filesystem");
                     ^
>> fs/fwsecurityfs/super.c:152:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
   MODULE_DESCRIPTION("Firmware Security Filesystem");
   ^
   int
>> fs/fwsecurityfs/super.c:152:19: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
   MODULE_DESCRIPTION("Firmware Security Filesystem");
                     ^
                                                    void
   fs/fwsecurityfs/super.c:153:15: error: expected parameter declarator
   MODULE_AUTHOR("Nayna Jain");
                 ^
   fs/fwsecurityfs/super.c:153:15: error: expected ')'
   fs/fwsecurityfs/super.c:153:14: note: to match this '('
   MODULE_AUTHOR("Nayna Jain");
                ^
   fs/fwsecurityfs/super.c:153:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
   MODULE_AUTHOR("Nayna Jain");
   ^
   int
   fs/fwsecurityfs/super.c:153:14: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
   MODULE_AUTHOR("Nayna Jain");
                ^
                             void
   fs/fwsecurityfs/super.c:154:16: error: expected parameter declarator
   MODULE_LICENSE("GPL");
                  ^
   fs/fwsecurityfs/super.c:154:16: error: expected ')'
   fs/fwsecurityfs/super.c:154:15: note: to match this '('
   MODULE_LICENSE("GPL");
                 ^
   fs/fwsecurityfs/super.c:154:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
   MODULE_LICENSE("GPL");
   ^
   int
   fs/fwsecurityfs/super.c:154:15: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
   MODULE_LICENSE("GPL");
                 ^
                       void
   12 errors generated.


vim +152 fs/fwsecurityfs/super.c

   134	
   135	static int __init fwsecurityfs_init(void)
   136	{
   137		int retval;
   138	
   139		retval = sysfs_create_mount_point(firmware_kobj, "security");
   140		if (retval)
   141			return retval;
   142	
   143		retval = register_filesystem(&fs_type);
   144		if (retval) {
   145			sysfs_remove_mount_point(firmware_kobj, "security");
   146			return retval;
   147		}
   148	
   149		return 0;
   150	}
   151	core_initcall(fwsecurityfs_init);
 > 152	MODULE_DESCRIPTION("Firmware Security Filesystem");

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp