public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: xfs@oss.sgi.com
Subject: [PATCH 4/4] xfsqa: add simple xfs_fsr test
Date: Tue, 12 Jan 2010 15:41:09 +1100	[thread overview]
Message-ID: <1263271269-18997-5-git-send-email-david@fromorbit.com> (raw)
In-Reply-To: <1263271269-18997-1-git-send-email-david@fromorbit.com>

We have very little fsr test coverage in the test suite right now.
042 is the only test that runs it, and it only defragments a single
file.

Improve the coverage by simply running xfs_fsr of the test
filesystem. This gives xfs_fsr a wide variety of inodes to try to
defragment. A simple test like this would have tripped over the
attr2 fork offset problems in the swap extents ioctl long ago
thanks to the varied nature of inodes the test suite leaves around
in this filesystem.

Signed-off-by: Dave Chinner <david@fromorbit.com>
---
 221     |   57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 221.out |    2 ++
 group   |    1 +
 3 files changed, 60 insertions(+), 0 deletions(-)
 create mode 100644 221
 create mode 100644 221.out

diff --git a/221 b/221
new file mode 100644
index 0000000..5fe5583
--- /dev/null
+++ b/221
@@ -0,0 +1,57 @@
+#! /bin/sh
+# FS QA Test No. 221
+#
+# xfs_fsr QA tests
+# run xfs_fsr over the test filesystem to give it a wide and varied set of
+# inodes to try to defragment. This is effectively a crash/assert failure
+# test looking for corruption induced by xfs_fsr runs.
+#
+#-----------------------------------------------------------------------
+# Copyright (c) 2010 Dave Chinner.  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=david@fromorbit.com
+
+seq=`basename $0`
+echo "QA output created by $seq"
+
+here=`pwd`
+tmp=/tmp/$$
+status=1	# failure is the default!
+
+_cleanup()
+{
+    rm -f $tmp.*
+}
+
+trap "_cleanup ; exit \$status" 0 1 2 3 15
+
+# get standard environment, filters and checks
+. ./common.rc
+. ./common.filter
+
+# real QA test starts here
+_supported_fs xfs
+_supported_os Linux
+
+[ "$XFS_FSR_PROG" = "" ] && _notrun "xfs_fsr not found"
+
+xfs_fsr $TEST_MNT > /dev/null 2>&1
+
+echo "--- silence is golden ---"
+status=0 ; exit
diff --git a/221.out b/221.out
new file mode 100644
index 0000000..34b5d1b
--- /dev/null
+++ b/221.out
@@ -0,0 +1,2 @@
+QA output created by 221
+--- silence is golden ---
diff --git a/group b/group
index 2c2e293..528f5e9 100644
--- a/group
+++ b/group
@@ -330,3 +330,4 @@ prealloc
 218 auto fsr quick
 219 auto quota quick
 220 auto quota quick
+221 fsr ioctl auto quick
-- 
1.6.5

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  parent reply	other threads:[~2010-01-12  4:44 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-12  4:41 [PATCH 0/4] xfsqa: make some tests work again, add xfs_fsr test Dave Chinner
2010-01-12  4:41 ` [PATCH 1/4] xfsqa: filter the fsstress seed output in 104 Dave Chinner
2010-01-12  9:31   ` Christoph Hellwig
2010-01-12  4:41 ` [PATCH 2/4] xfsqa: make unmounting loop devices in 073 work again Dave Chinner
2010-01-12 10:00   ` Christoph Hellwig
2010-01-12 11:51     ` Dave Chinner
2010-01-12  4:41 ` [PATCH 3/4] xfsqa: filter log size in 206 Dave Chinner
2010-01-12  9:47   ` Christoph Hellwig
2010-01-12  4:41 ` Dave Chinner [this message]
2010-01-12  9:50   ` [PATCH 4/4] xfsqa: add simple xfs_fsr test 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=1263271269-18997-5-git-send-email-david@fromorbit.com \
    --to=david@fromorbit.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox