iomap: pass writeback errors to the mapping
authorDarrick J. Wong <djwong@kernel.org>
Wed, 11 Aug 2021 01:32:55 +0000 (18:32 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 18 Sep 2021 11:40:29 +0000 (13:40 +0200)
[ Upstream commit b69eea82d37d9ee7cfb3bf05103549dd4ed5ffc3 ]

Modern-day mapping_set_error has the ability to squash the usual
negative error code into something appropriate for long-term storage in
a struct address_space -- ENOSPC becomes AS_ENOSPC, and everything else
becomes EIO.  iomap squashes /everything/ to EIO, just as XFS did before
that, but this doesn't make sense.

Fix this by making it so that we can pass ENOSPC to userspace when
writeback fails due to space problems.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
fs/iomap/buffered-io.c

index 10cc7979ce38070a7d6b16f3eca9d52804626e97..caed9d98c64aaf7fcd17a61a5c9dde34acfa7369 100644 (file)
@@ -1045,7 +1045,7 @@ iomap_finish_page_writeback(struct inode *inode, struct page *page,
 
        if (error) {
                SetPageError(page);
-               mapping_set_error(inode->i_mapping, -EIO);
+               mapping_set_error(inode->i_mapping, error);
        }
 
        WARN_ON_ONCE(i_blocks_per_page(inode, page) > 1 && !iop);