diff options
| author |   Benjamin Coddington <bcodding@redhat.com> | 2018-10-18 15:01:49 -0400 |
|---|---|---|
| committer |   Trond Myklebust <trond.myklebust@hammerspace.com> | 2018-10-18 17:20:57 -0400 |
| commit | fc187514d8af3a676c7bd7922439f9f5e5c6223f (patch) | |
| tree | 29d62c2d0b603883a98c4d9ea897a5a91589c049 /fs | |
| parent | nfs: Fix a missed page unlock after pg_doio() (diff) | |
| download | linux-dev-fc187514d8af3a676c7bd7922439f9f5e5c6223f.tar.xz linux-dev-fc187514d8af3a676c7bd7922439f9f5e5c6223f.zip | |
nfs: remove redundant call to nfs_context_set_write_error()
We don't need to call this in the direct, read, or pnfs resend paths and
the only other caller is the write path in nfs_page_async_flush() which
already checks and sets the pg_error on the context.
Signed-off-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/nfs/pagelist.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/nfs/pagelist.c b/fs/nfs/pagelist.c index 3b50edf6f883..5c4568a0804b 100644 --- a/fs/nfs/pagelist.c +++ b/fs/nfs/pagelist.c @@ -1174,10 +1174,6 @@ int nfs_pageio_add_request(struct nfs_pageio_descriptor *desc, return 1; out_failed: - /* remember fatal errors */ - if (nfs_error_is_fatal(desc->pg_error)) - nfs_context_set_write_error(req->wb_context, - desc->pg_error); nfs_pageio_error_cleanup(desc); return 0; } |