]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
Fix fallocate() test
authorJan Kara <jack@suse.cz>
Mon, 24 May 2010 09:46:44 +0000 (11:46 +0200)
committerJan Kara <jack@suse.cz>
Mon, 24 May 2010 09:46:44 +0000 (11:46 +0200)
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@suse.cz>
common.rc

index e6d818d005f4cec504d528858fd298599382aacd..6bf1e12782ec0f1de82223924d87e1a8b746a3a3 100644 (file)
--- 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" && \