From: Eric Sandeen <sandeen@redhat.com>
To: xfs-oss <xfs@oss.sgi.com>
Subject: [PATCH] xfstests: add _require_freeze and minor cleanups
Date: Wed, 19 Sep 2012 17:53:56 -0500 [thread overview]
Message-ID: <505A4D04.2080105@redhat.com> (raw)
More filesystems have grown freeze capability, so rather than
hardcoding several in _supported_fs, make tests 068 and 280
generic and then add a new _require_freeze() which checks whether
the fs under test can be frozen before beginning the test.
Minor other cleanups to 280:
- remove extra _supported_fs line
- clear $seq.full before beginning
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
diff --git a/068 b/068
index b595d1d..617420c 100755
--- a/068
+++ b/068
@@ -51,10 +51,11 @@ trap "_cleanup" 0 1 2 3 15
. ./common.filter
# real QA test starts here
-_supported_fs btrfs ext3 ext4 xfs
+_supported_fs generic
_supported_os Linux IRIX
_require_scratch
+_require_freeze
echo "*** init FS"
diff --git a/280 b/280
index 55849ed..5e26173 100755
--- a/280
+++ b/280
@@ -45,13 +45,15 @@ _cleanup()
_require_scratch
_require_quota
+_require_freeze
# real QA test starts here
# Modify as appropriate.
-_supported_fs generic
_supported_os Linux
-_supported_fs ext3 ext4 xfs
+_supported_fs generic
+
+rm -f $seq.full
umount $SCRATCH_DEV 2>/dev/null
_scratch_mkfs >> $seq.full 2>&1
diff --git a/common.rc b/common.rc
index 602513a..0e8a306 100644
--- a/common.rc
+++ b/common.rc
@@ -1758,6 +1758,15 @@ _require_btrfs()
[ $? -eq 0 ] || _notrun "$BTRFS_UTIL_PROG too old (must support $cmd)"
}
+# Does freeze work on this fs?
+_require_freeze()
+{
+ xfs_freeze -f "$TEST_DIR" >/dev/null 2>&1
+ result=$?
+ xfs_freeze -u "$TEST_DIR" >/dev/null 2>&1
+ [ $result -eq 0 ] || _notrun "$FSTYP does not support freezing"
+}
+
# arg 1 is dev to remove and is output of the below eg.
# ls -l /sys/class/block/sdd | rev | cut -d "/" -f 3 | rev
_devmgt_remove()
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next reply other threads:[~2012-09-19 22:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-19 22:53 Eric Sandeen [this message]
2012-09-21 16:38 ` [PATCH] xfstests: add _require_freeze and minor cleanups Ben Myers
2012-09-21 16:47 ` Eric Sandeen
2012-09-21 19:59 ` Ben Myers
2012-09-21 20:20 ` Eric Sandeen
2012-09-25 9:33 ` Christoph Hellwig
2012-09-25 15:28 ` Ben Myers
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=505A4D04.2080105@redhat.com \
--to=sandeen@redhat.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 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.