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 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 :::::: CC: Linus Torvalds --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation