Liu Bo
2013-Sep-18 05:17 UTC
[PATCH] xfstests/btrfs: do not test btrfs/010 with autodefrag
btrfs/010 is going to create a fragmented file, however, with autodefrag
this is impossible, so just skip the test when we''re with autodefrag.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
---
tests/btrfs/010 | 12 ++++++++++++
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tests/btrfs/010 b/tests/btrfs/010
index 371f1b8..608942d 100755
--- a/tests/btrfs/010
+++ b/tests/btrfs/010
@@ -37,6 +37,14 @@ _cleanup()
rm -f $tmp.*
}
+is_autodefrag_set()
+{
+ case "$MOUNT_OPTIONS" in
+ *autodefrag*) return 0;;
+ esac
+ return 1
+}
+
# get standard environment, filters and checks
. ./common/rc
. ./common/filter
@@ -49,6 +57,10 @@ _supported_os Linux
_require_scratch
_require_command "/usr/sbin/filefrag"
+if is_autodefrag_set; then
+ _notrun "this test makes no sense with autodefrag"
+fi
+
rm -f $seqres.full
_scratch_mkfs > /dev/null 2>&1
--
1.7.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
Josef Bacik
2013-Sep-27 13:45 UTC
Re: [PATCH] xfstests/btrfs: do not test btrfs/010 with autodefrag
On Wed, Sep 18, 2013 at 01:17:55PM +0800, Liu Bo wrote:> btrfs/010 is going to create a fragmented file, however, with autodefrag > this is impossible, so just skip the test when we''re with autodefrag. > > Signed-off-by: Liu Bo <bo.li.liu@oracle.com>I''d rather you just strip out the autodefrag option if it is there and add it back once the test is done (or hell just remove it, mount, and add it back). Thanks, Josef _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs
David Sterba
2013-Oct-01 14:40 UTC
Re: [PATCH] xfstests/btrfs: do not test btrfs/010 with autodefrag
On Fri, Sep 27, 2013 at 09:45:44AM -0400, Josef Bacik wrote:> On Wed, Sep 18, 2013 at 01:17:55PM +0800, Liu Bo wrote: > > btrfs/010 is going to create a fragmented file, however, with autodefrag > > this is impossible, so just skip the test when we''re with autodefrag. > > > > Signed-off-by: Liu Bo <bo.li.liu@oracle.com> > > I''d rather you just strip out the autodefrag option if it is there and add it > back once the test is done (or hell just remove it, mount, and add it back).I think the test should be skipped if it does not make sense with a specific mount option rather than temporarily removing it, because then it looks like the test passes with the option altough it does not. _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs