* [viro-vfs:work.fd 3/39] fs/open.c:1654:1: error: expected declaration or statement at end of input
@ 2024-08-10 14:17 kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-08-10 14:17 UTC (permalink / raw)
To: Al Viro; +Cc: oe-kbuild-all, linux-fsdevel, Christian Brauner
tree: https://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git work.fd
head: 9d58a36411c167b4126de90e5fe844270b858082
commit: f3270beef0d85432783be702bb9509879415e747 [3/39] struct fd: representation change
config: openrisc-allnoconfig (https://download.01.org/0day-ci/archive/20240810/202408102227.Zok7P1xp-lkp@intel.com/config)
compiler: or1k-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240810/202408102227.Zok7P1xp-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/202408102227.Zok7P1xp-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from fs/open.c:10:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/open.c: In function 'do_sys_ftruncate':
fs/open.c:196:25: error: expected ')' before 'return'
196 | if (!fd_file(f))
| ~ ^
| )
197 | return -EBADF;
| ~~~~~~
fs/open.c:196:9: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
196 | if (!fd_file(f))
| ^~
fs/open.c:196:9: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/open.c:1654:1: error: expected declaration or statement at end of input
1654 | EXPORT_SYMBOL(stream_open);
| ^~~~~~~~~~~~~
fs/open.c:191:13: warning: unused variable 'error' [-Wunused-variable]
191 | int error;
| ^~~~~
fs/open.c:1655: warning: control reaches end of non-void function [-Wreturn-type]
--
In file included from fs/read_write.c:12:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/read_write.c: In function 'ksys_lseek':
fs/read_write.c:297:25: error: expected ')' before 'return'
297 | if (!fd_file(f))
| ~ ^
| )
298 | return -EBADF;
| ~~~~~~
fs/read_write.c:297:9: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
297 | if (!fd_file(f))
| ^~
fs/read_write.c:297:9: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/read_write.c:1754:1: error: expected declaration or statement at end of input
1754 | }
| ^
fs/read_write.c:295:15: warning: unused variable 'retval' [-Wunused-variable]
295 | off_t retval;
| ^~~~~~
fs/read_write.c:1754:1: warning: no return statement in function returning non-void [-Wreturn-type]
1754 | }
| ^
fs/read_write.c: At top level:
fs/read_write.c:293:14: warning: 'ksys_lseek' defined but not used [-Wunused-function]
293 | static off_t ksys_lseek(unsigned int fd, off_t offset, unsigned int whence)
| ^~~~~~~~~~
--
In file included from include/linux/blkdev.h:27,
from fs/stat.c:8:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/stat.c: In function 'vfs_fstat':
fs/stat.c:227:25: error: expected ')' before 'return'
227 | if (!fd_file(f))
| ~ ^
| )
228 | return -EBADF;
| ~~~~~~
fs/stat.c:227:9: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
227 | if (!fd_file(f))
| ^~
fs/stat.c:227:9: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/stat.c:952:1: error: expected declaration or statement at end of input
952 | EXPORT_SYMBOL(inode_set_bytes);
| ^~~~~~~~~~~~~
fs/stat.c:224:13: warning: unused variable 'error' [-Wunused-variable]
224 | int error;
| ^~~~~
fs/stat.c:953: warning: control reaches end of non-void function [-Wreturn-type]
--
In file included from include/linux/kernel_read_file.h:5,
from include/linux/security.h:26,
from fs/namei.c:29:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/namei.c: In function 'path_init':
fs/namei.c:2495:33: error: expected ')' before 'return'
2495 | if (!fd_file(f))
| ~ ^
| )
2496 | return ERR_PTR(-EBADF);
| ~~~~~~
fs/namei.c:2495:17: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
2495 | if (!fd_file(f))
| ^~
fs/namei.c:2495:17: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/namei.c:5340:1: error: expected declaration or statement at end of input
5340 | EXPORT_SYMBOL(page_symlink_inode_operations);
| ^~~~~~~~~~~~~
fs/namei.c:2493:32: warning: unused variable 'dentry' [-Wunused-variable]
2493 | struct dentry *dentry;
| ^~~~~~
>> fs/namei.c:5340:1: error: expected declaration or statement at end of input
5340 | EXPORT_SYMBOL(page_symlink_inode_operations);
| ^~~~~~~~~~~~~
fs/namei.c: At top level:
fs/namei.c:2425:20: warning: 'path_init' defined but not used [-Wunused-function]
2425 | static const char *path_init(struct nameidata *nd, unsigned flags)
| ^~~~~~~~~
fs/namei.c:2329:12: warning: 'link_path_walk' defined but not used [-Wunused-function]
2329 | static int link_path_walk(const char *name, struct nameidata *nd)
| ^~~~~~~~~~~~~~
fs/namei.c:1243:12: warning: 'may_create_in_sticky' defined but not used [-Wunused-function]
1243 | static int may_create_in_sticky(struct mnt_idmap *idmap, struct nameidata *nd,
| ^~~~~~~~~~~~~~~~~~~~
fs/namei.c:883:12: warning: 'complete_walk' defined but not used [-Wunused-function]
883 | static int complete_walk(struct nameidata *nd)
| ^~~~~~~~~~~~~
fs/namei.c:687:13: warning: 'terminate_walk' defined but not used [-Wunused-function]
687 | static void terminate_walk(struct nameidata *nd)
| ^~~~~~~~~~~~~~
fs/namei.c:627:13: warning: 'restore_nameidata' defined but not used [-Wunused-function]
627 | static void restore_nameidata(void)
| ^~~~~~~~~~~~~~~~~
--
In file included from include/linux/kernel_read_file.h:5,
from include/linux/security.h:26,
from include/linux/perf_event.h:62,
from include/linux/trace_events.h:10,
from include/trace/syscall.h:7,
from include/linux/syscalls.h:93,
from fs/fcntl.c:8:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/fcntl.c: In function 'f_dupfd_query':
fs/fcntl.c:343:34: error: expected ')' before ';' token
343 | return fd_file(f) == filp;
| ^
| )
include/linux/file.h:49:20: note: to match this '('
49 | #define fd_file(f) ((struct file *)((f).word & ~(FDPUT_FPUT|FDPUT_POS_UNLOCK))
| ^
fs/fcntl.c:343:16: note: in expansion of macro 'fd_file'
343 | return fd_file(f) == filp;
| ^~~~~~~
fs/fcntl.c:343:35: error: expected ';' before '}' token
343 | return fd_file(f) == filp;
| ^
| ;
344 | }
| ~
fs/fcntl.c: In function '__do_sys_fcntl':
fs/fcntl.c:482:25: error: expected ')' before 'goto'
482 | if (!fd_file(f))
| ~ ^
| )
483 | goto out;
| ~~~~
fs/fcntl.c:482:9: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
482 | if (!fd_file(f))
| ^~
fs/fcntl.c:482:9: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/fcntl.c:1073:1: error: expected declaration or statement at end of input
1073 | module_init(fcntl_init)
| ^~~~~~~~~~~
fs/fcntl.c:480:14: warning: unused variable 'err' [-Wunused-variable]
480 | long err = -EBADF;
| ^~~
fs/fcntl.c:1073:1: warning: no return statement in function returning non-void [-Wreturn-type]
1073 | module_init(fcntl_init)
| ^~~~~~~~~~~
fs/fcntl.c: At top level:
fs/fcntl.c:463:12: warning: 'check_fcntl_cmd' defined but not used [-Wunused-function]
463 | static int check_fcntl_cmd(unsigned cmd)
| ^~~~~~~~~~~~~~~
fs/fcntl.c:346:13: warning: 'do_fcntl' defined but not used [-Wunused-function]
346 | static long do_fcntl(int fd, unsigned int cmd, unsigned long arg,
| ^~~~~~~~
--
In file included from include/linux/kernel_read_file.h:5,
from include/linux/security.h:26,
from include/linux/perf_event.h:62,
from include/linux/trace_events.h:10,
from include/trace/syscall.h:7,
from include/linux/syscalls.h:93,
from fs/ioctl.c:8:
include/linux/file.h: In function 'fdput':
include/linux/file.h:60:34: error: expected ')' before ';' token
60 | fput(fd_file(fd));
| ~ ^
| )
include/linux/file.h:60:35: error: expected ';' before '}' token
60 | fput(fd_file(fd));
| ^
| ;
61 | }
| ~
include/linux/file.h: In function 'fdput_pos':
include/linux/file.h:94:43: error: expected ')' before ';' token
94 | __f_unlock_pos(fd_file(f));
| ~ ^
| )
include/linux/file.h:95:18: error: expected ';' before '}' token
95 | fdput(f);
| ^
| ;
96 | }
| ~
fs/ioctl.c: In function 'ioctl_file_clone':
fs/ioctl.c:238:32: error: expected ')' before 'return'
238 | if (!fd_file(src_file))
| ~ ^
| )
239 | return -EBADF;
| ~~~~~~
fs/ioctl.c:238:9: note: '-Wmisleading-indentation' is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
238 | if (!fd_file(src_file))
| ^~
fs/ioctl.c:238:9: note: adding '-flarge-source-files' will allow for more column-tracking support, at the expense of compilation time and memory
>> fs/ioctl.c:912:1: error: expected declaration or statement at end of input
912 | }
| ^
fs/ioctl.c:236:13: warning: unused variable 'ret' [-Wunused-variable]
236 | int ret;
| ^~~
fs/ioctl.c:235:16: warning: unused variable 'cloned' [-Wunused-variable]
235 | loff_t cloned;
| ^~~~~~
fs/ioctl.c:912:1: warning: no return statement in function returning non-void [-Wreturn-type]
912 | }
| ^
fs/ioctl.c: At top level:
fs/ioctl.c:231:13: warning: 'ioctl_file_clone' defined but not used [-Wunused-function]
231 | static long ioctl_file_clone(struct file *dst_file, unsigned long srcfd,
| ^~~~~~~~~~~~~~~~
fs/ioctl.c:200:12: warning: 'ioctl_fiemap' defined but not used [-Wunused-function]
200 | static int ioctl_fiemap(struct file *filp, struct fiemap __user *ufiemap)
| ^~~~~~~~~~~~
fs/ioctl.c:59:12: warning: 'ioctl_fibmap' defined but not used [-Wunused-function]
59 | static int ioctl_fibmap(struct file *filp, int __user *p)
| ^~~~~~~~~~~~
..
vim +1654 fs/open.c
10dce8af34226d Kirill Smelkov 2019-03-26 1653
10dce8af34226d Kirill Smelkov 2019-03-26 @1654 EXPORT_SYMBOL(stream_open);
:::::: The code at line 1654 was first introduced by commit
:::::: 10dce8af34226d90fa56746a934f8da5dcdba3df fs: stream_open - opener for stream-like files so that read and write can run simultaneously without deadlock
:::::: TO: Kirill Smelkov <kirr@nexedi.com>
:::::: CC: Linus Torvalds <torvalds@linux-foundation.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2024-08-10 14:18 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-10 14:17 [viro-vfs:work.fd 3/39] fs/open.c:1654:1: error: expected declaration or statement at end of input kernel test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).