DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 044DB7F6B2
From: Jeff Layton <jlayton@redhat.com>
To: linux-fsdevel@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, tytso@mit.edu,
        jack@suse.cz, willy@infradead.org, neilb@suse.com,
        viro@zeniv.linux.org.uk
Subject: [PATCH v2 11/17] nilfs2: set the mapping error when calling SetPageError on writeback
Date: Wed, 12 Apr 2017 08:06:08 -0400
Message-Id: <20170412120614.6111-12-jlayton@redhat.com>
In-Reply-To: <20170412120614.6111-1-jlayton@redhat.com>
References: <20170412120614.6111-1-jlayton@redhat.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 907
Lines: 28

In a later patch, we're going to want to make the fsync codepath not do
a TestClearPageError call as that can override the error set in the
address space. To do that though, we need to ensure that filesystems
that are relying on the PG_error bit for reporting writeback errors
also set an error in the address space.

Ensure that this is set in nilfs2.

Cc: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
---
 fs/nilfs2/segment.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/fs/nilfs2/segment.c b/fs/nilfs2/segment.c
index febed1217b3f..612d4b446793 100644
--- a/fs/nilfs2/segment.c
+++ b/fs/nilfs2/segment.c
@@ -1745,6 +1745,7 @@ static void nilfs_end_page_io(struct page *page, int err)
 	} else {
 		__set_page_dirty_nobuffers(page);
 		SetPageError(page);
+		mapping_set_error(page_mapping(page), err);
 	}
 
 	end_page_writeback(page);
-- 
2.9.3