On Sun, Mar 09, 2025 at 01:06:40AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git test-merge > head: fac04210bb99888fd453db09239bed27436fd619 > commit: fac04210bb99888fd453db09239bed27436fd619 [13/13] Merge branch 'xfs-6.15-atomicwrites' into for-next > config: i386-buildonly-randconfig-003-20250308 (https://download.01.org/0day-ci/archive/20250309/202503090149.Wu0ag7zs-lkp@xxxxxxxxx/config) > compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90) > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250309/202503090149.Wu0ag7zs-lkp@xxxxxxxxx/reproduce) Huh? All xfs branches are monitored? This was by no means to be tested. Carlos > > 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@xxxxxxxxx> > | Closes: https://lore.kernel.org/oe-kbuild-all/202503090149.Wu0ag7zs-lkp@xxxxxxxxx/ > > All errors (new ones prefixed by >>): > > >> fs/xfs/xfs_file.c:746:49: error: too few arguments to function call, expected 4, have 3 > 746 | ret = xfs_file_write_checks(iocb, from, &iolock); > | ~~~~~~~~~~~~~~~~~~~~~ ^ > fs/xfs/xfs_file.c:434:1: note: 'xfs_file_write_checks' declared here > 434 | xfs_file_write_checks( > | ^ > 435 | struct kiocb *iocb, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 436 | struct iov_iter *from, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 437 | unsigned int *iolock, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 438 | struct xfs_zone_alloc_ctx *ac) > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 1 error generated. > > > vim +746 fs/xfs/xfs_file.c > > 2e2383405824b9 Christoph Hellwig 2025-01-27 730 > 307185b178ac26 John Garry 2025-03-03 731 static noinline ssize_t > 307185b178ac26 John Garry 2025-03-03 732 xfs_file_dio_write_atomic( > 307185b178ac26 John Garry 2025-03-03 733 struct xfs_inode *ip, > 307185b178ac26 John Garry 2025-03-03 734 struct kiocb *iocb, > 307185b178ac26 John Garry 2025-03-03 735 struct iov_iter *from) > 307185b178ac26 John Garry 2025-03-03 736 { > 307185b178ac26 John Garry 2025-03-03 737 unsigned int iolock = XFS_IOLOCK_SHARED; > 307185b178ac26 John Garry 2025-03-03 738 unsigned int dio_flags = 0; > 307185b178ac26 John Garry 2025-03-03 739 ssize_t ret; > 307185b178ac26 John Garry 2025-03-03 740 > 307185b178ac26 John Garry 2025-03-03 741 retry: > 307185b178ac26 John Garry 2025-03-03 742 ret = xfs_ilock_iocb_for_write(iocb, &iolock); > 307185b178ac26 John Garry 2025-03-03 743 if (ret) > 307185b178ac26 John Garry 2025-03-03 744 return ret; > 307185b178ac26 John Garry 2025-03-03 745 > 307185b178ac26 John Garry 2025-03-03 @746 ret = xfs_file_write_checks(iocb, from, &iolock); > 307185b178ac26 John Garry 2025-03-03 747 if (ret) > 307185b178ac26 John Garry 2025-03-03 748 goto out_unlock; > 307185b178ac26 John Garry 2025-03-03 749 > 307185b178ac26 John Garry 2025-03-03 750 if (dio_flags & IOMAP_DIO_FORCE_WAIT) > 307185b178ac26 John Garry 2025-03-03 751 inode_dio_wait(VFS_I(ip)); > 307185b178ac26 John Garry 2025-03-03 752 > 307185b178ac26 John Garry 2025-03-03 753 trace_xfs_file_direct_write(iocb, from); > 307185b178ac26 John Garry 2025-03-03 754 ret = iomap_dio_rw(iocb, from, &xfs_atomic_write_iomap_ops, > 307185b178ac26 John Garry 2025-03-03 755 &xfs_dio_write_ops, dio_flags, NULL, 0); > 307185b178ac26 John Garry 2025-03-03 756 > 307185b178ac26 John Garry 2025-03-03 757 if (ret == -EAGAIN && !(iocb->ki_flags & IOCB_NOWAIT) && > 307185b178ac26 John Garry 2025-03-03 758 !(dio_flags & IOMAP_DIO_ATOMIC_SW)) { > 307185b178ac26 John Garry 2025-03-03 759 xfs_iunlock(ip, iolock); > 307185b178ac26 John Garry 2025-03-03 760 dio_flags = IOMAP_DIO_ATOMIC_SW | IOMAP_DIO_FORCE_WAIT; > 307185b178ac26 John Garry 2025-03-03 761 iolock = XFS_IOLOCK_EXCL; > 307185b178ac26 John Garry 2025-03-03 762 goto retry; > 307185b178ac26 John Garry 2025-03-03 763 } > 307185b178ac26 John Garry 2025-03-03 764 > 307185b178ac26 John Garry 2025-03-03 765 out_unlock: > 307185b178ac26 John Garry 2025-03-03 766 if (iolock) > 307185b178ac26 John Garry 2025-03-03 767 xfs_iunlock(ip, iolock); > 307185b178ac26 John Garry 2025-03-03 768 return ret; > 307185b178ac26 John Garry 2025-03-03 769 } > 307185b178ac26 John Garry 2025-03-03 770 > > :::::: The code at line 746 was first introduced by commit > :::::: 307185b178ac2695cbd964e9b0a5a9b7513bba93 xfs: Add xfs_file_dio_write_atomic() > > :::::: TO: John Garry <john.g.garry@xxxxxxxxxx> > :::::: CC: Carlos Maiolino <cem@xxxxxxxxxx> > > -- > 0-DAY CI Kernel Test Service > https://github.com/intel/lkp-tests/wiki >
