Message ID | 1467343112-26227-1-git-send-email-eguan@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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@redhat.com> 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@redhat.com> Looks fine. Reviewed-by: Dave Chinner <dchinner@redhat.com>
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
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@redhat.com> --- tests/generic/067 | 1 + tests/generic/299 | 1 + tests/generic/300 | 1 + 3 files changed, 3 insertions(+)