[PATCH 2/8] Fix fallocate() test

Jan Kara jack at suse.cz
Mon May 24 09:14:13 CDT 2010


The test for fallocate was broken because it used $TEST_DIR/$tmp.io. Because
$tmp is usually something like /tmp/1234 or /mnt/1234 the file cannot be
created and xfs_io fails regardless of existance of fallocate support. Moreover
the subsequent message parsing decides that fallocate is actually supported
because it does not expect this message.

Fix the test to not use $tmp.

Signed-off-by: Jan Kara <jack at suse.cz>
---
 common.rc |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/common.rc b/common.rc
index e6d818d..6bf1e12 100644
--- a/common.rc
+++ b/common.rc
@@ -803,8 +803,9 @@ _require_user()
 #
 _require_xfs_io_falloc()
 {
-	testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $TEST_DIR/$tmp.io 2>&1`
-	rm -f $TEST_DIR/$tmp.io 2>&1 > /dev/null
+	testfile=$TEST_DIR/$$.falloc
+	testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $testfile 2>&1`
+	rm -f $testfile 2>&1 > /dev/null
 	echo $testio | grep -q "not found" && \
 		_notrun "xfs_io fallocate support is missing"
 	echo $testio | grep -q "Operation not supported" && \
-- 
1.6.4.2




More information about the xfs mailing list