]> git-server-git.apps.pok.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
fstests: btrfs: add a new test case for single data RAID56
authorQu Wenruo <wqu@suse.com>
Sun, 24 May 2026 05:08:48 +0000 (14:38 +0930)
committerZorro Lang <zlang@kernel.org>
Fri, 29 May 2026 08:54:30 +0000 (16:54 +0800)
Although single-data-RAID56, aka 2 disks raid5 or 3 disks raid6, is
not recommended for btrfs, btrfs still supports it as there are cases
like degraded writes can still lead to such cases.

But we do not have any proper tests for such corner cases.

Furthermore, RAID56 lib is going to drop the support of
single-data-RAID56, and btrfs will have its own handling of such cases,
thus coverage on single-data-RAID56 is required.

This new test case will have the following workload to verify the
behavior:

- Create a fs with single-data-RAID56

- Populate the fs with fsx and fsstress
  Fsx and fsstress will utilize read and write paths of RAID56.

- Mount each device in degraded mode and run read-only scrub on them
  Such scrub will utilize the following path of RAID56:
  * Regular data read
  * Reconstruction read
  * P/Q stripe scrub

- Re-scan all devices and add a new disk to the array
- Balance all data and scrub again
  This should utilized non-single-data-RAID56 handling.

Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: Zorro Lang <zlang@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Zorro Lang <zlang@kernel.org>
tests/btrfs/348 [new file with mode: 0755]
tests/btrfs/348.out [new file with mode: 0644]

diff --git a/tests/btrfs/348 b/tests/btrfs/348
new file mode 100755 (executable)
index 0000000..748e5a8
--- /dev/null
@@ -0,0 +1,68 @@
+#! /bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (c) 2026 SUSE S.A.  All Rights Reserved.
+#
+# FS QA Test 348
+#
+# Basic function test for single-data-RAID56.
+#
+. ./common/preamble
+_begin_fstest auto raid
+
+# 3 disks for single-data-raid6, but one extra to make sure btrfs can still properly
+# convert to regular 4 disks raid6.
+_require_scratch_dev_pool 4
+_require_btrfs_raid_type raid5
+_require_btrfs_raid_type raid6
+_require_btrfs_forget_or_module_loadable
+
+# Make sure we have support RAID1C34 first
+if [ ! -f "/sys/fs/btrfs/features/raid1c34" ]; then
+       _notrun "no RAID1C34 support"
+fi
+
+common_workload()
+{
+       _scratch_mount
+       _run_fsx_on_file $FSX_AVOID "$SCRATCH_MNT/foobar" -q -N 10000 >> $seqres.full
+       _run_fsstress -w -n 10000 -d "$SCRATCH_MNT"
+       _scratch_unmount
+
+       # Mount each device degraded and run a scrub to exercise:
+       # - Regular data read
+       # - Data reconstruction read
+       # - P/Q scrub
+       for dev in $SCRATCH_DEV_POOL; do
+               _btrfs_forget_or_module_reload
+
+               _mount -o ro,degraded "$dev" "$SCRATCH_MNT"
+               _btrfs scrub start -Br "$SCRATCH_MNT"
+               _scratch_unmount
+       done
+
+       # Add a new device, balance all data, and re-scrub to make sure
+       # regular RAID56 is still working.
+       _btrfs_rescan_devices
+       _spare_dev_get
+       _scratch_mount
+       _btrfs device add -f "$SPARE_DEV" "$SCRATCH_MNT"
+       _btrfs balance start -d "$SCRATCH_MNT"
+       _btrfs scrub start -B "$SCRATCH_MNT"
+       _scratch_unmount
+       _spare_dev_put
+}
+
+_scratch_dev_pool_get 2
+# There will be a warning about single-data-RAID56, thus need
+# to redirect stderr
+_scratch_pool_mkfs -m raid1 -d raid5 >> $seqres.full 2>&1
+common_workload
+_scratch_dev_pool_put
+
+_scratch_dev_pool_get 3
+_scratch_pool_mkfs -m raid1c3 -d raid6 >> $seqres.full 2>&1
+common_workload
+_scratch_dev_pool_put
+
+echo "Silence is golden"
+_exit 0
diff --git a/tests/btrfs/348.out b/tests/btrfs/348.out
new file mode 100644 (file)
index 0000000..ec5e470
--- /dev/null
@@ -0,0 +1,2 @@
+QA output created by 348
+Silence is golden