[linux-next:master 8923/10701] drivers/md/dm-zoned-metadata.c:224:14: warning: no previous prototype for function 'dmz_dev_zone_id'

[linux-next:master 8923/10701] drivers/md/dm-zoned-metadata.c:224:14: warning: no previous prototype for function 'dmz_dev_zone_id'

[kbuild test robot]

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   fb57b1fabcb28f358901b2df90abd2b48abc1ca8
commit: 58f6902a2eba7313d37d30bce18a331460f19699 [8923/10701] dm zoned: metadata version 2
config: arm64-randconfig-r005-20200520 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project e6658079aca6d971b4e9d7137a3a2ecbc9c34aec)
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        git checkout 58f6902a2eba7313d37d30bce18a331460f19699
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 

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

All warnings (new ones prefixed by >>, old ones prefixed by <<):

>> drivers/md/dm-zoned-metadata.c:224:14: warning: no previous prototype for function 'dmz_dev_zone_id' [-Wmissing-prototypes]
unsigned int dmz_dev_zone_id(struct dmz_metadata *zmd, struct dm_zone *zone)
^
drivers/md/dm-zoned-metadata.c:224:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
unsigned int dmz_dev_zone_id(struct dmz_metadata *zmd, struct dm_zone *zone)
^
static
drivers/md/dm-zoned-metadata.c:2656:6: warning: no previous prototype for function 'dmz_print_dev' [-Wmissing-prototypes]
void dmz_print_dev(struct dmz_metadata *zmd, int num)
^
drivers/md/dm-zoned-metadata.c:2656:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void dmz_print_dev(struct dmz_metadata *zmd, int num)
^
static
2 warnings generated.

vim +/dmz_dev_zone_id +224 drivers/md/dm-zoned-metadata.c

   209	
   210	#define dmz_zmd_info(zmd, format, args...)	\
   211		DMINFO("(%s): " format, (zmd)->label, ## args)
   212	
   213	#define dmz_zmd_err(zmd, format, args...)	\
   214		DMERR("(%s): " format, (zmd)->label, ## args)
   215	
   216	#define dmz_zmd_warn(zmd, format, args...)	\
   217		DMWARN("(%s): " format, (zmd)->label, ## args)
   218	
   219	#define dmz_zmd_debug(zmd, format, args...)	\
   220		DMDEBUG("(%s): " format, (zmd)->label, ## args)
   221	/*
   222	 * Various accessors
   223	 */
 > 224	unsigned int dmz_dev_zone_id(struct dmz_metadata *zmd, struct dm_zone *zone)
   225	{
   226		unsigned int zone_id;
   227	
   228		if (WARN_ON(!zone))
   229			return 0;
   230	
   231		zone_id = zone->id;
   232		if (zmd->nr_devs > 1 &&
   233		    (zone_id >= zmd->dev[1].zone_offset))
   234			zone_id -= zmd->dev[1].zone_offset;
   235		return zone_id;
   236	}
   237	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

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

Re: [linux-next:master 8923/10701] drivers/md/dm-zoned-metadata.c:224:14: warning: no previous prototype for function 'dmz_dev_zone_id'

[Mike Snitzer]

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

On Wed, May 20 2020 at  5:00pm -0400,
kbuild test robot <lkp@intel.com> wrote:

> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   fb57b1fabcb28f358901b2df90abd2b48abc1ca8
> commit: 58f6902a2eba7313d37d30bce18a331460f19699 [8923/10701] dm zoned: metadata version 2
> config: arm64-randconfig-r005-20200520 (attached as .config)
> compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project e6658079aca6d971b4e9d7137a3a2ecbc9c34aec)
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # install arm64 cross compiling tool for clang build
>         # apt-get install binutils-aarch64-linux-gnu
>         git checkout 58f6902a2eba7313d37d30bce18a331460f19699
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>, old ones prefixed by <<):
> 
> >> drivers/md/dm-zoned-metadata.c:224:14: warning: no previous prototype for function 'dmz_dev_zone_id' [-Wmissing-prototypes]
> unsigned int dmz_dev_zone_id(struct dmz_metadata *zmd, struct dm_zone *zone)
> ^
> drivers/md/dm-zoned-metadata.c:224:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
> unsigned int dmz_dev_zone_id(struct dmz_metadata *zmd, struct dm_zone *zone)
> ^

Fixed by making dmz_dev_zone_id static.

Thanks,
Mike