Re: generic/258 questions (mount issue)...

[Eric Sandeen <sandeen@sandeen.net>]

On 5/21/13 10:03 PM, Michael L. Semon wrote:

...

> From 7dc0667e1f8cd1c98f15ebf412dd0f038b55306e Mon Sep 17 00:00:00 2001
> From: "Michael L. Semon" <mlsemon35@gmail.com>
> Date: Tue, 21 May 2013 22:50:59 -0400
> Subject: [PATCH] Change mount method for generic/258
> 
> Use the built-in _test_mount function from xfstests so it will use
> the correct mount options for xfstests.  The script used a simple
> umount-and-mount sequence, which caused a test failure on an XFS
> filesystem that used both realtime and external log devices.
> 
> Signed-off-by: Michael L. Semon <mlsemon35@gmail.com>

Thanks, this was my fault  :(

Reviewed-by: Eric Sandeen <sandeen@rehat.com>

Want to consider dave's other suggestions in another patch?

> There are a handful of other tests that also unmount the TEST_DEV.
> Perhaps adding a _test_umount() wrapper to common/rc (similar to
> _scratch_umount) would be best. At least shared/243 needs the same
> _test_mount treatment as this test.

Eternal git-log fame would be yours.  ;)

-Eric

> ---
>  tests/generic/258 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/generic/258 b/tests/generic/258
> index fb091ae..3aeb5ce 100755
> --- a/tests/generic/258
> +++ b/tests/generic/258
> @@ -62,7 +62,7 @@ fi
>  # unmount, remount, and check the timestamp
>  echo "Remounting to flush cache"
>  umount $TEST_DEV
> -mount $TEST_DEV $TEST_DIR
> +_test_mount
>  
>  # Should yield -315593940 (prior to epoch)
>  echo "Testing for negative seconds since epoch"
> 

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