From: kernel test robot <lkp@intel.com>
To: Christian Brauner <brauner@kernel.org>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
Christoph Hellwig <hch@lst.de>
Subject: fs/nilfs2/ioctl.c:133: warning: Function parameter or member 'idmap' not described in 'nilfs_fileattr_set'
Date: Thu, 17 Aug 2023 15:44:05 +0800 [thread overview]
Message-ID: <202308171521.rNg7eaw1-lkp@intel.com> (raw)
Hi Christian,
FYI, the error/warning was bisected to this commit, please ignore it if it's irrelevant.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 4853c74bd7ab7fdb83f319bd9ace8a08c031e9b6
commit: 8782a9aea3ab4d697ad67d1f8ebca38a4e1c24ab fs: port ->fileattr_set() to pass mnt_idmap
date: 7 months ago
config: sparc-allyesconfig (https://download.01.org/0day-ci/archive/20230817/202308171521.rNg7eaw1-lkp@intel.com/config)
compiler: sparc64-linux-gcc (GCC) 12.3.0
reproduce: (https://download.01.org/0day-ci/archive/20230817/202308171521.rNg7eaw1-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202308171521.rNg7eaw1-lkp@intel.com/
All warnings (new ones prefixed by >>):
fs/nilfs2/ioctl.c:120: warning: Function parameter or member 'dentry' not described in 'nilfs_fileattr_get'
fs/nilfs2/ioctl.c:120: warning: Function parameter or member 'fa' not described in 'nilfs_fileattr_get'
>> fs/nilfs2/ioctl.c:133: warning: Function parameter or member 'idmap' not described in 'nilfs_fileattr_set'
fs/nilfs2/ioctl.c:133: warning: Function parameter or member 'dentry' not described in 'nilfs_fileattr_set'
fs/nilfs2/ioctl.c:133: warning: Function parameter or member 'fa' not described in 'nilfs_fileattr_set'
fs/nilfs2/ioctl.c:164: warning: Function parameter or member 'inode' not described in 'nilfs_ioctl_getversion'
fs/nilfs2/ioctl.c:164: warning: Function parameter or member 'argp' not described in 'nilfs_ioctl_getversion'
vim +133 fs/nilfs2/ioctl.c
cde98f0f84ccff Ryusuke Konishi 2011-01-20 127
d623a9420c9ae2 Vyacheslav Dubeyko 2014-01-23 128 /**
7c7c436e14b863 Miklos Szeredi 2021-04-07 129 * nilfs_fileattr_set - ioctl to support chattr
d623a9420c9ae2 Vyacheslav Dubeyko 2014-01-23 130 */
8782a9aea3ab4d Christian Brauner 2023-01-13 131 int nilfs_fileattr_set(struct mnt_idmap *idmap,
7c7c436e14b863 Miklos Szeredi 2021-04-07 132 struct dentry *dentry, struct fileattr *fa)
cde98f0f84ccff Ryusuke Konishi 2011-01-20 @133 {
7c7c436e14b863 Miklos Szeredi 2021-04-07 134 struct inode *inode = d_inode(dentry);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 135 struct nilfs_transaction_info ti;
cde98f0f84ccff Ryusuke Konishi 2011-01-20 136 unsigned int flags, oldflags;
cde98f0f84ccff Ryusuke Konishi 2011-01-20 137 int ret;
cde98f0f84ccff Ryusuke Konishi 2011-01-20 138
7c7c436e14b863 Miklos Szeredi 2021-04-07 139 if (fileattr_has_fsx(fa))
7c7c436e14b863 Miklos Szeredi 2021-04-07 140 return -EOPNOTSUPP;
cde98f0f84ccff Ryusuke Konishi 2011-01-20 141
7c7c436e14b863 Miklos Szeredi 2021-04-07 142 flags = nilfs_mask_flags(inode->i_mode, fa->flags);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 143
cde98f0f84ccff Ryusuke Konishi 2011-01-20 144 ret = nilfs_transaction_begin(inode->i_sb, &ti, 0);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 145 if (ret)
7c7c436e14b863 Miklos Szeredi 2021-04-07 146 return ret;
cde98f0f84ccff Ryusuke Konishi 2011-01-20 147
7c7c436e14b863 Miklos Szeredi 2021-04-07 148 oldflags = NILFS_I(inode)->i_flags & ~FS_FL_USER_MODIFIABLE;
7c7c436e14b863 Miklos Szeredi 2021-04-07 149 NILFS_I(inode)->i_flags = oldflags | (flags & FS_FL_USER_MODIFIABLE);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 150
cde98f0f84ccff Ryusuke Konishi 2011-01-20 151 nilfs_set_inode_flags(inode);
078cd8279e6599 Deepa Dinamani 2016-09-14 152 inode->i_ctime = current_time(inode);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 153 if (IS_SYNC(inode))
cde98f0f84ccff Ryusuke Konishi 2011-01-20 154 nilfs_set_transaction_flag(NILFS_TI_SYNC);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 155
cde98f0f84ccff Ryusuke Konishi 2011-01-20 156 nilfs_mark_inode_dirty(inode);
7c7c436e14b863 Miklos Szeredi 2021-04-07 157 return nilfs_transaction_commit(inode->i_sb);
cde98f0f84ccff Ryusuke Konishi 2011-01-20 158 }
cde98f0f84ccff Ryusuke Konishi 2011-01-20 159
:::::: The code at line 133 was first introduced by commit
:::::: cde98f0f84ccff78e87235cb7b551747d6ad00de nilfs2: implement FS_IOC_GETFLAGS/SETFLAGS/GETVERSION
:::::: TO: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
:::::: CC: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2023-08-17 7:45 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-17 7:44 kernel test robot [this message]
2023-10-17 15:21 kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202308171521.rNg7eaw1-lkp@intel.com \
--to=lkp@intel.com \
--cc=brauner@kernel.org \
--cc=hch@lst.de \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®