diff options
author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2024-04-20 03:50:06 +0100 |
---|---|---|
committer | Miklos Szeredi <mszeredi@redhat.com> | 2024-05-08 09:31:21 +0200 |
commit | 413e8f014c8b848e4ce939156f210df59fbd1c24 (patch) | |
tree | a7137dcf74182aa97196afee86debc01fc6290ce /fs/fuse/file.c | |
parent | 96d88f65adfbcaca153afd7d3e20d74ba379c599 (diff) |
fuse: Convert fuse_readpages_end() to use folio_end_read()
Nobody checks the error flag on fuse folios, so stop setting it.
Optimise the (optional) setting of the uptodate flag and clearing
of the lock flag by using folio_end_read().
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Diffstat (limited to 'fs/fuse/file.c')
-rw-r--r-- | fs/fuse/file.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index b57ce4157640..f39456c65ed7 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c @@ -935,14 +935,10 @@ static void fuse_readpages_end(struct fuse_mount *fm, struct fuse_args *args, } for (i = 0; i < ap->num_pages; i++) { - struct page *page = ap->pages[i]; + struct folio *folio = page_folio(ap->pages[i]); - if (!err) - SetPageUptodate(page); - else - SetPageError(page); - unlock_page(page); - put_page(page); + folio_end_read(folio, !err); + folio_put(folio); } if (ia->ff) fuse_file_put(ia->ff, false); |