NFS: Revalidate the file mapping on all fatal writeback errors
If a write or commit failed, and the mapping sees a fatal error, we
need to revalidate the contents of that mapping.
Fixes: 06c9fdf3b9
("NFS: On fatal writeback errors, we need to call nfs_inode_remove_request()")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
This commit is contained in:
parent
0df68ced55
commit
b8946d7bfb
@ -256,8 +256,11 @@ static void nfs_set_pageerror(struct address_space *mapping)
|
|||||||
|
|
||||||
static void nfs_mapping_set_error(struct page *page, int error)
|
static void nfs_mapping_set_error(struct page *page, int error)
|
||||||
{
|
{
|
||||||
|
struct address_space *mapping = page_file_mapping(page);
|
||||||
|
|
||||||
SetPageError(page);
|
SetPageError(page);
|
||||||
mapping_set_error(page_file_mapping(page), error);
|
mapping_set_error(mapping, error);
|
||||||
|
nfs_set_pageerror(mapping);
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
@ -600,7 +603,6 @@ release_request:
|
|||||||
|
|
||||||
static void nfs_write_error(struct nfs_page *req, int error)
|
static void nfs_write_error(struct nfs_page *req, int error)
|
||||||
{
|
{
|
||||||
nfs_set_pageerror(page_file_mapping(req->wb_page));
|
|
||||||
trace_nfs_write_error(req, error);
|
trace_nfs_write_error(req, error);
|
||||||
nfs_mapping_set_error(req->wb_page, error);
|
nfs_mapping_set_error(req->wb_page, error);
|
||||||
nfs_inode_remove_request(req);
|
nfs_inode_remove_request(req);
|
||||||
@ -1007,7 +1009,6 @@ static void nfs_write_completion(struct nfs_pgio_header *hdr)
|
|||||||
nfs_list_remove_request(req);
|
nfs_list_remove_request(req);
|
||||||
if (test_bit(NFS_IOHDR_ERROR, &hdr->flags) &&
|
if (test_bit(NFS_IOHDR_ERROR, &hdr->flags) &&
|
||||||
(hdr->good_bytes < bytes)) {
|
(hdr->good_bytes < bytes)) {
|
||||||
nfs_set_pageerror(page_file_mapping(req->wb_page));
|
|
||||||
trace_nfs_comp_error(req, hdr->error);
|
trace_nfs_comp_error(req, hdr->error);
|
||||||
nfs_mapping_set_error(req->wb_page, hdr->error);
|
nfs_mapping_set_error(req->wb_page, hdr->error);
|
||||||
goto remove_req;
|
goto remove_req;
|
||||||
|
Loading…
Reference in New Issue
Block a user