Building the Linux kernel with Clang and LLVM
 help / color / mirror / Atom feed
* Re: [RFC PATCH 16/19] famfs_fuse: Add holder_operations for dax notify_failure()
       [not found] <20250421013346.32530-17-john@groves.net>
@ 2025-04-21  3:27 ` kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2025-04-21  3:27 UTC (permalink / raw)
  To: John Groves; +Cc: llvm, oe-kbuild-all

Hi John,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:

[auto build test WARNING on 38fec10eb60d687e30c8c6b5420d86e8149f7557]

url:    https://github.com/intel-lab-lkp/linux/commits/John-Groves/dev_dax_iomap-Move-dax_pgoff_to_phys-from-device-c-to-bus-c/20250421-094020
base:   38fec10eb60d687e30c8c6b5420d86e8149f7557
patch link:    https://lore.kernel.org/r/20250421013346.32530-17-john%40groves.net
patch subject: [RFC PATCH 16/19] famfs_fuse: Add holder_operations for dax notify_failure()
config: hexagon-randconfig-001-20250421 (https://download.01.org/0day-ci/archive/20250421/202504211107.mDN9xA7M-lkp@intel.com/config)
compiler: clang version 16.0.6 (https://github.com/llvm/llvm-project 7cbf1a2591520c2491aa35339f227775f4d3adf6)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250421/202504211107.mDN9xA7M-lkp@intel.com/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/202504211107.mDN9xA7M-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> fs/fuse/famfs.c:325:1: warning: label at end of compound statement is a C2x extension [-Wc2x-extensions]
   }
   ^
   fs/fuse/famfs.c:402:30: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                          __func__, __LINE__, next_offset, fmap_buf_size);
                                              ^~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:402:43: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                          __func__, __LINE__, next_offset, fmap_buf_size);
                                                           ^~~~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:436:31: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                  __func__, __LINE__, next_offset, fmap_buf_size);
                                                      ^~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:436:44: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                  __func__, __LINE__, next_offset, fmap_buf_size);
                                                                   ^~~~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:501:32: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                          __func__, __LINE__, next_offset, fmap_buf_size);
                                                              ^~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:501:45: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                          __func__, __LINE__, next_offset, fmap_buf_size);
                                                                           ^~~~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:526:32: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                          __func__, __LINE__, next_offset, fmap_buf_size);
                                                              ^~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:526:45: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                                          __func__, __LINE__, next_offset, fmap_buf_size);
                                                                           ^~~~~~~~~~~~~
   include/linux/printk.h:544:33: note: expanded from macro 'pr_err'
           printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
                                  ~~~     ^~~~~~~~~~~
   include/linux/printk.h:501:60: note: expanded from macro 'printk'
   #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__)
                                                       ~~~    ^~~~~~~~~~~
   include/linux/printk.h:473:19: note: expanded from macro 'printk_index_wrap'
                   _p_func(_fmt, ##__VA_ARGS__);                           \
                           ~~~~    ^~~~~~~~~~~
   fs/fuse/famfs.c:593:20: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
                          __func__, meta->file_size, extent_total);

Kconfig warnings: (for reference only)
   WARNING: unmet direct dependencies detected for DEV_DAX_IOMAP
   Depends on [n]: DEV_DAX [=n] && DAX [=y]
   Selected by [y]:
   - FUSE_FAMFS_DAX [=y] && FUSE_FS [=y]


vim +325 fs/fuse/famfs.c

   294	
   295	static void
   296	famfs_set_daxdev_err(
   297		struct fuse_conn *fc,
   298		struct dax_device *dax_devp)
   299	{
   300		int i;
   301	
   302		/* Gotta search the list by dax_devp;
   303		 * read lock because we're not adding or removing daxdev entries
   304		 */
   305		down_read(&fc->famfs_devlist_sem);
   306		for (i = 0; i < fc->dax_devlist->nslots; i++) {
   307			if (fc->dax_devlist->devlist[i].valid) {
   308				struct famfs_daxdev *dd = &fc->dax_devlist->devlist[i];
   309	
   310				if (dd->devp != dax_devp)
   311					continue;
   312	
   313				dd->error = true;
   314				up_read(&fc->famfs_devlist_sem);
   315	
   316				pr_err("%s: memory error on daxdev %s (%d)\n",
   317				       __func__, dd->name, i);
   318				goto done;
   319			}
   320		}
   321		up_read(&fc->famfs_devlist_sem);
   322		pr_err("%s: memory err on unrecognized daxdev\n", __func__);
   323	
   324	done:
 > 325	}
   326	

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2025-04-21  3:28 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20250421013346.32530-17-john@groves.net>
2025-04-21  3:27 ` [RFC PATCH 16/19] famfs_fuse: Add holder_operations for dax notify_failure() kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox