Re: [PATCH 2/3] modpost: Extended modversion support

Re: [PATCH 2/3] modpost: Extended modversion support

[kernel test robot]

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

Hi Matthew,

kernel test robot noticed the following build errors:

[auto build test ERROR on mcgrof/modules-next]
[also build test ERROR on powerpc/next powerpc/fixes masahiroy-kbuild/for-next masahiroy-kbuild/fixes linus/master v6.7-rc1 next-20231115]
[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#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Matthew-Maurer/export_report-Rehabilitate-script/20231116-030301
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git modules-next
patch link:    https://lore.kernel.org/r/20231115185858.2110875-3-mmaurer%40google.com
patch subject: [PATCH 2/3] modpost: Extended modversion support
:::::: branch date: 7 hours ago
:::::: commit date: 7 hours ago
config: powerpc-allmodconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (attached as reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311161008.T2J9BoqP-lkp@intel.com/

All error/warnings (new ones prefixed by >>):

   arch/powerpc/kernel/module_64.c: In function 'dedotify_ext_version_names':
>> arch/powerpc/kernel/module_64.c:373:9: error: implicit declaration of function 'bzero' [-Werror=implicit-function-declaration]
     373 |         bzero(&str_seq[out], size - out);
         |         ^~~~~
>> arch/powerpc/kernel/module_64.c:373:9: warning: incompatible implicit declaration of built-in function 'bzero' [-Wbuiltin-declaration-mismatch]
   cc1: some warnings being treated as errors


vim +/bzero +373 arch/powerpc/kernel/module_64.c

^1da177e4c3f41 arch/ppc64/kernel/module.c      Linus Torvalds 2005-04-16  357  
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  358  static void dedotify_ext_version_names(char *str_seq, unsigned long size)
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  359  {
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  360  	unsigned long out = 0;
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  361  	unsigned long in;
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  362  	char last = '\0';
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  363  
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  364  	for (in = 0; in < size; in++) {
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  365  		if (last == '\0')
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  366  			/* Skip all leading dots */
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  367  			if (str_seq[in] == '.')
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  368  				continue;
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  369  		last = str_seq[in];
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  370  		str_seq[out++] = last;
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  371  	}
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  372  	/* Zero the trailing portion of the names table for robustness */
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15 @373  	bzero(&str_seq[out], size - out);
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  374  }
e29ad08853ac5b arch/powerpc/kernel/module_64.c Matthew Maurer 2023-11-15  375  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

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

[-- Attachment #3: reproduce --]
[-- Type: text/plain, Size: 808 bytes --]

reproduce (this is a W=1 build):
        mkdir -p ~/bin
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add mcgrof https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git
        git fetch mcgrof modules-next
        git checkout mcgrof/modules-next
        b4 shazam https://lore.kernel.org/r/20231115185858.2110875-3-mmaurer@google.com
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-13.2.0 ~/bin/make.cross W=1 O=build_dir ARCH=powerpc olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-13.2.0 ~/bin/make.cross W=1 O=build_dir ARCH=powerpc SHELL=/bin/bash arch/powerpc/kernel/