Re: [PATCH 1/2] common/config: remove default 4k blocksize from XFS_MKFS_OPTIONS

Eryu Guan
2021-05-30T12:23:06Z

On Tue, May 25, 2021 at 09:05:07AM -0700, Darrick J. Wong wrote:
> On Tue, May 25, 2021 at 12:46:41PM +0800, Zorro Lang wrote:
> > xfstests set "-bsize=4k" to XFS_MKFS_OPTIONS by default, then give
> > it to MKFS_OPTIOPNS. So MKFS_OPTIOPNS always contains "-bsize=4k"
> > except we set XFS_MKFS_OPTIONS manually.
> >
> > It's useless to set XFS_MKFS_OPTIONS to "-bsize=4096" by default,
> > especially that will cause all cases with _scratch_mkfs_blocksized()
> > always fail as "-b size option respecified", when test on XFS. For
> > exmaple: generic/222
> >
> > Signed-off-by: Zorro Lang <zlang@redhat.com>
>
> /me wonders what the historical context is (a) for having an
> $FSTYP-specific MKFS_OPTIONS variable, and (b) for setting bsize=4096
> here. All I can say is that I've tripped over this numerous times and
> wouldn't really mind if it went away. :P
>
> Acked-by: Darrick J. Wong <djwong@kernel.org>
>
> (acked in the sense that I'm not /opposed/ but really really wants to
> know the historical reasons)

Looked at the git log, and it was added in 2004 without any
explaination. I'll let the patch pending for another week, to see if
anyone knows the history behind this.

commit cb6beb975950121f0569c5dede6136fb8ae0b8b8
Author: ptools <ptools>
Date: Tue Jun 15 07:32:36 2004 +0000

Fixed merge problems

Thanks,
Eryu