summaryrefslogtreecommitdiff
path: root/io_uring
diff options
context:
space:
mode:
authorAmir Goldstein <amir73il@gmail.com>2023-06-11 15:24:29 +0300
committerJan Kara <jack@suse.cz>2023-06-12 11:43:45 +0300
commit7b8c9d7bb4570ee4800642009c8f2d9756004552 (patch)
tree807f50d866296846244e9cc88196c6dbd7e72022 /io_uring
parent7cdafe6cc4a6ee94c56a5c96d6edd80d066d5a3b (diff)
downloadlinux-7b8c9d7bb4570ee4800642009c8f2d9756004552.tar.xz
fsnotify: move fsnotify_open() hook into do_dentry_open()
fsnotify_open() hook is called only from high level system calls context and not called for the very many helpers to open files. This may makes sense for many of the special file open cases, but it is inconsistent with fsnotify_close() hook that is called for every last fput() of on a file object with FMODE_OPENED. As a result, it is possible to observe ACCESS, MODIFY and CLOSE events without ever observing an OPEN event. Fix this inconsistency by replacing all the fsnotify_open() hooks with a single hook inside do_dentry_open(). If there are special cases that would like to opt-out of the possible overhead of fsnotify() call in fsnotify_open(), they would probably also want to avoid the overhead of fsnotify() call in the rest of the fsnotify hooks, so they should be opening that file with the __FMODE_NONOTIFY flag. However, in the majority of those cases, the s_fsnotify_connectors optimization in fsnotify_parent() would be sufficient to avoid the overhead of fsnotify() call anyway. Signed-off-by: Amir Goldstein <amir73il@gmail.com> Signed-off-by: Jan Kara <jack@suse.cz> Message-Id: <20230611122429.1499617-1-amir73il@gmail.com>
Diffstat (limited to 'io_uring')
-rw-r--r--io_uring/openclose.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/io_uring/openclose.c b/io_uring/openclose.c
index a1b98c81a52d..10ca57f5bd24 100644
--- a/io_uring/openclose.c
+++ b/io_uring/openclose.c
@@ -150,7 +150,6 @@ int io_openat2(struct io_kiocb *req, unsigned int issue_flags)
if ((issue_flags & IO_URING_F_NONBLOCK) && !nonblock_set)
file->f_flags &= ~O_NONBLOCK;
- fsnotify_open(file);
if (!fixed)
fd_install(ret, file);