On Thu, Aug 21, 2014 at 07:01:05PM +1000, Dave Chinner wrote:
> On Thu, Aug 21, 2014 at 10:04:30AM +0800, Qu Wenruo wrote:
> >
> > -------- Original Message --------
> > Subject: [PATCH 01/15] btrfs: new test to run btrfs balance and
> > subvolume test simultaneously
> > From: Eryu Guan <eguan@xxxxxxxxxx>
> > To: <fstests@xxxxxxxxxxxxxxx>
> > Date: 2014年08月21日 01:33
> > >Run btrfs balance and subvolume create/mount/umount/delete simultaneously,
> > >with fsstress running in background.
> > >
> > >Signed-off-by: Eryu Guan <eguan@xxxxxxxxxx>
[snip]
> > >+# real QA test starts here
> > >+_supported_fs btrfs
> > >+_supported_os Linux
> > >+_require_scratch
> > >+_require_scratch_dev_pool 4
> > >+
> > >+rm -f $seqres.full
> > >+
> > >+# test case array
> > >+tcs=(
> > >+ "-m single -d single"
> > >+ "-m dup -d single"
> > >+ "-m raid0 -d raid0"
> > >+ "-m raid1 -d raid0"
> > >+ "-m raid1 -d raid1"
> > >+ "-m raid10 -d raid10"
> > >+ "-m raid5 -d raid5"
> > >+ "-m raid6 -d raid6"
> > >+)
> > I wonder should we add the mkfs options there.
> > Since xfstests already use environment MKFS_OPTIONS to do mkfs,
> > if really need to test all mkfs options, IMO it is better to change
> > MKFS_OPTIONS on each test round.
>
> Hmmm - I you didn't read the code, because:
>
> > >+run_test()
> > >+{
> > >+ local mkfs_opts=$1
> > >+ local saved_mkfs_opts=$MKFS_OPTIONS
> > >+ local subvol_mnt=$tmp.mnt
> > >+
> > >+ echo "Test $mkfs_opts" >>$seqres.full
> > >+
> > >+ MKFS_OPTIONS="$MKFS_OPTIONS $mkfs_opts"
> > >+ # dup only works on single device
>
> it's doing exactly what you suggest.
>
> And it's wrong. This:
>
> _scratch_mkfs $mkfs_opts
>
> is all that is needed. This wheel does not need reinventing. ;)
I just noticed _scratch_pool_mkfs could do the same, thanks for the reminder!
Thanks,
Eryu
>
> Cheers,
>
> Dave.
> --
> Dave Chinner
> david@xxxxxxxxxxxxx
--
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
