Simplify the way we check for a valid imap - we know we have a valid
mapping after xfs_map_blocks returned successfully, and we know we can
call xfs_imap_valid on any imap, as it will always fail on a
zero-initialized map.
Signed-off-by: Christoph Hellwig <hch@xxxxxx>
---
fs/xfs/xfs_aops.c | 11 ++---------
1 file changed, 2 insertions(+), 9 deletions(-)
diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c
index 5da2e99b0559..7ebd686cb723 100644
--- a/fs/xfs/xfs_aops.c
+++ b/fs/xfs/xfs_aops.c
@@ -42,7 +42,6 @@
*/
struct xfs_writepage_ctx {
struct xfs_bmbt_irec imap;
- bool imap_valid;
unsigned int io_type;
struct xfs_ioend *ioend;
sector_t last_block;
@@ -868,10 +867,6 @@ xfs_writepage_map(
continue;
}
- /* Check to see if current map spans this file offset */
- if (wpc->imap_valid)
- wpc->imap_valid = xfs_imap_valid(inode, &wpc->imap,
- file_offset);
/*
* If we don't have a valid map, now it's time to get a new one
* for this offset. This will convert delayed allocations
@@ -879,16 +874,14 @@ xfs_writepage_map(
* a valid map, it means we landed in a hole and we skip the
* block.
*/
- if (!wpc->imap_valid) {
+ if (!xfs_imap_valid(inode, &wpc->imap, file_offset)) {
error = xfs_map_blocks(inode, file_offset, &wpc->imap,
&wpc->io_type);
if (error)
goto out;
- wpc->imap_valid = xfs_imap_valid(inode, &wpc->imap,
- file_offset);
}
- if (!wpc->imap_valid || wpc->io_type == XFS_IO_HOLE) {
+ if (wpc->io_type == XFS_IO_HOLE) {
/*
* set_page_dirty dirties all buffers in a page, independent
* of their state. The dirty state however is entirely
--
2.17.0
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
