All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Goldwyn Rodrigues <rgoldwyn@suse.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
	linux-kernel@vger.kernel.org
Subject: [goldwynr:iomap 33/33] fs/btrfs/volumes.c:6893:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int')
Date: Thu, 30 Dec 2021 13:21:48 +0800	[thread overview]
Message-ID: <202112301331.RT7JMows-lkp@intel.com> (raw)

tree:   https://github.com/goldwynr/linux iomap
head:   30c74a8c201365178cae26d0d7aefa120c3245ab
commit: 30c74a8c201365178cae26d0d7aefa120c3245ab [33/33] btrfs: debug patch
config: hexagon-randconfig-r013-20211230 (https://download.01.org/0day-ci/archive/20211230/202112301331.RT7JMows-lkp@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project cd284b7ac0615afc6e0f1a30da2777e361de27a3)
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/goldwynr/linux/commit/30c74a8c201365178cae26d0d7aefa120c3245ab
        git remote add goldwynr https://github.com/goldwynr/linux
        git fetch --no-tags goldwynr iomap
        git checkout 30c74a8c201365178cae26d0d7aefa120c3245ab
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash fs/btrfs/

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

All warnings (new ones prefixed by >>):

>> fs/btrfs/volumes.c:6893:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                   ioend->io_offset, ioend->io_size);
                                                     ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6904:24: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                           ioend->io_offset, ioend->io_size);
                                                             ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6910:24: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                           ioend->io_offset, ioend->io_size);
                                                             ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6941:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                   ioend->io_offset, ioend->io_size);
                                                     ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   4 warnings generated.


vim +6893 fs/btrfs/volumes.c

  6878	
  6879	blk_status_t btrfs_map_bio(struct btrfs_fs_info *fs_info, struct bio *bio,
  6880				   int mirror_num)
  6881	{
  6882		const u64 orig_logical = bio->bi_iter.bi_sector << SECTOR_SHIFT;
  6883		const unsigned int orig_length = bio->bi_iter.bi_size;
  6884		const enum btrfs_map_op op = btrfs_op(bio);
  6885		u64 cur_logical = orig_logical;
  6886		int ret;
  6887		struct iomap_ioend *ioend = bio->bi_private;
  6888	
  6889		if (ioend)
  6890			pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6891					__func__, __LINE__,
  6892					(unsigned long)ioend, ioend->io_inode->i_ino,
> 6893					ioend->io_offset, ioend->io_size);
  6894	
  6895		while (cur_logical < orig_logical + orig_length) {
  6896			u64 map_length = orig_logical + orig_length - cur_logical;
  6897			struct btrfs_io_context *bioc = NULL;
  6898			struct bio *cur_bio;
  6899	
  6900			if (ioend)
  6901				pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6902						__func__, __LINE__,
  6903						(unsigned long)ioend, ioend->io_inode->i_ino,
  6904						ioend->io_offset, ioend->io_size);
  6905			btrfs_bio_save_iter(btrfs_bio(bio));
  6906			if (ioend)
  6907				pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6908						__func__, __LINE__,
  6909						(unsigned long)ioend, ioend->io_inode->i_ino,
  6910						ioend->io_offset, ioend->io_size);
  6911			ret = __btrfs_map_block(fs_info, op, cur_logical, &map_length,
  6912						&bioc, mirror_num, 1);
  6913			if (ret)
  6914				return errno_to_blk_status(ret);
  6915	
  6916			if (cur_logical + map_length < orig_logical + orig_length) {
  6917				/*
  6918				 * For now zoned write should never cross stripe
  6919				 * boundary
  6920				 */
  6921				ASSERT(bio_op(bio) != REQ_OP_ZONE_APPEND);
  6922	
  6923				/* Split the bio */
  6924				cur_bio = btrfs_bio_split(fs_info, bio, map_length);
  6925			} else {
  6926				/* Use the existing bio directly */
  6927				cur_bio = bio;
  6928			}
  6929			btrfs_bio_counter_inc_blocked(fs_info);
  6930			ret = submit_one_mapped_range(fs_info, cur_bio, bioc,
  6931						      map_length, mirror_num);
  6932			btrfs_bio_counter_dec(fs_info);
  6933			if (ret < 0)
  6934				return errno_to_blk_status(ret);
  6935			cur_logical += map_length;
  6936		}
  6937		if (ioend)
  6938			pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6939					__func__, __LINE__,
  6940					(unsigned long)ioend, ioend->io_inode->i_ino,
  6941					ioend->io_offset, ioend->io_size);
  6942		return BLK_STS_OK;
  6943	}
  6944	

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

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [goldwynr:iomap 33/33] fs/btrfs/volumes.c:6893:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int')
Date: Thu, 30 Dec 2021 13:21:48 +0800	[thread overview]
Message-ID: <202112301331.RT7JMows-lkp@intel.com> (raw)

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

tree:   https://github.com/goldwynr/linux iomap
head:   30c74a8c201365178cae26d0d7aefa120c3245ab
commit: 30c74a8c201365178cae26d0d7aefa120c3245ab [33/33] btrfs: debug patch
config: hexagon-randconfig-r013-20211230 (https://download.01.org/0day-ci/archive/20211230/202112301331.RT7JMows-lkp(a)intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project cd284b7ac0615afc6e0f1a30da2777e361de27a3)
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/goldwynr/linux/commit/30c74a8c201365178cae26d0d7aefa120c3245ab
        git remote add goldwynr https://github.com/goldwynr/linux
        git fetch --no-tags goldwynr iomap
        git checkout 30c74a8c201365178cae26d0d7aefa120c3245ab
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash fs/btrfs/

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

All warnings (new ones prefixed by >>):

>> fs/btrfs/volumes.c:6893:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                   ioend->io_offset, ioend->io_size);
                                                     ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6904:24: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                           ioend->io_offset, ioend->io_size);
                                                             ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6910:24: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                           ioend->io_offset, ioend->io_size);
                                                             ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/btrfs/volumes.c:6941:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                   ioend->io_offset, ioend->io_size);
                                                     ^~~~~~~~~~~~~~
   include/linux/printk.h:523:34: note: expanded from macro 'pr_info'
           printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
                                   ~~~     ^~~~~~~~~~~
   include/linux/printk.h:450:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:422:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   4 warnings generated.


vim +6893 fs/btrfs/volumes.c

  6878	
  6879	blk_status_t btrfs_map_bio(struct btrfs_fs_info *fs_info, struct bio *bio,
  6880				   int mirror_num)
  6881	{
  6882		const u64 orig_logical = bio->bi_iter.bi_sector << SECTOR_SHIFT;
  6883		const unsigned int orig_length = bio->bi_iter.bi_size;
  6884		const enum btrfs_map_op op = btrfs_op(bio);
  6885		u64 cur_logical = orig_logical;
  6886		int ret;
  6887		struct iomap_ioend *ioend = bio->bi_private;
  6888	
  6889		if (ioend)
  6890			pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6891					__func__, __LINE__,
  6892					(unsigned long)ioend, ioend->io_inode->i_ino,
> 6893					ioend->io_offset, ioend->io_size);
  6894	
  6895		while (cur_logical < orig_logical + orig_length) {
  6896			u64 map_length = orig_logical + orig_length - cur_logical;
  6897			struct btrfs_io_context *bioc = NULL;
  6898			struct bio *cur_bio;
  6899	
  6900			if (ioend)
  6901				pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6902						__func__, __LINE__,
  6903						(unsigned long)ioend, ioend->io_inode->i_ino,
  6904						ioend->io_offset, ioend->io_size);
  6905			btrfs_bio_save_iter(btrfs_bio(bio));
  6906			if (ioend)
  6907				pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6908						__func__, __LINE__,
  6909						(unsigned long)ioend, ioend->io_inode->i_ino,
  6910						ioend->io_offset, ioend->io_size);
  6911			ret = __btrfs_map_block(fs_info, op, cur_logical, &map_length,
  6912						&bioc, mirror_num, 1);
  6913			if (ret)
  6914				return errno_to_blk_status(ret);
  6915	
  6916			if (cur_logical + map_length < orig_logical + orig_length) {
  6917				/*
  6918				 * For now zoned write should never cross stripe
  6919				 * boundary
  6920				 */
  6921				ASSERT(bio_op(bio) != REQ_OP_ZONE_APPEND);
  6922	
  6923				/* Split the bio */
  6924				cur_bio = btrfs_bio_split(fs_info, bio, map_length);
  6925			} else {
  6926				/* Use the existing bio directly */
  6927				cur_bio = bio;
  6928			}
  6929			btrfs_bio_counter_inc_blocked(fs_info);
  6930			ret = submit_one_mapped_range(fs_info, cur_bio, bioc,
  6931						      map_length, mirror_num);
  6932			btrfs_bio_counter_dec(fs_info);
  6933			if (ret < 0)
  6934				return errno_to_blk_status(ret);
  6935			cur_logical += map_length;
  6936		}
  6937		if (ioend)
  6938			pr_info("%s: %d ioend: %lx inode %lu %llu/%lu",
  6939					__func__, __LINE__,
  6940					(unsigned long)ioend, ioend->io_inode->i_ino,
  6941					ioend->io_offset, ioend->io_size);
  6942		return BLK_STS_OK;
  6943	}
  6944	

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

             reply	other threads:[~2021-12-30  5:22 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-30  5:21 kernel test robot [this message]
2021-12-30  5:21 ` [goldwynr:iomap 33/33] fs/btrfs/volumes.c:6893:23: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202112301331.RT7JMows-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=llvm@lists.linux.dev \
    --cc=rgoldwyn@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.