[Top] [All Lists]

[PATCH v2] xfstests: use correct size value in generic/273

To: xfs@xxxxxxxxxxx
Subject: [PATCH v2] xfstests: use correct size value in generic/273
From: Brian Foster <bfoster@xxxxxxxxxx>
Date: Thu, 21 Nov 2013 08:32:27 -0500
Delivered-to: xfs@xxxxxxxxxxx
generic/273 factors the "space available" output from df into the
calculation for the size of the origin data set. Recent commit

  bfdd1e72b358 xfstests: added -P option to $DF_PROG

... converted the use of 'df' to $DF_PROG. This implicitly adds the
-T parameter to add the fs type column, shifts the available space
column over by one and unintentionally causes 273 to look at "used
space" and create too small of a data set for a useful test.
Realign to the available space value.

Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx>

- Use tail and simplified awk command to improve readability. [dchinner]

 tests/generic/273 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/generic/273 b/tests/generic/273
index 63bbf9b..8e121a2 100755
--- a/tests/generic/273
+++ b/tests/generic/273
@@ -68,7 +68,7 @@ _file_create()
        cd $SCRATCH_MNT/origin
-       _disksize=`$DF_PROG --block-size=1 $SCRATCH_DEV | awk -v 
sd=$SCRATCH_DEV 'BEGIN{c=0}{for(i=1;i<=NF;++i){a[c]=$i;++c}}END{for(entry in 
a){if(a[entry] ~ sd){print a[entry + 3]; break}}}'`
+       _disksize=`$DF_PROG --block-size=1 $SCRATCH_DEV | tail -1 | awk '{ 
print $5 }'`
        _disksize=$(($_disksize / 3))
        _num=$(($_disksize / $count / $threads / 4096))

<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH v2] xfstests: use correct size value in generic/273, Brian Foster <=