在 2023/12/20 10:43, Darrick J. Wong 写道: > On Wed, Dec 20, 2023 at 09:59:04AM +0800, Wu Guanghao wrote: >> According to the man page description, su=value and sunit=value are both >> used to specify the unit for a RAID device/logical volume. And swidth and >> sw are both used to specify the stripe width. >> >> So in the prompt we need to associate su with sunit and sw with swidth. >> >> Signed-by-off: Wu Guanghao <wuguanghao3@xxxxxxxxxx> >> --- >> mkfs/xfs_mkfs.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c >> index dd3360dc..c667b904 100644 >> --- a/mkfs/xfs_mkfs.c >> +++ b/mkfs/xfs_mkfs.c >> @@ -993,7 +993,7 @@ usage( void ) >> /* metadata */ [-m crc=0|1,finobt=0|1,uuid=xxx,rmapbt=0|1,reflink=0|1,\n\ >> inobtcount=0|1,bigtime=0|1]\n\ >> /* data subvol */ [-d agcount=n,agsize=n,file,name=xxx,size=num,\n\ >> - (sunit=value,swidth=value|su=num,sw=num|noalign),\n\ >> + (sunit=value|su=num,swidth=value|sw=num,noalign),\n\ > > Doesn't mkfs require sunit/swidth or su/sw to be used together, but not > intermixed? > > --D > I think the '|' in usage() should be related to the modification of the same feature, so there is also 'sunit|su' in the -l parameter. There are already other prompts for mixed use of su/sw, and I don’t think there is a need to remind it in usage(). The current 'swidth=value|su=num' prompt makes me think that the two modifications are the same. There may be other people who think so too, so I suggest changing the description. >> sectsize=num\n\ >> /* force overwrite */ [-f]\n\ >> /* inode size */ [-i perblock=n|size=num,maxpct=n,attr=0|1|2,\n\ >> -- >> 2.27.0 >> > > . >
