On Fri, 27 Feb 2015, Filipe Manana wrote: > Date: Fri, 27 Feb 2015 10:41:36 +0000 > From: Filipe Manana <fdmanana@xxxxxxxx> > To: fstests@xxxxxxxxxxxxxxx > Cc: Filipe Manana <fdmanana@xxxxxxxx>, Lukas Czerner <lczerner@xxxxxxxxxx>, > Jaegeuk Kim <jaegeuk@xxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxx> > Subject: [PATCH] generic/065: fsync file 'hello' before checking its content > > Explicitly fsync the file named 'hello' before checking its content. > This way there's only one expected result for all filesystems. Looks good. Thanks! Reviewed-by: Lukas Czerner <lczerner@xxxxxxxxxx> > > Cc: Lukas Czerner <lczerner@xxxxxxxxxx> > Cc: Jaegeuk Kim <jaegeuk@xxxxxxxxxx> > Cc: Eric Sandeen <sandeen@xxxxxxxxxx> > Signed-off-by: Filipe Manana <fdmanana@xxxxxxxx> > --- > tests/generic/065 | 28 +++++++++------------------- > tests/generic/065.out | 5 ++++- > 2 files changed, 13 insertions(+), 20 deletions(-) > > diff --git a/tests/generic/065 b/tests/generic/065 > index b5a296d..739a4d5 100755 > --- a/tests/generic/065 > +++ b/tests/generic/065 > @@ -96,6 +96,11 @@ touch $SCRATCH_MNT/mydir/x/y/z/qwerty > # Now fsync only our top directory. > $XFS_IO_PROG -c "fsync" $SCRATCH_MNT/mydir > > +# And fsync now our new file named 'hello', just to verify later that it has > +# the expected content and that the previous fsync on the directory 'mydir' had > +# no bad influence on this fsync. > +$XFS_IO_PROG -c "fsync" $SCRATCH_MNT/hello > + > # Simulate a crash/power loss. > _load_flakey_table $FLAKEY_DROP_WRITES > _unmount_flakey > @@ -131,25 +136,10 @@ rm -f $SCRATCH_MNT/foo > [ -f $SCRATCH_MNT/mydir/x/y/z/qwerty ] || \ > echo "File mydir/x/y/z/qwerty is missing" > > -digest_ok=no > -hello_digest=$(md5sum $SCRATCH_MNT/hello | cut -d ' ' -f 1) > - > -case "$FSTYP" in > -ext3) > - # a 64Kb file, with all bytes having the value 0xff > - [ $hello_digest == "ecb99e6ffea7be1e5419350f725da86b" ] && digest_ok=yes > - ;; > -*) > - # an empty file > - [ $hello_digest == "d41d8cd98f00b204e9800998ecf8427e" ] && digest_ok=yes > - ;; > -esac > - > -if [ $digest_ok == "yes" ]; then > - echo "file 'hello' has expected size and content" > -else > - echo "file 'hello' has unexpected size or content" > -fi > +# We expect our file here to have a size of 64Kb and all the bytes having the > +# value 0xff. > +echo "file 'hello' content after log replay:" > +od -t x1 $SCRATCH_MNT/hello > > # Now remove all files/links, under our test directory 'mydir', and verify we > # can remove all the directories. > diff --git a/tests/generic/065.out b/tests/generic/065.out > index ad376b3..6d394b0 100644 > --- a/tests/generic/065.out > +++ b/tests/generic/065.out > @@ -8,4 +8,7 @@ File 'foo' content after log replay: > * > 0020000 > file 'foo' link count after log replay: 5 > -file 'hello' has expected size and content > +file 'hello' content after log replay: > +0000000 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > +* > +0200000 > -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
