From: Eryu Guan Date: Wed, 23 Mar 2016 06:39:11 +0000 (+1100) Subject: xfs/073: avoid finobt warning from mkfs X-Git-Tag: v2022.05.01~2560 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=5af5146ef16c7996ee096b8f6190335c48f00658;p=xfstests-dev.git xfs/073: avoid finobt warning from mkfs When testing xfs/073 with MKFS_OPTIONS="-m crc=1,finobt=1" set, it fails due to extra warning about disabling finobt feature: +warning: finobt not supported without CRC support, disabled. Because xfs/073 disables crc unconditionally and finobt can not be enabled either. Fix it by explicitly disabling finobt as well. Also remove all meta related mkfs options in _scratch_mkfs_xfs_opts() if mkfs.xfs has no metadata support, not only the crc option. So that test still passes on distros with such old binaries. Suggested-by: Christoph Hellwig Signed-off-by: Eryu Guan Reviewed-by: Dave Chinner Signed-off-by: Dave Chinner --- diff --git a/common/rc b/common/rc index 8f925a8a..2ac159d9 100644 --- a/common/rc +++ b/common/rc @@ -444,9 +444,9 @@ _scratch_mkfs_xfs_opts() { mkfs_opts=$* - # remove crc related mkfs options if mkfs.xfs doesn't support v5 xfs + # remove metadata related mkfs options if mkfs.xfs doesn't them if [ -n "$XFS_MKFS_HAS_NO_META_SUPPORT" ]; then - mkfs_opts=`echo $mkfs_opts | sed "s/-m\s\+crc=.//"` + mkfs_opts=`echo $mkfs_opts | sed "s/-m\s\+\S\+//g"` fi _scratch_options mkfs diff --git a/tests/xfs/073 b/tests/xfs/073 index e1335df9..9e292234 100755 --- a/tests/xfs/073 +++ b/tests/xfs/073 @@ -138,7 +138,7 @@ _require_loop rm -f $seqres.full -_scratch_mkfs_xfs -m crc=0 -dsize=41m,agcount=2 | _filter_mkfs 2>/dev/null +_scratch_mkfs_xfs -m crc=0 -dsize=41m,agcount=2 >>$seqres.full 2>&1 _scratch_mount 2>/dev/null || _fail "initial scratch mount failed" echo diff --git a/tests/xfs/073.out b/tests/xfs/073.out index fb035cbe..5f36ebe8 100644 --- a/tests/xfs/073.out +++ b/tests/xfs/073.out @@ -1,13 +1,4 @@ QA output created by 073 -warning: finobt not supported without CRC support, disabled. -warning: rmapbt not supported without CRC support, disabled. -warning: reflink not supported without CRC support, disabled. -meta-data=DDEV isize=XXX agcount=N, agsize=XXX blks -data = bsize=XXX blocks=XXX, imaxpct=PCT - = sunit=XXX swidth=XXX, unwritten=X -naming =VERN bsize=XXX -log =LDEV bsize=XXX blocks=XXX -realtime =RDEV extsz=XXX blocks=XXX, rtextents=XXX === populating scratch device 1000+0 records in