From 50e2a514d9db26e37d45d78a8805fb596697b14e Mon Sep 17 00:00:00 2001 From: Christoph Hellwig Date: Wed, 6 Nov 2013 10:59:32 +0000 Subject: [PATCH] xfstests: add a helper to get the minimum dio size Various tests opencode checks to find out the minimum support direct I/O size. Replace those with a generic helper that handles network filesystems as well. Also remove the Linux 2.4 workaround we had in once place. Signed-off-by: Christoph Hellwig Reviewed-by: Eric Sandeen Signed-off-by: Rich Johnston --- common/rc | 15 +++++++++++++++ tests/generic/091 | 6 +----- tests/generic/240 | 2 +- tests/generic/263 | 2 +- 4 files changed, 18 insertions(+), 7 deletions(-) diff --git a/common/rc b/common/rc index ea3af128..f73414bd 100644 --- a/common/rc +++ b/common/rc @@ -2179,6 +2179,21 @@ _scale_fsstress_args() echo $args } +# +# Return the logical block size if running on a block device, +# else substitute the page size. +# +_min_dio_alignment() +{ + dev=$1 + + if [ -b "$dev" ]; then + blockdev --getss $dev + else + $here/src/feature -s + fi +} + run_check() { echo "# $@" >> $seqres.full 2>&1 diff --git a/tests/generic/091 b/tests/generic/091 index cee012dc..bb176f13 100755 --- a/tests/generic/091 +++ b/tests/generic/091 @@ -54,11 +54,7 @@ run_fsx() } psize=`$here/src/feature -s` -bsize=`blockdev --getss $TEST_DEV` -kernel=`uname -r | sed -e 's/\(2\..\).*/\1/'` - -# 2.4 Linux kernels support bsize aligned direct I/O only -[ "$HOSTOS" = "Linux" -a "$kernel" = "2.4" ] && bsize=$psize +bsize=`_min_dio_alignment $TEST_DEV` # fsx usage: # diff --git a/tests/generic/240 b/tests/generic/240 index e6923180..e26f4951 100755 --- a/tests/generic/240 +++ b/tests/generic/240 @@ -60,7 +60,7 @@ rm -f $seqres.full rm -f $TEST_DIR/aiodio_sparse -logical_block_size=`blockdev --getss $TEST_DEV` +logical_block_size=`_min_dio_alignment $TEST_DEV` fs_block_size=`stat -f $TEST_DIR | grep "Block size:" | awk '{print $3}'` if [ $fs_block_size -le $logical_block_size ]; then diff --git a/tests/generic/263 b/tests/generic/263 index 377b199a..bc59865e 100755 --- a/tests/generic/263 +++ b/tests/generic/263 @@ -54,7 +54,7 @@ run_fsx() } psize=`$here/src/feature -s` -bsize=`blockdev --getss $TEST_DEV` +bsize=`_min_dio_alignment $TEST_DEV` run_fsx -N 10000 -o 8192 -l 500000 -r PSIZE -t BSIZE -w BSIZE -Z run_fsx -N 10000 -o 128000 -l 500000 -r PSIZE -t BSIZE -w BSIZE -Z -- 2.39.5