diff options
| author | Jiachen Zhang <zhangjiachen.jaycee@bytedance.com> | 2023-06-08 10:46:09 +0200 |
|---|---|---|
| committer | Miklos Szeredi <mszeredi@redhat.com> | 2024-03-06 09:56:54 +0100 |
| commit | 8a5fb186431326886ccc7b71d40aaf5e53b5d91a (patch) | |
| tree | 7c3d8c139a2f1e45eb010d78f13f699c896c94da /fs/fuse | |
| parent | fuse: Track process write operations in both direct and writethrough modes (diff) | |
| download | linux-8a5fb186431326886ccc7b71d40aaf5e53b5d91a.tar.xz linux-8a5fb186431326886ccc7b71d40aaf5e53b5d91a.zip | |
fuse: remove an unnecessary if statement
FUSE remote locking code paths never add any locking state to
inode->i_flctx, so the locks_remove_posix() function called on
file close will return without calling fuse_setlk().
Therefore, as the if statement to be removed in this commit will
always be false, remove it for clearness.
Signed-off-by: Jiachen Zhang <zhangjiachen.jaycee@bytedance.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Diffstat (limited to 'fs/fuse')
| -rw-r--r-- | fs/fuse/file.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index 761ae81ef41d..bd62be510a94 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c @@ -2678,10 +2678,6 @@ static int fuse_setlk(struct file *file, struct file_lock *fl, int flock) return -ENOLCK; } - /* Unlock on close is handled by the flush method */ - if ((fl->fl_flags & FL_CLOSE_POSIX) == FL_CLOSE_POSIX) - return 0; - fuse_lk_fill(&args, file, fl, opcode, pid_nr, flock, &inarg); err = fuse_simple_request(fm, &args); |