Hi Jiapeng, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on xfs-linux/for-next] [also build test WARNING on v5.18-rc7 next-20220519] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/intel-lab-lkp/linux/commits/Jiapeng-Chong/xfs-Remove-the-redundant-assignment/20220520-120736 base: https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git for-next config: x86_64-randconfig-a005 (https://download.01.org/0day-ci/archive/20220520/202205202134.P3H4KQ7S-lkp@xxxxxxxxx/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project e00cbbec06c08dc616a0d52a20f678b8fbd4e304) 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/intel-lab-lkp/linux/commit/890587fbe3e3934d20fcd39effdcefedd65e6b7f git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Jiapeng-Chong/xfs-Remove-the-redundant-assignment/20220520-120736 git checkout 890587fbe3e3934d20fcd39effdcefedd65e6b7f # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash fs/xfs/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): >> fs/xfs/xfs_log_recover.c:3542:20: warning: unused variable 'agfp' [-Wunused-variable] struct xfs_agf *agfp = agfbp->b_addr; ^ >> fs/xfs/xfs_log_recover.c:3551:20: warning: unused variable 'agi' [-Wunused-variable] struct xfs_agi *agi = agibp->b_addr; ^ 2 warnings generated. vim +/agfp +3542 fs/xfs/xfs_log_recover.c ^1da177e4c3f41 Linus Torvalds 2005-04-16 3519 ^1da177e4c3f41 Linus Torvalds 2005-04-16 3520 #if defined(DEBUG) ^1da177e4c3f41 Linus Torvalds 2005-04-16 3521 /* ^1da177e4c3f41 Linus Torvalds 2005-04-16 3522 * Read all of the agf and agi counters and check that they ^1da177e4c3f41 Linus Torvalds 2005-04-16 3523 * are consistent with the superblock counters. ^1da177e4c3f41 Linus Torvalds 2005-04-16 3524 */ e89fbb5ee1893f Christoph Hellwig 2017-11-06 3525 STATIC void ^1da177e4c3f41 Linus Torvalds 2005-04-16 3526 xlog_recover_check_summary( 9a8d2fdbb47aaa Mark Tinguely 2012-06-14 3527 struct xlog *log) ^1da177e4c3f41 Linus Torvalds 2005-04-16 3528 { 934933c3eec9e4 Dave Chinner 2021-06-02 3529 struct xfs_mount *mp = log->l_mp; 934933c3eec9e4 Dave Chinner 2021-06-02 3530 struct xfs_perag *pag; e82226138b20d4 Dave Chinner 2020-12-16 3531 struct xfs_buf *agfbp; e82226138b20d4 Dave Chinner 2020-12-16 3532 struct xfs_buf *agibp; ^1da177e4c3f41 Linus Torvalds 2005-04-16 3533 xfs_agnumber_t agno; 5e1be0fb1a3950 Christoph Hellwig 2008-11-28 3534 int error; ^1da177e4c3f41 Linus Torvalds 2005-04-16 3535 934933c3eec9e4 Dave Chinner 2021-06-02 3536 for_each_perag(mp, agno, pag) { 934933c3eec9e4 Dave Chinner 2021-06-02 3537 error = xfs_read_agf(mp, NULL, pag->pag_agno, 0, &agfbp); 4805621a37d9b2 Christoph Hellwig 2008-11-28 3538 if (error) { a0fa2b679ecd15 Dave Chinner 2011-03-07 3539 xfs_alert(mp, "%s agf read failed agno %d error %d", 934933c3eec9e4 Dave Chinner 2021-06-02 3540 __func__, pag->pag_agno, error); 4805621a37d9b2 Christoph Hellwig 2008-11-28 3541 } else { 9798f615ad2be4 Christoph Hellwig 2020-03-10 @3542 struct xfs_agf *agfp = agfbp->b_addr; ^1da177e4c3f41 Linus Torvalds 2005-04-16 3543 xfs_buf_relse(agfbp); 4805621a37d9b2 Christoph Hellwig 2008-11-28 3544 } ^1da177e4c3f41 Linus Torvalds 2005-04-16 3545 934933c3eec9e4 Dave Chinner 2021-06-02 3546 error = xfs_read_agi(mp, NULL, pag->pag_agno, &agibp); a0fa2b679ecd15 Dave Chinner 2011-03-07 3547 if (error) { a0fa2b679ecd15 Dave Chinner 2011-03-07 3548 xfs_alert(mp, "%s agi read failed agno %d error %d", 934933c3eec9e4 Dave Chinner 2021-06-02 3549 __func__, pag->pag_agno, error); a0fa2b679ecd15 Dave Chinner 2011-03-07 3550 } else { 370c782b98436b Christoph Hellwig 2020-03-10 @3551 struct xfs_agi *agi = agibp->b_addr; -- 0-DAY CI Kernel Test Service https://01.org/lkp
