| Message ID | 20250312064541.664334-11-hch@lst.de (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [01/17] xfs/177: force a small file system size | expand |
On Wed, Mar 12, 2025 at 07:45:02AM +0100, Christoph Hellwig wrote: > Looking at the max_open_zones sysfs attribute to see if a file system is > zoned or not, as various tests depend on that. > > Signed-off-by: Christoph Hellwig <hch@lst.de> Looks fine to me, Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --D > --- > common/xfs | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/common/xfs b/common/xfs > index 807454d3e03b..86953b7310d9 100644 > --- a/common/xfs > +++ b/common/xfs > @@ -2076,6 +2076,33 @@ _scratch_xfs_force_no_metadir() > fi > } > > +# do not run on zoned file systems > +_require_xfs_scratch_non_zoned() > +{ > + if _has_fs_sysfs_attr $SCRATCH_DEV "zoned/max_open_zones"; then > + _notrun "Not supported on zoned file systems" > + fi > +} > + > +# only run on zoned file systems > +_require_xfs_scratch_zoned() > +{ > + local attr="zoned/max_open_zones" > + local min_open_zones=$1 > + > + if ! _has_fs_sysfs_attr $SCRATCH_DEV $attr; then > + _notrun "Requires zoned file system" > + fi > + > + if [ -n "${min_open_zones}" ]; then > + local has_open_zones=`_get_fs_sysfs_attr $SCRATCH_DEV $attr` > + > + if [ "${min_open_zones}" -gt "${has_open_zones}" ]; then > + _notrun "Requires at least ${min_open_zones} open zones" > + fi > + fi > +} > + > # Decide if a mount filesystem has metadata directory trees. > _xfs_mount_has_metadir() { > local mount="$1" > -- > 2.45.2 > >
diff --git a/common/xfs b/common/xfs index 807454d3e03b..86953b7310d9 100644 --- a/common/xfs +++ b/common/xfs @@ -2076,6 +2076,33 @@ _scratch_xfs_force_no_metadir() fi } +# do not run on zoned file systems +_require_xfs_scratch_non_zoned() +{ + if _has_fs_sysfs_attr $SCRATCH_DEV "zoned/max_open_zones"; then + _notrun "Not supported on zoned file systems" + fi +} + +# only run on zoned file systems +_require_xfs_scratch_zoned() +{ + local attr="zoned/max_open_zones" + local min_open_zones=$1 + + if ! _has_fs_sysfs_attr $SCRATCH_DEV $attr; then + _notrun "Requires zoned file system" + fi + + if [ -n "${min_open_zones}" ]; then + local has_open_zones=`_get_fs_sysfs_attr $SCRATCH_DEV $attr` + + if [ "${min_open_zones}" -gt "${has_open_zones}" ]; then + _notrun "Requires at least ${min_open_zones} open zones" + fi + fi +} + # Decide if a mount filesystem has metadata directory trees. _xfs_mount_has_metadir() { local mount="$1"
Looking at the max_open_zones sysfs attribute to see if a file system is zoned or not, as various tests depend on that. Signed-off-by: Christoph Hellwig <hch@lst.de> --- common/xfs | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+)