Eric, Could you PTAL?
On 06/05, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git > dev-test > head: adf3d3a53cf13d0998c699ba43d8582c875179e3 > commit: adf3d3a53cf13d0998c699ba43d8582c875179e3 [48/48] f2fs: don't return > vmalloc() memory from f2fs_kmalloc() > config: powerpc64-randconfig-r011-20200605 (attached as .config) > compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project > ac47588bc4ff5927a01ed6fcd269ce86aba52a7c) > 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 > # install powerpc64 cross compiling tool for clang build > # apt-get install binutils-powerpc64-linux-gnu > git checkout adf3d3a53cf13d0998c699ba43d8582c875179e3 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross > ARCH=powerpc64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <[email protected]> > > All warnings (new ones prefixed by >>, old ones prefixed by <<): > > In file included from arch/powerpc/include/asm/hardirq.h:6: > In file included from include/linux/irq.h:20: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:605: > arch/powerpc/include/asm/io-defs.h:45:1: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > DEF_PCI_AC_NORET(insw, (unsigned long p, void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:602:3: note: expanded from macro > 'DEF_PCI_AC_NORET' > __do_##name al; > ^~~~~~~~~~~~~~ > <scratch space>:50:1: note: expanded from here > __do_insw > ^ > arch/powerpc/include/asm/io.h:543:56: note: expanded from macro '__do_insw' > #define __do_insw(p, b, n) readsw((PCI_IO_ADDR)_IO_BASE+(p), (b), (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from fs/f2fs/super.c:12: > In file included from include/linux/buffer_head.h:14: > In file included from include/linux/pagemap.h:11: > In file included from include/linux/highmem.h:10: > In file included from include/linux/hardirq.h:9: > In file included from arch/powerpc/include/asm/hardirq.h:6: > In file included from include/linux/irq.h:20: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:605: > arch/powerpc/include/asm/io-defs.h:47:1: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > DEF_PCI_AC_NORET(insl, (unsigned long p, void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:602:3: note: expanded from macro > 'DEF_PCI_AC_NORET' > __do_##name al; > ^~~~~~~~~~~~~~ > <scratch space>:52:1: note: expanded from here > __do_insl > ^ > arch/powerpc/include/asm/io.h:544:56: note: expanded from macro '__do_insl' > #define __do_insl(p, b, n) readsl((PCI_IO_ADDR)_IO_BASE+(p), (b), (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from fs/f2fs/super.c:12: > In file included from include/linux/buffer_head.h:14: > In file included from include/linux/pagemap.h:11: > In file included from include/linux/highmem.h:10: > In file included from include/linux/hardirq.h:9: > In file included from arch/powerpc/include/asm/hardirq.h:6: > In file included from include/linux/irq.h:20: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:605: > arch/powerpc/include/asm/io-defs.h:49:1: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > DEF_PCI_AC_NORET(outsb, (unsigned long p, const void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:602:3: note: expanded from macro > 'DEF_PCI_AC_NORET' > __do_##name al; > ^~~~~~~~~~~~~~ > <scratch space>:54:1: note: expanded from here > __do_outsb > ^ > arch/powerpc/include/asm/io.h:545:58: note: expanded from macro '__do_outsb' > #define __do_outsb(p, b, n) writesb((PCI_IO_ADDR)_IO_BASE+(p),(b),(n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from fs/f2fs/super.c:12: > In file included from include/linux/buffer_head.h:14: > In file included from include/linux/pagemap.h:11: > In file included from include/linux/highmem.h:10: > In file included from include/linux/hardirq.h:9: > In file included from arch/powerpc/include/asm/hardirq.h:6: > In file included from include/linux/irq.h:20: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:605: > arch/powerpc/include/asm/io-defs.h:51:1: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > DEF_PCI_AC_NORET(outsw, (unsigned long p, const void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:602:3: note: expanded from macro > 'DEF_PCI_AC_NORET' > __do_##name al; > ^~~~~~~~~~~~~~ > <scratch space>:56:1: note: expanded from here > __do_outsw > ^ > arch/powerpc/include/asm/io.h:546:58: note: expanded from macro '__do_outsw' > #define __do_outsw(p, b, n) writesw((PCI_IO_ADDR)_IO_BASE+(p),(b),(n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from fs/f2fs/super.c:12: > In file included from include/linux/buffer_head.h:14: > In file included from include/linux/pagemap.h:11: > In file included from include/linux/highmem.h:10: > In file included from include/linux/hardirq.h:9: > In file included from arch/powerpc/include/asm/hardirq.h:6: > In file included from include/linux/irq.h:20: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:605: > arch/powerpc/include/asm/io-defs.h:53:1: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > DEF_PCI_AC_NORET(outsl, (unsigned long p, const void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:602:3: note: expanded from macro > 'DEF_PCI_AC_NORET' > __do_##name al; > ^~~~~~~~~~~~~~ > <scratch space>:58:1: note: expanded from here > __do_outsl > ^ > arch/powerpc/include/asm/io.h:547:58: note: expanded from macro '__do_outsl' > #define __do_outsl(p, b, n) writesl((PCI_IO_ADDR)_IO_BASE+(p),(b),(n)) > ~~~~~~~~~~~~~~~~~~~~~^ > >> fs/f2fs/super.c:3303:12: warning: stack frame size of 2064 bytes in > >> function 'f2fs_fill_super' [-Wframe-larger-than=] > static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > ^ > 13 warnings generated. > > vim +/f2fs_fill_super +3303 fs/f2fs/super.c > > 84b89e5d943d8d Jaegeuk Kim 2018-02-22 3302 > aff063e266cbf4 Jaegeuk Kim 2012-11-02 @3303 static int > f2fs_fill_super(struct super_block *sb, void *data, int silent) > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3304 { > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3305 struct f2fs_sb_info > *sbi; > da554e48caab95 hujianyang 2015-05-21 3306 struct f2fs_super_block > *raw_super; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3307 struct inode *root; > 99e3e858a486cc Sheng Yong 2016-05-11 3308 int err; > aa2c8c43e4a5c2 Chao Yu 2019-02-19 3309 bool skip_recovery = > false, need_fsck = false; > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3310 char *options = NULL; > e8240f656d4d5d Chao Yu 2015-12-15 3311 int recovery, i, > valid_super_block; > 8f1dbbbbdfe9ba Shuoran Liu 2016-01-27 3312 struct curseg_info > *seg_i; > aa2c8c43e4a5c2 Chao Yu 2019-02-19 3313 int retry_cnt = 1; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3314 > ed2e621a95d704 Jaegeuk Kim 2014-08-08 3315 try_onemore: > da554e48caab95 hujianyang 2015-05-21 3316 err = -EINVAL; > da554e48caab95 hujianyang 2015-05-21 3317 raw_super = NULL; > e8240f656d4d5d Chao Yu 2015-12-15 3318 valid_super_block = -1; > da554e48caab95 hujianyang 2015-05-21 3319 recovery = 0; > da554e48caab95 hujianyang 2015-05-21 3320 > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3321 /* allocate memory for > f2fs-specific super block info */ > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3322 sbi = > kzalloc(sizeof(struct f2fs_sb_info), GFP_KERNEL); > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3323 if (!sbi) > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3324 return -ENOMEM; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3325 > df728b0f6954c3 Jaegeuk Kim 2016-03-23 3326 sbi->sb = sb; > df728b0f6954c3 Jaegeuk Kim 2016-03-23 3327 > 43b6573bac95d7 Keith Mok 2016-03-02 3328 /* Load the checksum > driver */ > 43b6573bac95d7 Keith Mok 2016-03-02 3329 sbi->s_chksum_driver = > crypto_alloc_shash("crc32", 0, 0); > 43b6573bac95d7 Keith Mok 2016-03-02 3330 if > (IS_ERR(sbi->s_chksum_driver)) { > dcbb4c10e6d969 Joe Perches 2019-06-18 3331 f2fs_err(sbi, > "Cannot load crc32 driver."); > 43b6573bac95d7 Keith Mok 2016-03-02 3332 err = > PTR_ERR(sbi->s_chksum_driver); > 43b6573bac95d7 Keith Mok 2016-03-02 3333 > sbi->s_chksum_driver = NULL; > 43b6573bac95d7 Keith Mok 2016-03-02 3334 goto free_sbi; > 43b6573bac95d7 Keith Mok 2016-03-02 3335 } > 43b6573bac95d7 Keith Mok 2016-03-02 3336 > ff9234ad4e9747 Namjae Jeon 2013-01-12 3337 /* set a block size */ > 6bacf52fb58aeb Jaegeuk Kim 2013-12-06 3338 if > (unlikely(!sb_set_blocksize(sb, F2FS_BLKSIZE))) { > dcbb4c10e6d969 Joe Perches 2019-06-18 3339 f2fs_err(sbi, > "unable to set blocksize"); > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3340 goto free_sbi; > a07ef784356cf9 Namjae Jeon 2012-12-30 3341 } > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3342 > df728b0f6954c3 Jaegeuk Kim 2016-03-23 3343 err = > read_raw_super_block(sbi, &raw_super, &valid_super_block, > e8240f656d4d5d Chao Yu 2015-12-15 3344 > &recovery); > c0d39e65ba3243 Namjae Jeon 2013-03-17 3345 if (err) > 9076a75f8e0f23 Gu Zheng 2013-10-14 3346 goto free_sbi; > 9076a75f8e0f23 Gu Zheng 2013-10-14 3347 > 5fb08372a68936 Gu Zheng 2013-06-07 3348 sb->s_fs_info = sbi; > 52763a4b7a2112 Jaegeuk Kim 2016-06-13 3349 sbi->raw_super = > raw_super; > 52763a4b7a2112 Jaegeuk Kim 2016-06-13 3350 > 704956ecf5bcdc Chao Yu 2017-07-31 3351 /* precompute checksum > seed for metadata */ > 7beb01f74415c5 Chao Yu 2018-10-24 3352 if > (f2fs_sb_has_inode_chksum(sbi)) > 704956ecf5bcdc Chao Yu 2017-07-31 3353 > sbi->s_chksum_seed = f2fs_chksum(sbi, ~0, raw_super->uuid, > 704956ecf5bcdc Chao Yu 2017-07-31 3354 > sizeof(raw_super->uuid)); > 704956ecf5bcdc Chao Yu 2017-07-31 3355 > d1b959c8770260 Damien Le Moal 2016-10-28 3356 /* > d1b959c8770260 Damien Le Moal 2016-10-28 3357 * The BLKZONED feature > indicates that the drive was formatted with > d1b959c8770260 Damien Le Moal 2016-10-28 3358 * zone alignment > optimization. This is optional for host-aware > d1b959c8770260 Damien Le Moal 2016-10-28 3359 * devices, but > mandatory for host-managed zoned block devices. > d1b959c8770260 Damien Le Moal 2016-10-28 3360 */ > d1b959c8770260 Damien Le Moal 2016-10-28 3361 #ifndef CONFIG_BLK_DEV_ZONED > 7beb01f74415c5 Chao Yu 2018-10-24 3362 if > (f2fs_sb_has_blkzoned(sbi)) { > dcbb4c10e6d969 Joe Perches 2019-06-18 3363 f2fs_err(sbi, > "Zoned block device support is not enabled"); > 1727f317219bfc Chao Yu 2017-06-12 3364 err = > -EOPNOTSUPP; > d1b959c8770260 Damien Le Moal 2016-10-28 3365 goto > free_sb_buf; > d1b959c8770260 Damien Le Moal 2016-10-28 3366 } > d1b959c8770260 Damien Le Moal 2016-10-28 3367 #endif > 498c5e9fcd10c8 Yunlei He 2015-05-07 3368 default_options(sbi); > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3369 /* parse mount options > */ > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3370 options = > kstrdup((const char *)data, GFP_KERNEL); > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3371 if (data && !options) { > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3372 err = -ENOMEM; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3373 goto > free_sb_buf; > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3374 } > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3375 > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3376 err = parse_options(sb, > options); > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3377 if (err) > dabc4a5c60f796 Jaegeuk Kim 2015-01-23 3378 goto > free_options; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3379 > e0afc4d6d0d3e7 Chao Yu 2015-12-31 3380 sbi->max_file_blocks = > max_file_blocks(); > e0afc4d6d0d3e7 Chao Yu 2015-12-31 3381 sb->s_maxbytes = > sbi->max_file_blocks << > e0afc4d6d0d3e7 Chao Yu 2015-12-31 3382 > le32_to_cpu(raw_super->log_blocksize); > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3383 sb->s_max_links = > F2FS_LINK_MAX; > aff063e266cbf4 Jaegeuk Kim 2012-11-02 3384 > 5aba54302a46fd Daniel Rosenberg 2019-07-23 3385 err = > f2fs_setup_casefold(sbi); > 5aba54302a46fd Daniel Rosenberg 2019-07-23 3386 if (err) > 5aba54302a46fd Daniel Rosenberg 2019-07-23 3387 goto > free_options; > 5aba54302a46fd Daniel Rosenberg 2019-07-23 3388 > > :::::: The code at line 3303 was first introduced by commit > :::::: aff063e266cbf4754021d8e5d16ee418560906fd f2fs: add super block > operations > > :::::: TO: Jaegeuk Kim <[email protected]> > :::::: CC: Jaegeuk Kim <[email protected]> > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/[email protected] _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
