On Fri, Jul 01, 2016 at 11:18:32AM +0800, Eryu Guan wrote: > generic/067 mounts $SCRATCH_DEV directly in the test, assuming it's > a block device. generic/299 and generic/300 query the size of > $SCRATCH_DEV by running 'blockdev --getsz $SCRATCH_DEV'. > > So add the check to make sure $SCRATCH_DEV is a real block device in > these tests. > > Signed-off-by: Eryu Guan <eguan@xxxxxxxxxx> Ping on this as well. Thanks, Eryu > --- > tests/generic/067 | 1 + > tests/generic/299 | 1 + > tests/generic/300 | 1 + > 3 files changed, 3 insertions(+) > > diff --git a/tests/generic/067 b/tests/generic/067 > index 8a7a55c..8f2baf5 100755 > --- a/tests/generic/067 > +++ b/tests/generic/067 > @@ -56,6 +56,7 @@ _require_test_symlinks > _require_test > _require_scratch > _require_loop > +_require_block_device $SCRATCH_DEV > > rm -f $seqres.full > > diff --git a/tests/generic/299 b/tests/generic/299 > index 40265c8..c9ed0e0 100755 > --- a/tests/generic/299 > +++ b/tests/generic/299 > @@ -43,6 +43,7 @@ _supported_fs generic > _supported_os Linux > _require_scratch > _require_odirect > +_require_block_device $SCRATCH_DEV > > NUM_JOBS=$((4*LOAD_FACTOR)) > BLK_DEV_SIZE=`blockdev --getsz $SCRATCH_DEV` > diff --git a/tests/generic/300 b/tests/generic/300 > index c187f43..181594e 100755 > --- a/tests/generic/300 > +++ b/tests/generic/300 > @@ -43,6 +43,7 @@ _supported_fs generic > _supported_os Linux > _require_scratch > _require_odirect > +_require_block_device $SCRATCH_DEV > > # xfs_io is not required for this test, but it's the best way to verify > # the test system supports fallocate() for allocation and hole punching > -- > 2.7.4 > -- 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
