From: Zorro Lang <zlang@kernel.org>
To: fstests@vger.kernel.org
Cc: linux-xfs@vger.kernel.org, linux-btrfs@vger.kernel.org,
Anand Suveer Jain <asj@kernel.org>, Chao Yu <chao@kernel.org>,
Christoph Hellwig <hch@infradead.org>
Subject: [PATCH v3 0/6] Refactor *_fs_sysfs_* helpers
Date: Thu, 5 Nov 2026 02:47:30 +0800 [thread overview]
Message-ID: <20261104184736.1564283-1-zlang@kernel.org> (raw)
As we talked in:
https://lore.kernel.org/fstests/amhKovVDjwawoWdQ@zlang-mailbox/
This patchset refactors two redundant and messy sets of *_fs_sysfs_* helpers
(PATCH 1/6 2/6), move btrfs specific sysfs policy helpers to common/btrfs
(PATCH 3/6), introduces helpers for /sys/fs/$FSTYP/features (PATCH 4/6),
and replaces the original btrfs-specific helpers with these common ones
(PATCH 6/6).
V2 did below changes:
1) According to review point:
https://lore.kernel.org/fstests/aqjsBs6QUtIukk05@infradead.org/
split original patch 1/5 to two patches 1/6 and 2/6.
2) According to review point:
https://lore.kernel.org/fstests/aqjsZZD4RPVK-uBn@infradead.org/
rename functions in common/sysfs and move them to common/btrfs.
V3 did below changes:
1) Address Christoph's review points on [PATCH 2/6] by introducing
_require_{test,scratch}_sysfs_dev_attr and wrapping lines that were too
long (Waiting for RVB).
2) Since an upcoming test case (btrfs/363) is also affected by [PATCH 6/6],
I have rebased patch 6/6 and the entire patchset onto the latest
patches-in-queue branch.
Thanks,
Zorro
Zorro Lang (6):
fstests: rename _*_fs_sysfs_attr helpers to _*_fs_sysfs_dev_attr
fstests: consolidate _require_fs_sysfs and _require_fs_sysfs_dev_attr
common: move btrfs-only sysfs policy helpers out of common/sysfs
common/rc: add helpers for /sys/fs/$FSTYP/features/
common/rc: use common helper in _require_fanotify_ioerrors for f2fs
fstests: replace btrfs specific helpers with common sysfs helpers
common/btrfs | 176 ++++++++++++++++++++++++++++++++++++----------
common/rc | 109 +++++++++++++++++++++-------
common/sysfs | 145 --------------------------------------
common/xfs | 24 ++++---
tests/btrfs/010 | 3 +-
tests/btrfs/055 | 2 +-
tests/btrfs/056 | 2 +-
tests/btrfs/112 | 2 +-
tests/btrfs/131 | 2 +-
tests/btrfs/165 | 2 +-
tests/btrfs/188 | 2 +-
tests/btrfs/192 | 2 +-
tests/btrfs/201 | 2 +-
tests/btrfs/205 | 2 +-
tests/btrfs/209 | 2 +-
tests/btrfs/211 | 2 +-
tests/btrfs/219 | 3 +-
tests/btrfs/231 | 2 +-
tests/btrfs/253 | 29 ++++----
tests/btrfs/285 | 3 +-
tests/btrfs/296 | 8 +--
tests/btrfs/304 | 4 +-
tests/btrfs/305 | 4 +-
tests/btrfs/306 | 4 +-
tests/btrfs/307 | 4 +-
tests/btrfs/308 | 4 +-
tests/btrfs/311 | 2 +-
tests/btrfs/313 | 2 +-
tests/btrfs/314 | 2 +-
tests/btrfs/315 | 2 +-
tests/btrfs/329 | 5 +-
tests/btrfs/334 | 5 +-
tests/btrfs/342 | 2 +-
tests/btrfs/352 | 2 +-
tests/btrfs/363 | 2 +-
tests/f2fs/025 | 2 +-
tests/generic/098 | 2 +-
tests/generic/101 | 2 +-
tests/xfs/006 | 4 +-
tests/xfs/177 | 4 +-
tests/xfs/264 | 14 ++--
tests/xfs/668 | 6 +-
tests/xfs/669 | 6 +-
tests/xfs/670 | 2 +-
44 files changed, 307 insertions(+), 303 deletions(-)
delete mode 100644 common/sysfs
--
2.56.0
next reply other threads:[~2026-11-04 18:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-04 18:47 Zorro Lang [this message]
2026-11-04 18:47 ` [PATCH v3 1/6] fstests: rename _*_fs_sysfs_attr helpers to _*_fs_sysfs_dev_attr Zorro Lang
2026-11-04 18:47 ` [PATCH v3 2/6] fstests: consolidate _require_fs_sysfs and _require_fs_sysfs_dev_attr Zorro Lang
2026-11-05 7:50 ` Christoph Hellwig
2026-11-04 18:47 ` [PATCH v3 3/6] common: move btrfs-only sysfs policy helpers out of common/sysfs Zorro Lang
2026-11-04 18:47 ` [PATCH v3 4/6] common/rc: add helpers for /sys/fs/$FSTYP/features/ Zorro Lang
2026-11-04 18:47 ` [PATCH v3 5/6] common/rc: use common helper in _require_fanotify_ioerrors for f2fs Zorro Lang
2026-11-04 18:47 ` [PATCH v3 6/6] fstests: replace btrfs specific helpers with common sysfs helpers 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=20261104184736.1564283-1-zlang@kernel.org \
--to=zlang@kernel.org \
--cc=asj@kernel.org \
--cc=chao@kernel.org \
--cc=fstests@vger.kernel.org \
--cc=hch@infradead.org \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-xfs@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.