From: Eric Sandeen <sandeen@sandeen.net>
To: Christoph Hellwig <hch@infradead.org>
Cc: ext4 development <linux-ext4@vger.kernel.org>, xfs-oss <xfs@oss.sgi.com>
Subject: Re: [PATCH 1/3] xfstests: basic fallocate boundary tests
Date: Sat, 20 Jun 2009 16:33:55 -0500 [thread overview]
Message-ID: <4A3D55C3.6020902@sandeen.net> (raw)
In-Reply-To: <20090620175824.GA3984@infradead.org>
Christoph Hellwig wrote:
> On Mon, Jun 15, 2009 at 10:06:20PM -0500, Eric Sandeen wrote:
>
>> This is just like test 072, but using fallocate instead
>> of the xfs ioctl. Just very basic fallocate tests.
>>
>> Also adds a "prealloc" group (./check -g prealloc)
>>
>
> Looks good to me, and does the right thing on my 32 bit test VM (notrun)
>
>
>> +echo $testio | grep -q "not found" && \
>> + _notrun "xfs_io fallocate support is missing"
>> +echo $testio | grep -q "Operation not supported" && \
>> + _notrun "xfs_io fallocate command failed (old kernel? wrong fs?)"
>>
>
> Maybe add a _require_fallocate as you need the same check in the next
> testcase?
>
hmm this is really testing that xfs_io in particular has fallocate...
so maybe like this?
diff --git a/common.rc b/common.rc
index 39350b3..55aecd6 100644
--- a/common.rc
+++ b/common.rc
@@ -704,6 +704,18 @@ _require_user()
[ "$?" == "0" ] || _notrun "$qa_user user not defined."
}
+# check that xfs_io, glibc, kernel, and filesystem all (!) support
+# fallocate
+#
+_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
+ echo $testio | grep -q "not found" && \
+ _notrun "xfs_io fallocate support is missing"
+ echo $testio | grep -q "Operation not supported" && \
+ _notrun "xfs_io fallocate command failed (old kernel/wrong fs?)"
+}
+
# check that a FS on a device is mounted
# if so, return mount point
#
diff --git a/213 b/213
new file mode 100755
index 0000000..3cd55f7
--- /dev/null
+++ b/213
@@ -0,0 +1,84 @@
+#! /bin/sh
+# FS QA Test No. 213
+#
+# Check some unwritten extent boundary conditions, fallocate version.
+#
+# Based on xfs-specific test 072
+#
+#-----------------------------------------------------------------------
+# Copyright (c) 2000-2003 Silicon Graphics, Inc. All Rights Reserved.
+# Copyright (c) 2009 Red Hat, Inc. All Rights Reserved.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#
+#-----------------------------------------------------------------------
+#
+# creator
+owner=sandeen@sandeen.net
+
+seq=`basename $0`
+echo "QA output created by $seq"
+
+_cleanup()
+{
+ cd /
+ rm -f $tmp.*
+}
+
+here=`pwd`
+tmp=$TEST_DIR/$$
+status=1 # failure is the default!
+trap "_cleanup; exit \$status" 0 1 2 3 15
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+# generic, but xfs_io's fallocate must work
+_supported_fs generic
+# only Linux supports fallocate
+_supported_os Linux
+
+[ -n "$XFS_IO_PROG" ] || _notrun "xfs_io executable not found"
+
+rm -f $seq.full
+
+_require_xfs_io_falloc
+
+# check there's enough freespace on $TEST_DIR ... (1GiB + 1MiB)
+avail=`df -P $TEST_DIR | awk 'END {print $4}'`
+[ "$avail" -ge 1049600 ] || _notrun "Test device is too small ($avail KiB)"
+
+echo Silence is golden
+
+# reserve 1GiB, truncate at 100bytes
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 100' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, truncate at 1GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 1g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, truncate at 2GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 2g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, 1GiB hole, reserve 1MiB, truncate at 3GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'falloc 2g 1m' -c 'truncate 3g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# success, all done
+status=0
+exit
diff --git a/group b/group
index 1268ddd..578fa05 100644
--- a/group
+++ b/group
@@ -101,6 +101,10 @@ nfs4acl tes@sgi.com donaldd@sgi.com
# test access time
atime
+# Test preallocation calls
+prealloc
+
+#
# test-group association ... one line per test
#
001 rw dir udf auto quick
@@ -111,7 +115,7 @@ atime
006 dir udf auto quick
007 dir udf auto quick
008 rw ioctl auto quick
-009 rw ioctl auto quick
+009 rw ioctl auto prealloc quick
010 other udf auto
011 dir udf auto quick
012 rw auto quick
@@ -174,7 +178,7 @@ atime
069 rw udf auto quick
070 attr udf auto quick
071 rw
-072 rw auto quick
+072 rw auto prealloc quick
073 copy auto
074 rw udf auto
075 rw udf auto quick
@@ -206,7 +210,7 @@ atime
101 udf
102 udf
103 metadata dir ioctl auto quick
-104 growfs ioctl
+104 growfs ioctl prealloc
105 acl auto quick
106 quota
107 quota
@@ -266,8 +270,8 @@ atime
161 dmapi auto
162 dmapi auto
163 dmapi auto
-164 rw pattern auto quick
-165 rw pattern auto quick
+164 rw pattern auto prealloc quick
+165 rw pattern auto prealloc quick
166 rw metadata auto quick
167 rw metadata auto
168 dmapi auto
@@ -317,3 +321,4 @@ atime
210 auto aio quick
211 auto aio quick
212 auto aio quick
+213 rw auto prealloc quick
WARNING: multiple messages have this Message-ID (diff)
From: Eric Sandeen <sandeen@sandeen.net>
To: Christoph Hellwig <hch@infradead.org>
Cc: ext4 development <linux-ext4@vger.kernel.org>, xfs-oss <xfs@oss.sgi.com>
Subject: Re: [PATCH 1/3] xfstests: basic fallocate boundary tests
Date: Sat, 20 Jun 2009 16:33:55 -0500 [thread overview]
Message-ID: <4A3D55C3.6020902@sandeen.net> (raw)
In-Reply-To: <20090620175824.GA3984@infradead.org>
Christoph Hellwig wrote:
> On Mon, Jun 15, 2009 at 10:06:20PM -0500, Eric Sandeen wrote:
>
>> This is just like test 072, but using fallocate instead
>> of the xfs ioctl. Just very basic fallocate tests.
>>
>> Also adds a "prealloc" group (./check -g prealloc)
>>
>
> Looks good to me, and does the right thing on my 32 bit test VM (notrun)
>
>
>> +echo $testio | grep -q "not found" && \
>> + _notrun "xfs_io fallocate support is missing"
>> +echo $testio | grep -q "Operation not supported" && \
>> + _notrun "xfs_io fallocate command failed (old kernel? wrong fs?)"
>>
>
> Maybe add a _require_fallocate as you need the same check in the next
> testcase?
>
hmm this is really testing that xfs_io in particular has fallocate...
so maybe like this?
diff --git a/common.rc b/common.rc
index 39350b3..55aecd6 100644
--- a/common.rc
+++ b/common.rc
@@ -704,6 +704,18 @@ _require_user()
[ "$?" == "0" ] || _notrun "$qa_user user not defined."
}
+# check that xfs_io, glibc, kernel, and filesystem all (!) support
+# fallocate
+#
+_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
+ echo $testio | grep -q "not found" && \
+ _notrun "xfs_io fallocate support is missing"
+ echo $testio | grep -q "Operation not supported" && \
+ _notrun "xfs_io fallocate command failed (old kernel/wrong fs?)"
+}
+
# check that a FS on a device is mounted
# if so, return mount point
#
diff --git a/213 b/213
new file mode 100755
index 0000000..3cd55f7
--- /dev/null
+++ b/213
@@ -0,0 +1,84 @@
+#! /bin/sh
+# FS QA Test No. 213
+#
+# Check some unwritten extent boundary conditions, fallocate version.
+#
+# Based on xfs-specific test 072
+#
+#-----------------------------------------------------------------------
+# Copyright (c) 2000-2003 Silicon Graphics, Inc. All Rights Reserved.
+# Copyright (c) 2009 Red Hat, Inc. All Rights Reserved.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it would be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write the Free Software Foundation,
+# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#
+#-----------------------------------------------------------------------
+#
+# creator
+owner=sandeen@sandeen.net
+
+seq=`basename $0`
+echo "QA output created by $seq"
+
+_cleanup()
+{
+ cd /
+ rm -f $tmp.*
+}
+
+here=`pwd`
+tmp=$TEST_DIR/$$
+status=1 # failure is the default!
+trap "_cleanup; exit \$status" 0 1 2 3 15
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+# generic, but xfs_io's fallocate must work
+_supported_fs generic
+# only Linux supports fallocate
+_supported_os Linux
+
+[ -n "$XFS_IO_PROG" ] || _notrun "xfs_io executable not found"
+
+rm -f $seq.full
+
+_require_xfs_io_falloc
+
+# check there's enough freespace on $TEST_DIR ... (1GiB + 1MiB)
+avail=`df -P $TEST_DIR | awk 'END {print $4}'`
+[ "$avail" -ge 1049600 ] || _notrun "Test device is too small ($avail KiB)"
+
+echo Silence is golden
+
+# reserve 1GiB, truncate at 100bytes
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 100' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, truncate at 1GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 1g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, truncate at 2GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'truncate 2g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# reserve 1GiB, 1GiB hole, reserve 1MiB, truncate at 3GiB
+$XFS_IO_PROG -F -f -c 'falloc 0 1g' -c 'falloc 2g 1m' -c 'truncate 3g' $TEST_DIR/ouch
+rm -f $TEST_DIR/ouch
+
+# success, all done
+status=0
+exit
diff --git a/group b/group
index 1268ddd..578fa05 100644
--- a/group
+++ b/group
@@ -101,6 +101,10 @@ nfs4acl tes@sgi.com donaldd@sgi.com
# test access time
atime
+# Test preallocation calls
+prealloc
+
+#
# test-group association ... one line per test
#
001 rw dir udf auto quick
@@ -111,7 +115,7 @@ atime
006 dir udf auto quick
007 dir udf auto quick
008 rw ioctl auto quick
-009 rw ioctl auto quick
+009 rw ioctl auto prealloc quick
010 other udf auto
011 dir udf auto quick
012 rw auto quick
@@ -174,7 +178,7 @@ atime
069 rw udf auto quick
070 attr udf auto quick
071 rw
-072 rw auto quick
+072 rw auto prealloc quick
073 copy auto
074 rw udf auto
075 rw udf auto quick
@@ -206,7 +210,7 @@ atime
101 udf
102 udf
103 metadata dir ioctl auto quick
-104 growfs ioctl
+104 growfs ioctl prealloc
105 acl auto quick
106 quota
107 quota
@@ -266,8 +270,8 @@ atime
161 dmapi auto
162 dmapi auto
163 dmapi auto
-164 rw pattern auto quick
-165 rw pattern auto quick
+164 rw pattern auto prealloc quick
+165 rw pattern auto prealloc quick
166 rw metadata auto quick
167 rw metadata auto
168 dmapi auto
@@ -317,3 +321,4 @@ atime
210 auto aio quick
211 auto aio quick
212 auto aio quick
+213 rw auto prealloc quick
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2009-06-20 21:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-16 3:06 [PATCH 1/3] xfstests: basic fallocate boundary tests Eric Sandeen
2009-06-20 17:58 ` Christoph Hellwig
2009-06-20 17:58 ` Christoph Hellwig
2009-06-20 21:33 ` Eric Sandeen [this message]
2009-06-20 21:33 ` Eric Sandeen
2009-06-21 19:16 ` Christoph Hellwig
2009-06-21 19:16 ` Christoph Hellwig
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A3D55C3.6020902@sandeen.net \
--to=sandeen@sandeen.net \
--cc=hch@infradead.org \
--cc=linux-ext4@vger.kernel.org \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.