[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202203220253.CbyFvh3n-lkp@intel.com>
Date: Tue, 22 Mar 2022 02:18:46 +0800
From: kernel test robot <lkp@...el.com>
To: Amir Goldstein <amir73il@...il.com>
Cc: kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: [amir73il:fsnotify-volatile 3/10]
fs/notify/fanotify/fanotify_user.c:1147:8: error: too few arguments to
function 'fsnotify_add_mark_locked'
tree: https://github.com/amir73il/linux fsnotify-volatile
head: 46cbd4482bb2350a2d667d18272e2f0172c1633b
commit: 66f27fc99e46b12f1078e8e2915793040ce50ee7 [3/10] fsnotify: preallocate connector outside of group lock
config: x86_64-rhel-8.3-func (https://download.01.org/0day-ci/archive/20220322/202203220253.CbyFvh3n-lkp@intel.com/config)
compiler: gcc-9 (Ubuntu 9.4.0-1ubuntu1~20.04) 9.4.0
reproduce (this is a W=1 build):
# https://github.com/amir73il/linux/commit/66f27fc99e46b12f1078e8e2915793040ce50ee7
git remote add amir73il https://github.com/amir73il/linux
git fetch --no-tags amir73il fsnotify-volatile
git checkout 66f27fc99e46b12f1078e8e2915793040ce50ee7
# save the config file to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
Note: the amir73il/fsnotify-volatile HEAD 46cbd4482bb2350a2d667d18272e2f0172c1633b builds fine.
It only hurts bisectability.
All errors (new ones prefixed by >>):
fs/notify/fanotify/fanotify_user.c: In function 'fanotify_add_new_mark':
>> fs/notify/fanotify/fanotify_user.c:1147:8: error: too few arguments to function 'fsnotify_add_mark_locked'
1147 | ret = fsnotify_add_mark_locked(mark, connp, obj_type, 0, fsid);
| ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/notify/fanotify/fanotify_user.c:8:
include/linux/fsnotify_backend.h:642:12: note: declared here
642 | extern int fsnotify_add_mark_locked(struct fsnotify_mark *mark,
| ^~~~~~~~~~~~~~~~~~~~~~~~
vim +/fsnotify_add_mark_locked +1147 fs/notify/fanotify/fanotify_user.c
912ee3946c5e57 Andreas Gruenbacher 2009-12-17 1121
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1122 static struct fsnotify_mark *fanotify_add_new_mark(struct fsnotify_group *group,
b812a9f5896379 Amir Goldstein 2018-06-23 1123 fsnotify_connp_t *connp,
ad69cd9972e79a Amir Goldstein 2021-11-29 1124 unsigned int obj_type,
77115225acc67d Amir Goldstein 2019-01-10 1125 __kernel_fsid_t *fsid)
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1126 {
5b8fea65d197f4 Amir Goldstein 2021-03-04 1127 struct ucounts *ucounts = group->fanotify_data.ucounts;
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1128 struct fsnotify_mark *mark;
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1129 int ret;
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1130
5b8fea65d197f4 Amir Goldstein 2021-03-04 1131 /*
5b8fea65d197f4 Amir Goldstein 2021-03-04 1132 * Enforce per user marks limits per user in all containing user ns.
5b8fea65d197f4 Amir Goldstein 2021-03-04 1133 * A group with FAN_UNLIMITED_MARKS does not contribute to mark count
5b8fea65d197f4 Amir Goldstein 2021-03-04 1134 * in the limited groups account.
5b8fea65d197f4 Amir Goldstein 2021-03-04 1135 */
5b8fea65d197f4 Amir Goldstein 2021-03-04 1136 if (!FAN_GROUP_FLAG(group, FAN_UNLIMITED_MARKS) &&
5b8fea65d197f4 Amir Goldstein 2021-03-04 1137 !inc_ucount(ucounts->ns, ucounts->uid, UCOUNT_FANOTIFY_MARKS))
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1138 return ERR_PTR(-ENOSPC);
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1139
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1140 mark = kmem_cache_alloc(fanotify_mark_cache, GFP_KERNEL);
5b8fea65d197f4 Amir Goldstein 2021-03-04 1141 if (!mark) {
5b8fea65d197f4 Amir Goldstein 2021-03-04 1142 ret = -ENOMEM;
5b8fea65d197f4 Amir Goldstein 2021-03-04 1143 goto out_dec_ucounts;
5b8fea65d197f4 Amir Goldstein 2021-03-04 1144 }
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1145
054c636e5c8054 Jan Kara 2016-12-21 1146 fsnotify_init_mark(mark, group);
ad69cd9972e79a Amir Goldstein 2021-11-29 @1147 ret = fsnotify_add_mark_locked(mark, connp, obj_type, 0, fsid);
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1148 if (ret) {
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1149 fsnotify_put_mark(mark);
5b8fea65d197f4 Amir Goldstein 2021-03-04 1150 goto out_dec_ucounts;
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1151 }
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1152
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1153 return mark;
5b8fea65d197f4 Amir Goldstein 2021-03-04 1154
5b8fea65d197f4 Amir Goldstein 2021-03-04 1155 out_dec_ucounts:
5b8fea65d197f4 Amir Goldstein 2021-03-04 1156 if (!FAN_GROUP_FLAG(group, FAN_UNLIMITED_MARKS))
5b8fea65d197f4 Amir Goldstein 2021-03-04 1157 dec_ucount(ucounts, UCOUNT_FANOTIFY_MARKS);
5b8fea65d197f4 Amir Goldstein 2021-03-04 1158 return ERR_PTR(ret);
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1159 }
5e9c070ca08543 Lino Sanfilippo 2013-07-08 1160
:::::: The code at line 1147 was first introduced by commit
:::::: ad69cd9972e79aba103ba5365de0acd35770c265 fsnotify: clarify object type argument
:::::: TO: Amir Goldstein <amir73il@...il.com>
:::::: CC: Jan Kara <jack@...e.cz>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
Powered by blists - more mailing lists