[openeuler:openEuler-1.0-LTS 1755/1755] fs/xattr.c:222: warning: Function parameter or member 'dentry' not described in '__vfs_setxattr_locked'

Hi Frank, First bad commit (maybe != root cause): tree: https://gitee.com/openeuler/kernel.git openEuler-1.0-LTS head: 8723dc57de6371f9cc8764e15d4cc9e57ae592ee commit: 9f6e50a6f05d96c8ae495e278d86daa132b84085 [1755/1755] xattr: break delegations in {set, remove}xattr config: x86_64-buildonly-randconfig-2002-20250811 (https://download.01.org/0day-ci/archive/20250815/202508150542.jyItpbw4-lkp@i...) compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250815/202508150542.jyItpbw4-lkp@i...) 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@intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202508150542.jyItpbw4-lkp@intel.com/ All warnings (new ones prefixed by >>): fs/xattr.c:171: warning: Function parameter or member 'dentry' not described in '__vfs_setxattr_noperm' fs/xattr.c:171: warning: Function parameter or member 'name' not described in '__vfs_setxattr_noperm' fs/xattr.c:171: warning: Function parameter or member 'value' not described in '__vfs_setxattr_noperm' fs/xattr.c:171: warning: Function parameter or member 'size' not described in '__vfs_setxattr_noperm' fs/xattr.c:171: warning: Function parameter or member 'flags' not described in '__vfs_setxattr_noperm'
fs/xattr.c:222: warning: Function parameter or member 'dentry' not described in '__vfs_setxattr_locked' fs/xattr.c:222: warning: Function parameter or member 'name' not described in '__vfs_setxattr_locked' fs/xattr.c:222: warning: Function parameter or member 'value' not described in '__vfs_setxattr_locked' fs/xattr.c:222: warning: Function parameter or member 'size' not described in '__vfs_setxattr_locked' fs/xattr.c:222: warning: Function parameter or member 'flags' not described in '__vfs_setxattr_locked' fs/xattr.c:222: warning: Function parameter or member 'delegated_inode' not described in '__vfs_setxattr_locked' fs/xattr.c:429: warning: Function parameter or member 'dentry' not described in '__vfs_removexattr_locked' fs/xattr.c:429: warning: Function parameter or member 'name' not described in '__vfs_removexattr_locked' fs/xattr.c:429: warning: Function parameter or member 'delegated_inode' not described in '__vfs_removexattr_locked'
vim +222 fs/xattr.c b1ab7e4b2a88d3 David P. Quigley 2009-09-03 205 9f6e50a6f05d96 Frank van der Linden 2020-09-22 206 /** 9f6e50a6f05d96 Frank van der Linden 2020-09-22 207 * __vfs_setxattr_locked: set an extended attribute while holding the inode 9f6e50a6f05d96 Frank van der Linden 2020-09-22 208 * lock 9f6e50a6f05d96 Frank van der Linden 2020-09-22 209 * 9f6e50a6f05d96 Frank van der Linden 2020-09-22 210 * @dentry - object to perform setxattr on 9f6e50a6f05d96 Frank van der Linden 2020-09-22 211 * @name - xattr name to set 9f6e50a6f05d96 Frank van der Linden 2020-09-22 212 * @value - value to set @name to 9f6e50a6f05d96 Frank van der Linden 2020-09-22 213 * @size - size of @value 9f6e50a6f05d96 Frank van der Linden 2020-09-22 214 * @flags - flags to pass into filesystem operations 9f6e50a6f05d96 Frank van der Linden 2020-09-22 215 * @delegated_inode - on return, will contain an inode pointer that 9f6e50a6f05d96 Frank van der Linden 2020-09-22 216 * a delegation was broken on, NULL if none. 9f6e50a6f05d96 Frank van der Linden 2020-09-22 217 */ b1ab7e4b2a88d3 David P. Quigley 2009-09-03 218 int 9f6e50a6f05d96 Frank van der Linden 2020-09-22 219 __vfs_setxattr_locked(struct dentry *dentry, const char *name, 9f6e50a6f05d96 Frank van der Linden 2020-09-22 220 const void *value, size_t size, int flags, 9f6e50a6f05d96 Frank van der Linden 2020-09-22 221 struct inode **delegated_inode) b1ab7e4b2a88d3 David P. Quigley 2009-09-03 @222 { b1ab7e4b2a88d3 David P. Quigley 2009-09-03 223 struct inode *inode = dentry->d_inode; b1ab7e4b2a88d3 David P. Quigley 2009-09-03 224 int error; b1ab7e4b2a88d3 David P. Quigley 2009-09-03 225 b1ab7e4b2a88d3 David P. Quigley 2009-09-03 226 error = xattr_permission(inode, name, MAY_WRITE); b1ab7e4b2a88d3 David P. Quigley 2009-09-03 227 if (error) b1ab7e4b2a88d3 David P. Quigley 2009-09-03 228 return error; b1ab7e4b2a88d3 David P. Quigley 2009-09-03 229 b1ab7e4b2a88d3 David P. Quigley 2009-09-03 230 error = security_inode_setxattr(dentry, name, value, size, flags); b1ab7e4b2a88d3 David P. Quigley 2009-09-03 231 if (error) b1ab7e4b2a88d3 David P. Quigley 2009-09-03 232 goto out; b1ab7e4b2a88d3 David P. Quigley 2009-09-03 233 9f6e50a6f05d96 Frank van der Linden 2020-09-22 234 error = try_break_deleg(inode, delegated_inode); 9f6e50a6f05d96 Frank van der Linden 2020-09-22 235 if (error) 9f6e50a6f05d96 Frank van der Linden 2020-09-22 236 goto out; 9f6e50a6f05d96 Frank van der Linden 2020-09-22 237 b1ab7e4b2a88d3 David P. Quigley 2009-09-03 238 error = __vfs_setxattr_noperm(dentry, name, value, size, flags); b1ab7e4b2a88d3 David P. Quigley 2009-09-03 239 5be196e5f925da Christoph Hellwig 2006-01-09 240 out: 9f6e50a6f05d96 Frank van der Linden 2020-09-22 241 return error; 9f6e50a6f05d96 Frank van der Linden 2020-09-22 242 } 9f6e50a6f05d96 Frank van der Linden 2020-09-22 243 EXPORT_SYMBOL_GPL(__vfs_setxattr_locked); 9f6e50a6f05d96 Frank van der Linden 2020-09-22 244 :::::: The code at line 222 was first introduced by commit :::::: b1ab7e4b2a88d3ac13771463be8f302ce1616cfc VFS: Factor out part of vfs_setxattr so it can be called from the SELinux hook for inode_setsecctx. :::::: TO: David P. Quigley <dpquigl@tycho.nsa.gov> :::::: CC: James Morris <jmorris@namei.org> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
participants (1)
-
kernel test robot