Commit fc187514 authored Oct 18, 2018 by Benjamin Coddington Committed by Trond Myklebust Oct 18, 2018

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>

parent fdbd1a2e

fs/nfs/pagelist.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -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;
		}