aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Burkov <[email protected]>2023-03-28 14:19:56 +0900
committerDavid Sterba <[email protected]>2023-04-17 18:01:21 +0200
commitf0f5329a00ba5e9aec012212e3d9d8c2ff8f5cfa (patch)
tree0dff27bebb01d4b816834ba0c77adad2370bc721
parent7edd339c8a416afed9d58b5d20778d5ee49e079f (diff)
btrfs: don't split NOCOW extent_maps in btrfs_extract_ordered_extent
NOCOW writes just overwrite an existing extent map, which thus should not be split in btrfs_extract_ordered_extent. The NOCOW case can't currently happen as btrfs_extract_ordered_extent is only used on zoned devices that do not support NOCOW writes, but this will change soon. Reviewed-by: Josef Bacik <[email protected]> Tested-by: Johannes Thumshirn <[email protected]> Signed-off-by: Boris Burkov <[email protected]> [ hch: split from a larger patch, wrote a commit log ] Signed-off-by: Christoph Hellwig <[email protected]> Signed-off-by: David Sterba <[email protected]>
-rw-r--r--fs/btrfs/inode.c8
1 files changed, 8 insertions, 0 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 357d1626df1a..26aeb4f000f5 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -2621,6 +2621,14 @@ int btrfs_extract_ordered_extent(struct btrfs_bio *bbio,
ret = btrfs_split_ordered_extent(ordered, len);
if (ret)
return ret;
+
+ /*
+ * Don't split the extent_map for NOCOW extents, as we're writing into
+ * a pre-existing one.
+ */
+ if (test_bit(BTRFS_ORDERED_NOCOW, &ordered->flags))
+ return 0;
+
return split_extent_map(inode, bbio->file_offset, ordered_len, len);
}