generic/258 questions (mount issue)...

[Eric Sandeen]

On 5/22/13 11:15 AM, Michael L. Semon wrote:
> On 05/22/2013 10:10 AM, Eric Sandeen wrote:
>> 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 at 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 at gmail.com>
>>
>> Thanks, this was my fault  :(
>>
>> Reviewed-by: Eric Sandeen <sandeen at 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
> 
> Will do.  Let me get my feet back under me.  The initial 
> `grep -l _test_mount [0-9][0-9][0-9]` returned 014, 097, 192; so 
> the thread was started in the spirit of "there's no way this can 
> possibly be the correct function to use."
> 
> I'll look at shared/243.
> 
> One day, I would like to earn git-log fame by incorporating better 
> support for JFS and especially NILFS2 into xfstests.  However, 
> that's a topic for another day.  I know that I have uses for both 
> filesystems, but that doesn't mean anybody else does.  Their 
> mailing lists don't give much hint of user community or progress.

The generic tests hopefully work; if not, it should be a fairly
simple fixup.  And you're free to add fs-specific tests :)

(I'm not sure if we fall back by default to mkfs.$FSTYP and fsck.$FSTYP;
if so, it might just work)

-Eric

> Thanks!
> 
> Michael
> 
>>> ---
>>>  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"
>>>
>>
>