All of lore.kernel.org
 help / color / mirror / Atom feed
From: Qu Wenruo <wqu@suse.com>
To: fstests@vger.kernel.org, linux-btrfs@vger.kernel.org
Subject: [PATCH] fstests: btrfs: add a new test case for single data RAID56
Date: Sun, 24 May 2026 14:38:48 +0930	[thread overview]
Message-ID: <20260524050848.57740-1-wqu@suse.com> (raw)

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>
---
 tests/btrfs/348     | 68 +++++++++++++++++++++++++++++++++++++++++++++
 tests/btrfs/348.out |  2 ++
 2 files changed, 70 insertions(+)
 create mode 100755 tests/btrfs/348
 create mode 100644 tests/btrfs/348.out

diff --git a/tests/btrfs/348 b/tests/btrfs/348
new file mode 100755
index 00000000..748e5a86
--- /dev/null
+++ b/tests/btrfs/348
@@ -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
index 00000000..ec5e4701
--- /dev/null
+++ b/tests/btrfs/348.out
@@ -0,0 +1,2 @@
+QA output created by 348
+Silence is golden
-- 
2.51.2


             reply	other threads:[~2026-05-24  5:14 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-24  5:08 Qu Wenruo [this message]
2026-05-25  6:17 ` [PATCH] fstests: btrfs: add a new test case for single data RAID56 Christoph Hellwig
2026-05-29  8:04 ` Zorro Lang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260524050848.57740-1-wqu@suse.com \
    --to=wqu@suse.com \
    --cc=fstests@vger.kernel.org \
    --cc=linux-btrfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.