diff options
| author |   Miklos Szeredi <mszeredi@redhat.com> | 2021-10-22 17:03:02 +0200 |
|---|---|---|
| committer | Miklos Szeredi <mszeredi@redhat.com> | 2021-10-22 17:03:02 +0200 |
| commit | 84840efc3c0f225ee5597e0a013e9da03afc73c6 (patch) | |
| tree | 79c45ff9ea1e88cdb647753539aa2568fbe492c3 /fs/fuse/file.c | |
| parent | fuse: move fuse_invalidate_attr() into fuse_update_ctime() (diff) | |
| download | linux-dev-84840efc3c0f225ee5597e0a013e9da03afc73c6.tar.xz linux-dev-84840efc3c0f225ee5597e0a013e9da03afc73c6.zip | |
fuse: simplify __fuse_write_file_get()
Use list_first_entry_or_null() instead of list_empty() + list_entry().
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Diffstat (limited to 'fs/fuse/file.c')
| -rw-r--r-- | fs/fuse/file.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index be13f695e1d6..31266ca9c1f2 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c @@ -1815,14 +1815,13 @@ static void fuse_writepage_end(struct fuse_mount *fm, struct fuse_args *args, static struct fuse_file *__fuse_write_file_get(struct fuse_inode *fi) { - struct fuse_file *ff = NULL; + struct fuse_file *ff; spin_lock(&fi->lock); - if (!list_empty(&fi->write_files)) { - ff = list_entry(fi->write_files.next, struct fuse_file, - write_entry); + ff = list_first_entry_or_null(&fi->write_files, struct fuse_file, + write_entry); + if (ff) fuse_file_get(ff); - } spin_unlock(&fi->lock); return ff; |