btrfs: test direct IO write on NOCOW file when low on unallocated space
authorFilipe Manana <>
Thu, 28 Oct 2021 15:04:33 +0000 (16:04 +0100)
committerEryu Guan <>
Sun, 31 Oct 2021 14:32:52 +0000 (22:32 +0800)
Test that if we write to a range of a NOCOW file that has allocated
extents and there is not enough available free space for allocating new
data extents, the write succeeds. Test for direct IO and buffered IO

This currently fails on btrfs for the direct IO write scenario, only, but
while at it also test a buffered IO write, to help prevent regressions in
the future.

The patch that fixes the direct IO case has the following subject:

 "btrfs: fix ENOSPC failure when attempting direct IO write into NOCOW range"

Signed-off-by: Filipe Manana <>
Reviewed-by: Eryu Guan <>
Signed-off-by: Eryu Guan <>
tests/btrfs/250 [new file with mode: 0755]
tests/btrfs/250.out [new file with mode: 0644]

diff --git a/tests/btrfs/250 b/tests/btrfs/250
new file mode 100755 (executable)
index 0000000..e7ddc2f
--- /dev/null
@@ -0,0 +1,76 @@
+#! /bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (C) 2021 SUSE Linux Products GmbH. All Rights Reserved.
+# FS QA Test No. 250
+# Test that if we write to a range of a NOCOW file that has allocated extents
+# and there is not enough available free space for allocating new data extents,
+# the write succeeds. Test for direct IO and buffered IO writes.
+# The patch that fixes the direct IO case has the following subject:
+# "btrfs: fix ENOSPC failure when attempting direct IO write into NOCOW range"
+. ./common/preamble
+_begin_fstest auto quick enospc
+       cd /
+       rm -r -f $tmp.*
+# Import common functions.
+. ./common/filter
+# real QA test starts here
+_supported_fs btrfs
+_require_chattr C
+# Use a small fixed size filesystem so that it's quick to fill it up.
+# Make sure the fs size is > 256M, so that the mixed block groups feature is
+# not enabled by _scatch_mkfs_sized(), because we later want to not have more
+# space available for allocating data extents but still have enough metadata
+# space free for the file writes.
+fs_size=$((1024 * 1024 * 1024)) # 1G
+_scratch_mkfs_sized $fs_size >>$seqres.full 2>&1
+# Create our test file with the NOCOW attribute set.
+touch $SCRATCH_MNT/foobar
+# Now fill in all unallocated space with data for our test file.
+# This will allocate a data block group that will be full and leave no (or a
+# very small amount of) unallocated space in the device, so that it will not be
+# possible to allocate a new block group later.
+echo "Creating test file with initial data..."
+$XFS_IO_PROG -c "pwrite -S 0xab -b 1M 0 900M" $SCRATCH_MNT/foobar | _filter_xfs_io
+# Now try a direct IO write against file range [0, 10M[.
+# This should succeed since this is a NOCOW file and an extent for the range was
+# previously allocated.
+echo "Trying direct IO write over allocated space..."
+$XFS_IO_PROG -d -c "pwrite -S 0xcd -b 10M 0 10M" $SCRATCH_MNT/foobar | _filter_xfs_io
+# Now try a buffered IO write against file range [10M, 20M[.
+# This should also succeed since this is a NOCOW file and an extent for the range
+# was previously allocated.
+echo "Trying buffered IO write over allocated space..."
+$XFS_IO_PROG -c "pwrite -S 0xef -b 10M 10M 10M" $SCRATCH_MNT/foobar | _filter_xfs_io
+# Unmount and mount again the filesystem to clear any data from our file from the
+# page cache.
+# Now read the file and verify that all the writes we did before were durably
+# persisted.
+echo "File data after mounting again the filesystem:"
+od -A d -t x1 $SCRATCH_MNT/foobar
+# success, all done
diff --git a/tests/btrfs/250.out b/tests/btrfs/250.out
new file mode 100644 (file)
index 0000000..3912b70
--- /dev/null
@@ -0,0 +1,18 @@
+QA output created by 250
+Creating test file with initial data...
+wrote 943718400/943718400 bytes at offset 0
+XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+Trying direct IO write over allocated space...
+wrote 10485760/10485760 bytes at offset 0
+XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+Trying buffered IO write over allocated space...
+wrote 10485760/10485760 bytes at offset 10485760
+XXX Bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
+File data after mounting again the filesystem:
+0000000 cd cd cd cd cd cd cd cd cd cd cd cd cd cd cd cd
+10485760 ef ef ef ef ef ef ef ef ef ef ef ef ef ef ef ef
+20971520 ab ab ab ab ab ab ab ab ab ab ab ab ab ab ab ab