Re: [PATCH 1/3] Change inode_operations.mkdir to return struct dentry *

From: kernel test robot
Date: Mon Feb 17 2025 - 11:07:42 EST


Hi NeilBrown,

kernel test robot noticed the following build warnings:

[auto build test WARNING on brauner-vfs/vfs.all]
[also build test WARNING on trondmy-nfs/linux-next driver-core/driver-core-testing driver-core/driver-core-next driver-core/driver-core-linus cifs/for-next xfs-linux/for-next linus/master v6.14-rc3 next-20250217]
[cannot apply to ericvh-v9fs/for-next tyhicks-ecryptfs/next]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/NeilBrown/nfs-change-mkdir-inode_operation-to-return-alternate-dentry-if-needed/20250214-141741
base: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git vfs.all
patch link: https://lore.kernel.org/r/20250214052204.3105610-2-neilb%40suse.de
patch subject: [PATCH 1/3] Change inode_operations.mkdir to return struct dentry *
config: um-randconfig-r122-20250217 (https://download.01.org/0day-ci/archive/20250217/202502172326.hdZnGqWS-lkp@xxxxxxxxx/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250217/202502172326.hdZnGqWS-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202502172326.hdZnGqWS-lkp@xxxxxxxxx/

sparse warnings: (new ones prefixed by >>)
>> fs/hostfs/hostfs_kern.c:689:24: sparse: sparse: incorrect type in return expression (different base types) @@ expected struct dentry * @@ got int @@
fs/hostfs/hostfs_kern.c:689:24: sparse: expected struct dentry *
fs/hostfs/hostfs_kern.c:689:24: sparse: got int

vim +689 fs/hostfs/hostfs_kern.c

^1da177e4c3f41 Linus Torvalds 2005-04-16 681
456289e0bd14ce NeilBrown 2025-02-14 682 static struct dentry *hostfs_mkdir(struct mnt_idmap *idmap, struct inode *ino,
549c7297717c32 Christian Brauner 2021-01-21 683 struct dentry *dentry, umode_t mode)
^1da177e4c3f41 Linus Torvalds 2005-04-16 684 {
^1da177e4c3f41 Linus Torvalds 2005-04-16 685 char *file;
^1da177e4c3f41 Linus Torvalds 2005-04-16 686 int err;
^1da177e4c3f41 Linus Torvalds 2005-04-16 687
c5322220eb91b9 Al Viro 2010-06-06 688 if ((file = dentry_name(dentry)) == NULL)
f1adc05e773830 Jeff Dike 2007-05-08 @689 return -ENOMEM;
^1da177e4c3f41 Linus Torvalds 2005-04-16 690 err = do_mkdir(file, mode);
e9193059b1b373 Al Viro 2010-06-06 691 __putname(file);
456289e0bd14ce NeilBrown 2025-02-14 692 if (err)
456289e0bd14ce NeilBrown 2025-02-14 693 return ERR_PTR(err);
456289e0bd14ce NeilBrown 2025-02-14 694 else
456289e0bd14ce NeilBrown 2025-02-14 695 return dentry;
^1da177e4c3f41 Linus Torvalds 2005-04-16 696 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 697

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