Re: fs/smb/server/vfs.c:929: warning: Function parameter or member 'path' not described in 'ksmbd_vfs_setxattr'

From: Namjae Jeon
Date: Fri Oct 06 2023 - 21:01:20 EST


2023-10-07 5:41 GMT+09:00, kernel test robot <lkp@xxxxxxxxx>:
> Hi Namjae,
Hi,
>
> FYI, the error/warning still remains.
Okay, I will fix it.
Thanks for your report:)
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> master
> head: af95dc6fdc25e616051d0234aad638e15c02ec8f
> commit: 40b268d384a22276dca1450549f53eed60e21deb ksmbd: add mnt_want_write
> to ksmbd vfs functions
> date: 4 months ago
> config: m68k-allyesconfig
> (https://download.01.org/0day-ci/archive/20231007/202310070441.73IUVG65-lkp@xxxxxxxxx/config)
> compiler: m68k-linux-gcc (GCC) 13.2.0
> reproduce (this is a W=1 build):
> (https://download.01.org/0day-ci/archive/20231007/202310070441.73IUVG65-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/202310070441.73IUVG65-lkp@xxxxxxxxx/
>
> All warnings (new ones prefixed by >>):
>
> fs/smb/server/vfs.c:54: warning: Function parameter or member 'parent'
> not described in 'ksmbd_vfs_lock_parent'
> fs/smb/server/vfs.c:54: warning: Function parameter or member 'child' not
> described in 'ksmbd_vfs_lock_parent'
> fs/smb/server/vfs.c:372: warning: Function parameter or member 'fp' not
> described in 'ksmbd_vfs_read'
> fs/smb/server/vfs.c:372: warning: Excess function parameter 'fid'
> description in 'ksmbd_vfs_read'
> fs/smb/server/vfs.c:488: warning: Function parameter or member 'fp' not
> described in 'ksmbd_vfs_write'
> fs/smb/server/vfs.c:488: warning: Excess function parameter 'fid'
> description in 'ksmbd_vfs_write'
> fs/smb/server/vfs.c:551: warning: Function parameter or member 'path' not
> described in 'ksmbd_vfs_getattr'
> fs/smb/server/vfs.c:551: warning: Function parameter or member 'stat' not
> described in 'ksmbd_vfs_getattr'
> fs/smb/server/vfs.c:551: warning: Excess function parameter 'work'
> description in 'ksmbd_vfs_getattr'
> fs/smb/server/vfs.c:551: warning: Excess function parameter 'fid'
> description in 'ksmbd_vfs_getattr'
> fs/smb/server/vfs.c:551: warning: Excess function parameter 'attrs'
> description in 'ksmbd_vfs_getattr'
> fs/smb/server/vfs.c:568: warning: Function parameter or member 'p_id' not
> described in 'ksmbd_vfs_fsync'
> fs/smb/server/vfs.c:591: warning: Function parameter or member 'work' not
> described in 'ksmbd_vfs_remove_file'
> fs/smb/server/vfs.c:591: warning: Function parameter or member 'path' not
> described in 'ksmbd_vfs_remove_file'
> fs/smb/server/vfs.c:591: warning: Excess function parameter 'name'
> description in 'ksmbd_vfs_remove_file'
> fs/smb/server/vfs.c:634: warning: Function parameter or member 'work' not
> described in 'ksmbd_vfs_link'
> fs/smb/server/vfs.c:811: warning: Function parameter or member 'fp' not
> described in 'ksmbd_vfs_truncate'
> fs/smb/server/vfs.c:811: warning: Excess function parameter 'fid'
> description in 'ksmbd_vfs_truncate'
> fs/smb/server/vfs.c:852: warning: Excess function parameter 'size'
> description in 'ksmbd_vfs_listxattr'
>>> fs/smb/server/vfs.c:929: warning: Function parameter or member 'path' not
>>> described in 'ksmbd_vfs_setxattr'
>>> fs/smb/server/vfs.c:929: warning: Excess function parameter 'dentry'
>>> description in 'ksmbd_vfs_setxattr'
> fs/smb/server/vfs.c:954: warning: Function parameter or member 'option'
> not described in 'ksmbd_vfs_set_fadvise'
> fs/smb/server/vfs.c:954: warning: Excess function parameter 'options'
> description in 'ksmbd_vfs_set_fadvise'
> fs/smb/server/vfs.c:1168: warning: Function parameter or member 'um' not
> described in 'ksmbd_vfs_lookup_in_dir'
> fs/smb/server/vfs.c:1203: warning: Function parameter or member 'work'
> not described in 'ksmbd_vfs_kern_path_locked'
>
>
> vim +929 fs/smb/server/vfs.c
>
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 914
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 915 /**
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 916 *
> ksmbd_vfs_setxattr() - vfs helper for smb set extended attributes value
> 4609e1f18e19c3 fs/ksmbd/vfs.c Christian Brauner 2023-01-13 917 *
> @idmap: idmap of the relevant mount
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 918 *
> @dentry: dentry to set XATTR at
> 63f09a9986eb58 fs/ksmbd/vfs.c Jiapeng Chong 2023-02-08 919 *
> @attr_name: xattr name for setxattr
> 63f09a9986eb58 fs/ksmbd/vfs.c Jiapeng Chong 2023-02-08 920 *
> @attr_value: xattr value to set
> 63f09a9986eb58 fs/ksmbd/vfs.c Jiapeng Chong 2023-02-08 921 *
> @attr_size: size of xattr value
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 922 *
> @flags: destination buffer length
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 923 *
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 924 *
> Return: 0 on success, otherwise error
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 925 */
> 4609e1f18e19c3 fs/ksmbd/vfs.c Christian Brauner 2023-01-13 926 int
> ksmbd_vfs_setxattr(struct mnt_idmap *idmap,
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 927
> const struct path *path, const char *attr_name,
> 0c5fd887d2bb47 fs/ksmbd/vfs.c Christian Brauner 2022-07-06 928
> void *attr_value, size_t attr_size, int flags)
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 @929 {
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 930 int
> err;
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 931
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 932 err =
> mnt_want_write(path->mnt);
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 933 if
> (err)
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 934
> return err;
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 935
> 4609e1f18e19c3 fs/ksmbd/vfs.c Christian Brauner 2023-01-13 936 err =
> vfs_setxattr(idmap,
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 937
> path->dentry,
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 938
> attr_name,
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 939
> attr_value,
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 940
> attr_size,
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 941
> flags);
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 942 if
> (err)
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 943
> ksmbd_debug(VFS, "setxattr failed, err %d\n", err);
> 40b268d384a222 fs/smb/server/vfs.c Namjae Jeon 2023-06-15 944
> mnt_drop_write(path->mnt);
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 945
> return err;
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 946 }
> f44158485826c0 fs/cifsd/vfs.c Namjae Jeon 2021-03-16 947
>
> :::::: The code at line 929 was first introduced by commit
> :::::: f44158485826c076335d6860d35872271a83791d cifsd: add file operations
>
> :::::: TO: Namjae Jeon <namjae.jeon@xxxxxxxxxxx>
> :::::: CC: Steve French <stfrench@xxxxxxxxxxxxx>
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
>