Commit 879db184 authored by Trond Myklebust's avatar Trond Myklebust Committed by Wang Zhaolong
Browse files

NFS: Use of mapping_set_error() results in spurious errors 

mainline inclusion
from mainline-v5.18-rc1
commit 6c984083
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IACBGS

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=6c984083ec2453dfd3fcf98f392f34500c73e3f2



--------------------------------

The use of mapping_set_error() in conjunction with calls to
filemap_check_errors() is problematic because every error gets reported
as either an EIO or an ENOSPC by filemap_check_errors() in functions
such as filemap_write_and_wait() or filemap_write_and_wait_range().
In almost all cases, we prefer to use the more nuanced wb errors.

Fixes: b8946d7b ("NFS: Revalidate the file mapping on all fatal writeback errors")
Signed-off-by: default avatarTrond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: default avatarWang Zhaolong <wangzhaolong1@huawei.com>
parent 7dce08b9

fs/nfs/write.c

+4 −1
Original line number Diff line number Diff line
@@ -311,7 +311,10 @@ static void nfs_mapping_set_error(struct page *page, int error) 
	struct address_space *mapping = page_file_mapping(page);



	SetPageError(page);

	mapping_set_error(mapping, error);

	filemap_set_wb_err(mapping, error);

	if (mapping->host)

		errseq_set(&mapping->host->i_sb->s_wb_err,

			   error == -ENOSPC ? -ENOSPC : -EIO);

	nfs_set_pageerror(mapping);

}