Subject: linux-next: build failure after merge of the ext3 tree

From: Stephen Rothwell <[email-masked]>
Date: Wed, 16 Aug 2023 10:37:51 +1000

Hi all,

After merging the ext3 tree, today's linux-next build (x86_64
allmodconfig) failed like this:

In file included from include/linux/kernel.h:30,
                 from arch/x86/include/asm/percpu.h:27,
                 from arch/x86/include/asm/preempt.h:6,
                 from include/linux/preempt.h:79,
                 from include/linux/spinlock.h:56,
                 from include/linux/wait.h:9,
                 from include/linux/wait_bit.h:8,
                 from include/linux/fs.h:6,
                 from include/linux/highmem.h:5,
                 from include/linux/bvec.h:10,
                 from include/linux/blk_types.h:10,
                 from include/linux/buffer_head.h:12,
                 from fs/ext2/xattr.c:57:
fs/ext2/xattr.c: In function 'ext2_xattr_set2':
fs/ext2/xattr.c:748:42: error: format '%d' expects argument of type 'int', but argument 2 has type 'ext2_fsblk_t' {aka 'long unsigned int'} [-Werror=format=]
  748 |                         ea_idebug(inode, "creating block %d", block);
      |                                          ^~~~~~~~~~~~~~~~~~~  ~~~~~
      |                                                               |
      |                                                               ext2_fsblk_t {aka long unsigned int}
include/linux/printk.h:427:25: note: in definition of macro 'printk_index_wrap'
  427 |                 _p_func(_fmt, ##__VA_ARGS__);                           \
      |                         ^~~~
include/linux/printk.h:129:17: note: in expansion of macro 'printk'
  129 |                 printk(fmt, ##__VA_ARGS__);             \
      |                 ^~~~~~
fs/ext2/xattr.c:88:33: note: in expansion of macro 'no_printk'
   88 | # define ea_idebug(inode, f...) no_printk(f)
      |                                 ^~~~~~~~~
fs/ext2/xattr.c:748:25: note: in expansion of macro 'ea_idebug'
  748 |                         ea_idebug(inode, "creating block %d", block);
      |                         ^~~~~~~~~
fs/ext2/xattr.c:748:59: note: format string is defined here
  748 |                         ea_idebug(inode, "creating block %d", block);
      |                                                          ~^
      |                                                           |
      |                                                           int
      |                                                          %ld

Caused by commit

  38e38567c5bd ("ext2: fix datatype of block number in ext2_xattr_set2()")

I have used the ext3 tree from next-20230815 for today.

-- 
Cheers,
Stephen Rothwell