diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2024-05-30 21:21:01 +0100 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-05-31 12:31:41 +0200 |
| commit | 8f3ab6e4bebe789f23690cf57fb1a648c4422b9d (patch) | |
| tree | 4789710384ed481b1aa7882201c913e83985c3e1 /fs/nfs/read.c | |
| parent | d15b00b1e588ea88a747bd70f6e360870088f5d2 (diff) | |
| download | linux-8f3ab6e4bebe789f23690cf57fb1a648c4422b9d.tar.gz linux-8f3ab6e4bebe789f23690cf57fb1a648c4422b9d.tar.bz2 linux-8f3ab6e4bebe789f23690cf57fb1a648c4422b9d.zip | |
nfs: Remove calls to folio_set_error
Common code doesn't test the error flag, so we don't need to set it in
nfs. We can use folio_end_read() to combine the setting (or not)
of the uptodate flag and clearing the lock flag.
Cc: Trond Myklebust <trond.myklebust@hammerspace.com>
Cc: Anna Schumaker <anna@kernel.org>
Cc: linux-nfs@vger.kernel.org
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Link: https://lore.kernel.org/r/20240530202110.2653630-10-willy@infradead.org
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/nfs/read.c')
| -rw-r--r-- | fs/nfs/read.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/nfs/read.c b/fs/nfs/read.c index a142287d86f6..cca80b5f54e0 100644 --- a/fs/nfs/read.c +++ b/fs/nfs/read.c @@ -122,8 +122,6 @@ static void nfs_readpage_release(struct nfs_page *req, int error) { struct folio *folio = nfs_page_to_folio(req); - if (nfs_error_is_fatal_on_server(error) && error != -ETIMEDOUT) - folio_set_error(folio); if (nfs_page_group_sync_on_bit(req, PG_UNLOCKPAGE)) if (nfs_netfs_folio_unlock(folio)) folio_unlock(folio); |