Re: [PATCH 2/2] debugfs: remove modular code

From: kbuild test robot
Date: Sun Nov 24 2019 - 16:34:49 EST

Next message: Steven Rostedt: "Re: [BUGFIX PATCH 1/3] selftests/ftrace: Fix to check the existence of set_ftrace_filter"
Previous message: Dan Williams: "[PATCH v3 3/3] libnvdimm, MAINTAINERS: Maintainer Entry Profile"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Masahiro,

I love your patch! Yet something to improve:

[auto build test ERROR on driver-core/driver-core-testing]
[also build test ERROR on v5.4-rc8 next-20191122]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url: https://github.com/0day-ci/linux/commits/Masahiro-Yamada/drivers-component-remove-modular-code/20191125-040836
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git 0e4a459f56c32d3e52ae69a4b447db2f48a65f44
config: parisc-c3000_defconfig (attached as .config)
compiler: hppa-linux-gcc (GCC) 7.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=7.4.0 make.cross ARCH=parisc

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

In file included from fs//debugfs/file.c:12:0:
fs//debugfs/file.c: In function 'open_proxy_open':
>> include/linux/fs.h:2286:14: error: implicit declaration of function 'try_module_get'; did you mean '__node_set'? [-Werror=implicit-function-declaration]
(((fops) && try_module_get((fops)->owner) ? (fops) : NULL))
^
>> fs//debugfs/file.c:174:14: note: in expansion of macro 'fops_get'
real_fops = fops_get(real_fops);
^~~~~~~~
>> include/linux/fs.h:2288:17: error: implicit declaration of function 'module_put'; did you mean 'mangle_path'? [-Werror=implicit-function-declaration]
do { if (fops) module_put((fops)->owner); } while(0)
^
>> include/linux/fs.h:2297:3: note: in expansion of macro 'fops_put'
fops_put(__file->f_op); \
^~~~~~~~
>> fs//debugfs/file.c:182:2: note: in expansion of macro 'replace_fops'
replace_fops(filp, real_fops);
^~~~~~~~~~~~
cc1: some warnings being treated as errors
--
In file included from fs/debugfs/file.c:12:0:
fs/debugfs/file.c: In function 'open_proxy_open':
>> include/linux/fs.h:2286:14: error: implicit declaration of function 'try_module_get'; did you mean '__node_set'? [-Werror=implicit-function-declaration]
(((fops) && try_module_get((fops)->owner) ? (fops) : NULL))
^
fs/debugfs/file.c:174:14: note: in expansion of macro 'fops_get'
real_fops = fops_get(real_fops);
^~~~~~~~
>> include/linux/fs.h:2288:17: error: implicit declaration of function 'module_put'; did you mean 'mangle_path'? [-Werror=implicit-function-declaration]
do { if (fops) module_put((fops)->owner); } while(0)
^
>> include/linux/fs.h:2297:3: note: in expansion of macro 'fops_put'
fops_put(__file->f_op); \
^~~~~~~~
fs/debugfs/file.c:182:2: note: in expansion of macro 'replace_fops'
replace_fops(filp, real_fops);
^~~~~~~~~~~~
cc1: some warnings being treated as errors

vim +2286 include/linux/fs.h

7f78e035139405 Eric W. Biederman 2013-03-02 2239
f175f307dd0bd1 Al Viro 2017-10-15 2240 #ifdef CONFIG_BLOCK
152a0836667108 Al Viro 2010-07-25 2241 extern struct dentry *mount_bdev(struct file_system_type *fs_type,
152a0836667108 Al Viro 2010-07-25 2242 int flags, const char *dev_name, void *data,
152a0836667108 Al Viro 2010-07-25 2243 int (*fill_super)(struct super_block *, void *, int));
f175f307dd0bd1 Al Viro 2017-10-15 2244 #else
f175f307dd0bd1 Al Viro 2017-10-15 2245 static inline struct dentry *mount_bdev(struct file_system_type *fs_type,
f175f307dd0bd1 Al Viro 2017-10-15 2246 int flags, const char *dev_name, void *data,
f175f307dd0bd1 Al Viro 2017-10-15 2247 int (*fill_super)(struct super_block *, void *, int))
f175f307dd0bd1 Al Viro 2017-10-15 2248 {
f175f307dd0bd1 Al Viro 2017-10-15 2249 return ERR_PTR(-ENODEV);
f175f307dd0bd1 Al Viro 2017-10-15 2250 }
f175f307dd0bd1 Al Viro 2017-10-15 2251 #endif
fc14f2fef682df Al Viro 2010-07-25 2252 extern struct dentry *mount_single(struct file_system_type *fs_type,
fc14f2fef682df Al Viro 2010-07-25 2253 int flags, void *data,
fc14f2fef682df Al Viro 2010-07-25 2254 int (*fill_super)(struct super_block *, void *, int));
3c26ff6e499ee7 Al Viro 2010-07-25 2255 extern struct dentry *mount_nodev(struct file_system_type *fs_type,
3c26ff6e499ee7 Al Viro 2010-07-25 2256 int flags, void *data,
3c26ff6e499ee7 Al Viro 2010-07-25 2257 int (*fill_super)(struct super_block *, void *, int));
ea441d1104cf1e Al Viro 2011-11-16 2258 extern struct dentry *mount_subtree(struct vfsmount *mnt, const char *path);
^1da177e4c3f41 Linus Torvalds 2005-04-16 2259 void generic_shutdown_super(struct super_block *sb);
f175f307dd0bd1 Al Viro 2017-10-15 2260 #ifdef CONFIG_BLOCK
^1da177e4c3f41 Linus Torvalds 2005-04-16 2261 void kill_block_super(struct super_block *sb);
f175f307dd0bd1 Al Viro 2017-10-15 2262 #else
f175f307dd0bd1 Al Viro 2017-10-15 2263 static inline void kill_block_super(struct super_block *sb)
f175f307dd0bd1 Al Viro 2017-10-15 2264 {
f175f307dd0bd1 Al Viro 2017-10-15 2265 BUG();
f175f307dd0bd1 Al Viro 2017-10-15 2266 }
f175f307dd0bd1 Al Viro 2017-10-15 2267 #endif
^1da177e4c3f41 Linus Torvalds 2005-04-16 2268 void kill_anon_super(struct super_block *sb);
^1da177e4c3f41 Linus Torvalds 2005-04-16 2269 void kill_litter_super(struct super_block *sb);
^1da177e4c3f41 Linus Torvalds 2005-04-16 2270 void deactivate_super(struct super_block *sb);
74dbbdd7fdc117 Al Viro 2009-05-06 2271 void deactivate_locked_super(struct super_block *sb);
^1da177e4c3f41 Linus Torvalds 2005-04-16 2272 int set_anon_super(struct super_block *s, void *data);
cb50b348c71ffa Al Viro 2018-12-23 2273 int set_anon_super_fc(struct super_block *s, struct fs_context *fc);
0ee5dc676a5f8f Al Viro 2011-07-07 2274 int get_anon_bdev(dev_t *);
0ee5dc676a5f8f Al Viro 2011-07-07 2275 void free_anon_bdev(dev_t);
cb50b348c71ffa Al Viro 2018-12-23 2276 struct super_block *sget_fc(struct fs_context *fc,
cb50b348c71ffa Al Viro 2018-12-23 2277 int (*test)(struct super_block *, struct fs_context *),
cb50b348c71ffa Al Viro 2018-12-23 2278 int (*set)(struct super_block *, struct fs_context *));
^1da177e4c3f41 Linus Torvalds 2005-04-16 2279 struct super_block *sget(struct file_system_type *type,
^1da177e4c3f41 Linus Torvalds 2005-04-16 2280 int (*test)(struct super_block *,void *),
^1da177e4c3f41 Linus Torvalds 2005-04-16 2281 int (*set)(struct super_block *,void *),
9249e17fe094d8 David Howells 2012-06-25 2282 int flags, void *data);
bba0bd31b117cb Andreas Gruenbacher 2016-09-29 2283
^1da177e4c3f41 Linus Torvalds 2005-04-16 2284 /* Alas, no aliases. Too much hassle with bringing module.h everywhere */
^1da177e4c3f41 Linus Torvalds 2005-04-16 2285 #define fops_get(fops) \
^1da177e4c3f41 Linus Torvalds 2005-04-16 @2286 (((fops) && try_module_get((fops)->owner) ? (fops) : NULL))
^1da177e4c3f41 Linus Torvalds 2005-04-16 2287 #define fops_put(fops) \
^1da177e4c3f41 Linus Torvalds 2005-04-16 @2288 do { if (fops) module_put((fops)->owner); } while(0)
e84f9e57b90ca8 Al Viro 2013-09-22 2289 /*
e84f9e57b90ca8 Al Viro 2013-09-22 2290 * This one is to be used *ONLY* from ->open() instances.
e84f9e57b90ca8 Al Viro 2013-09-22 2291 * fops must be non-NULL, pinned down *and* module dependencies
e84f9e57b90ca8 Al Viro 2013-09-22 2292 * should be sufficient to pin the caller down as well.
e84f9e57b90ca8 Al Viro 2013-09-22 2293 */
e84f9e57b90ca8 Al Viro 2013-09-22 2294 #define replace_fops(f, fops) \
e84f9e57b90ca8 Al Viro 2013-09-22 2295 do { \
e84f9e57b90ca8 Al Viro 2013-09-22 2296 struct file *__file = (f); \
e84f9e57b90ca8 Al Viro 2013-09-22 @2297 fops_put(__file->f_op); \
e84f9e57b90ca8 Al Viro 2013-09-22 2298 BUG_ON(!(__file->f_op = (fops))); \
e84f9e57b90ca8 Al Viro 2013-09-22 2299 } while(0)
^1da177e4c3f41 Linus Torvalds 2005-04-16 2300

:::::: The code at line 2286 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2

:::::: TO: Linus Torvalds <torvalds@xxxxxxxxxxxxxxx>
:::::: CC: Linus Torvalds <torvalds@xxxxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx Intel Corporation

Attachment: .config.gz
Description: application/gzip

Next message: Steven Rostedt: "Re: [BUGFIX PATCH 1/3] selftests/ftrace: Fix to check the existence of set_ftrace_filter"
Previous message: Dan Williams: "[PATCH v3 3/3] libnvdimm, MAINTAINERS: Maintainer Entry Profile"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]