On Thu, Oct 09, 2014 at 08:16:57PM -0400, Wang Shilong wrote: > while running ./check tests/shared/032, it outputs something like: > > FSTYP -- btrfs > PLATFORM -- Linux/x86_64 vm01 3.17.0-rc5+ > MKFS_OPTIONS -- /dev/sdc > MOUNT_OPTIONS -- /dev/sdc /mnt/scratch > > shared/032 2s ... 2s > _check_btrfs_filesystem: filesystem on /dev/sdc is inconsistent (see /root/xfstests-dev/results//shared/032.full) > Ran: shared/032 > Passed all 1 tests > > This is because xfstests will run fsck after each unit test, > for tests/shared/032, we have reformated SCRATCH_DEV to other fs, > fix this problem by skipping fsck here. > > Signed-off-by: Wang Shilong <wangshilong1991@xxxxxxxxx> > --- > v2->v3: fix commit message.... I was about to point this out, and saw your v3 :) One nitpick, the "xfstests," part in summary is not necessary to me, "shared/032: ..." is good enough. Anyway, Reviewed-by: Eryu Guan <eguan@xxxxxxxxxx> > v1->v2: use _require_scratch_nocheck helper to skip fsck.(pointed by Eryu Guan) > --- > tests/shared/032 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/shared/032 b/tests/shared/032 > index 7835e30..a410003 100755 > --- a/tests/shared/032 > +++ b/tests/shared/032 > @@ -40,7 +40,7 @@ rm -f $seqres.full > _supported_fs xfs btrfs > _supported_os Linux > > -_require_scratch > +_require_scratch_nocheck > _require_no_large_scratch_dev > > # mkfs.btrfs did not have overwrite detection at first > -- > 1.8.3.1 > > -- > To unsubscribe from this list: send the line "unsubscribe fstests" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
