[PATCH 1/3] Skip _check_test_fs unless a test is actually run

[Felix Blyakher]

On Jun 19, 2009, at 12:53 PM, Theodore Ts'o wrote:

> Running _check_test_fs can take a non-trivial amount of time, and if a
> test has been skipped because it doesn't work on Linux, or it doesn't
> work of for ext4, it's pointless to re-run _check_test_fs.  So move
> the call to _check_test_fs so it is only run if a test is actually  
> run.

Definitely makes sense.

> Signed-off-by: "Theodore Ts'o" <tytso at mit.edu>

Reviewed-by: Felix Blyakher <felixb at sgi.com>

>
> ---
> check |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/check b/check
> index 545c010..1aae528 100755
> --- a/check
> +++ b/check
> @@ -308,10 +308,10 @@ do
>     then
> 	try="$try $seq"
> 	n_try=`expr $n_try + 1`
> +        _check_test_fs
>     fi
>
>     seq="after_$seq"
> -    _check_test_fs
> done
>
> interrupt=false
> -- 
> 1.6.3.2.1.gb9f7d.dirty
>
> _______________________________________________
> xfs mailing list
> xfs at oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs