[PATCH 1/2] Allow to specify unsupported file systems

[PATCH 1/2] Allow to specify unsupported file systems

[Lukas Czerner]

As an addition to checking for supported file systems it would be nice
to be able to specify unsupported file system. In combination with
"generic" fs it can be useful. This patch adds that feature. For example
to set ext3 as an unsupported fs add "!ext3" into list.

Also specify ext3 and ex4 as unsupported file systems in 213 and 214
since those fs does not support fallocate. There may be others though.

Signed-off-by: Lukas Czerner <lczerner@redhat.com>
---
 213       |    2 +-
 214       |    2 +-
 common.rc |   22 +++++++++++++++++-----
 3 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/213 b/213
index 9982dd4..3e90c0a 100755
--- a/213
+++ b/213
@@ -47,7 +47,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
 
 # real QA test starts here
 # generic, but xfs_io's fallocate must work
-_supported_fs generic
+_supported_fs generic !ext3 !ext3
 # only Linux supports fallocate
 _supported_os Linux
 
diff --git a/214 b/214
index cef7626..c01ef30 100755
--- a/214
+++ b/214
@@ -45,7 +45,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
 
 # real QA test starts here
 # generic, but xfs_io's fallocate must work
-_supported_fs generic
+_supported_fs generic !ext3 !ext2
 # only Linux supports fallocate
 _supported_os Linux
 
diff --git a/common.rc b/common.rc
index 6bf1e12..f2f4cc3 100644
--- a/common.rc
+++ b/common.rc
@@ -656,15 +656,27 @@ _fail()
 #
 _supported_fs()
 {
+    supp=0
     for f
     do
-	if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
-	then
-	    return
-	fi
+        if [ "$f" = \!"$FSTYP" ]
+        then
+            supp=0
+            break
+        fi
+
+        if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
+        then
+            supp=1
+        fi
     done
 
-    _notrun "not suitable for this filesystem type: $FSTYP"
+    if [ $supp -eq 1 ]
+    then
+        return
+    else
+        _notrun "not suitable for this filesystem type: $FSTYP"
+    fi
 }
 
 # tests whether $FSTYP is one of the supported OSes for a test
-- 
1.7.1.1

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

[PATCH 2/2] Show duration of current test

[Lukas Czerner]

It may be useful to compare both times, last and current, in the
xfstests output.

Signed-off-by: Lukas Czerner <lczerner@redhat.com>
---
 check |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/check b/check
index 560ac7b..f792009 100755
--- a/check
+++ b/check
@@ -275,13 +275,14 @@ do
 	    else
 		if diff $seq.out $tmp.out >/dev/null 2>&1
 		then
-		    echo ""
 		    if $err
 		    then
 			:
 		    else
 			echo "$seq `expr $stop - $start`" >>$tmp.time
+			echo -n " `expr $stop - $start`s"
 		    fi
+		    echo ""
 		else
 		    echo " - output mismatch (see $seq.out.bad)"
 		    mv $tmp.out $seq.out.bad
-- 
1.7.1.1

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

Re: [PATCH 2/2] Show duration of current test

[Alex Elder]

On Thu, 2010-07-08 at 17:23 +0200, Lukas Czerner wrote:
> It may be useful to compare both times, last and current, in the
> xfstests output.

This one looks good.

Signed-off-by: Alex Elder <aelder@sgi.com>

> Signed-off-by: Lukas Czerner <lczerner@redhat.com>
> ---
>  check |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/check b/check
> index 560ac7b..f792009 100755
> --- a/check
> +++ b/check
> @@ -275,13 +275,14 @@ do
>  	    else
>  		if diff $seq.out $tmp.out >/dev/null 2>&1
>  		then
> -		    echo ""
>  		    if $err
>  		    then
>  			:
>  		    else
>  			echo "$seq `expr $stop - $start`" >>$tmp.time
> +			echo -n " `expr $stop - $start`s"
>  		    fi
> +		    echo ""
>  		else
>  		    echo " - output mismatch (see $seq.out.bad)"
>  		    mv $tmp.out $seq.out.bad



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

Re: [PATCH 1/2] Allow to specify unsupported file systems

[Alex Elder]

On Thu, 2010-07-08 at 17:23 +0200, Lukas Czerner wrote:
> As an addition to checking for supported file systems it would be nice
> to be able to specify unsupported file system. In combination with
> "generic" fs it can be useful. This patch adds that feature. For example
> to set ext3 as an unsupported fs add "!ext3" into list.

Interesting idea, I agree it could be useful to
exclude rather than have to enumerate everything.

I don't think there's any need to make use of it
except where needed--i.e., don't go changing things,
just use it where it fits from here on.

One comment below.

> Also specify ext3 and ex4 as unsupported file systems in 213 and 214
> since those fs does not support fallocate. There may be others though.
> 
> Signed-off-by: Lukas Czerner <lczerner@redhat.com>
> ---
>  213       |    2 +-
>  214       |    2 +-
>  common.rc |   22 +++++++++++++++++-----
>  3 files changed, 19 insertions(+), 7 deletions(-)
> 
> diff --git a/213 b/213
> index 9982dd4..3e90c0a 100755
> --- a/213
> +++ b/213
> @@ -47,7 +47,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
>  
>  # real QA test starts here
>  # generic, but xfs_io's fallocate must work
> -_supported_fs generic
> +_supported_fs generic !ext3 !ext3

Did you mean !ext2 for the second one here?

>  # only Linux supports fallocate
>  _supported_os Linux
>  
> diff --git a/214 b/214
> index cef7626..c01ef30 100755
> --- a/214
> +++ b/214
> @@ -45,7 +45,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
>  
>  # real QA test starts here
>  # generic, but xfs_io's fallocate must work
> -_supported_fs generic
> +_supported_fs generic !ext3 !ext2
>  # only Linux supports fallocate
>  _supported_os Linux
>  
> diff --git a/common.rc b/common.rc
> index 6bf1e12..f2f4cc3 100644
> --- a/common.rc
> +++ b/common.rc
> @@ -656,15 +656,27 @@ _fail()
>  #
>  _supported_fs()
>  {
> +    supp=0
>      for f
>      do
> -	if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
> -	then
> -	    return
> -	fi
> +        if [ "$f" = \!"$FSTYP" ]
> +        then
> +            supp=0
> +            break
> +        fi
> +
> +        if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
> +        then
> +            supp=1
> +        fi
>      done
>  
> -    _notrun "not suitable for this filesystem type: $FSTYP"
> +    if [ $supp -eq 1 ]
> +    then
> +        return
> +    else
> +        _notrun "not suitable for this filesystem type: $FSTYP"
> +    fi
>  }
>  
>  # tests whether $FSTYP is one of the supported OSes for a test





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

Re: [PATCH 1/2] Allow to specify unsupported file systems

[Lukas Czerner]

On Fri, 9 Jul 2010, Alex Elder wrote:

> On Thu, 2010-07-08 at 17:23 +0200, Lukas Czerner wrote:
> > As an addition to checking for supported file systems it would be nice
> > to be able to specify unsupported file system. In combination with
> > "generic" fs it can be useful. This patch adds that feature. For example
> > to set ext3 as an unsupported fs add "!ext3" into list.
> 
> Interesting idea, I agree it could be useful to
> exclude rather than have to enumerate everything.
> 
> I don't think there's any need to make use of it
> except where needed--i.e., don't go changing things,
> just use it where it fits from here on.

ok, agreed.

Thanks.

> 
> One comment below.
> 
> > Also specify ext3 and ex4 as unsupported file systems in 213 and 214
> > since those fs does not support fallocate. There may be others though.
> > 
> > Signed-off-by: Lukas Czerner <lczerner@redhat.com>
> > ---
> >  213       |    2 +-
> >  214       |    2 +-
> >  common.rc |   22 +++++++++++++++++-----
> >  3 files changed, 19 insertions(+), 7 deletions(-)
> > 
> > diff --git a/213 b/213
> > index 9982dd4..3e90c0a 100755
> > --- a/213
> > +++ b/213
> > @@ -47,7 +47,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
> >  
> >  # real QA test starts here
> >  # generic, but xfs_io's fallocate must work
> > -_supported_fs generic
> > +_supported_fs generic !ext3 !ext3
> 
> Did you mean !ext2 for the second one here?

Oh, my bad, it should be !ext2. Do you want me to send it again ?

> 
> >  # only Linux supports fallocate
> >  _supported_os Linux
> >  
> > diff --git a/214 b/214
> > index cef7626..c01ef30 100755
> > --- a/214
> > +++ b/214
> > @@ -45,7 +45,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
> >  
> >  # real QA test starts here
> >  # generic, but xfs_io's fallocate must work
> > -_supported_fs generic
> > +_supported_fs generic !ext3 !ext2
> >  # only Linux supports fallocate
> >  _supported_os Linux
> >  
> > diff --git a/common.rc b/common.rc
> > index 6bf1e12..f2f4cc3 100644
> > --- a/common.rc
> > +++ b/common.rc
> > @@ -656,15 +656,27 @@ _fail()
> >  #
> >  _supported_fs()
> >  {
> > +    supp=0
> >      for f
> >      do
> > -	if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
> > -	then
> > -	    return
> > -	fi
> > +        if [ "$f" = \!"$FSTYP" ]
> > +        then
> > +            supp=0
> > +            break
> > +        fi
> > +
> > +        if [ "$f" = "$FSTYP" -o "$f" = "generic" ]
> > +        then
> > +            supp=1
> > +        fi
> >      done
> >  
> > -    _notrun "not suitable for this filesystem type: $FSTYP"
> > +    if [ $supp -eq 1 ]
> > +    then
> > +        return
> > +    else
> > +        _notrun "not suitable for this filesystem type: $FSTYP"
> > +    fi
> >  }
> >  
> >  # tests whether $FSTYP is one of the supported OSes for a test
> 
> 
> 
> 
> 
> 

-- 

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