[PATCH] xfstests: 311: fsck the dmflakey device instead of the real device V2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dave pointed out that xfs was having issues with 311 because of caching issues.
He suggested that I fsck the dm-flakey device to make sure we don't have this
problem.  Make _check_scratch_fs take an optional argument to use as the device
to fsck.  Thanks,

Signed-off-by: Josef Bacik <jbacik@xxxxxxxxxxxx>
---
V1->V2: make _check_scratch_fs() take the optional argument of the device to
check

 common/rc         |   11 +++++++----
 tests/generic/311 |    2 +-
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/common/rc b/common/rc
index 7adf8b8..fe6bbfc 100644
--- a/common/rc
+++ b/common/rc
@@ -1678,6 +1678,9 @@ _check_test_fs()
 
 _check_scratch_fs()
 {
+    device=$SCRATCH_DEV
+    [ $# -eq 1 ] && device=$1
+
     case $FSTYP in
     xfs)
 	SCRATCH_LOG="none"
@@ -1688,19 +1691,19 @@ _check_scratch_fs()
 	[ "$USE_EXTERNAL" = yes -a ! -z "$SCRATCH_RTDEV" ] && \
 	    SCRATCH_RT="$SCRATCH_RTDEV"
 
-	_check_xfs_filesystem $SCRATCH_DEV $SCRATCH_LOG $SCRATCH_RT
+	_check_xfs_filesystem $device $SCRATCH_LOG $SCRATCH_RT
 	;;
     udf)
-	_check_udf_filesystem $SCRATCH_DEV $udf_fsize
+	_check_udf_filesystem $device $udf_fsize
 	;;
     nfs*)
 	# Don't know how to check an NFS filesystem, yet.
 	;;
     btrfs)
-	_check_btrfs_filesystem $SCRATCH_DEV
+	_check_btrfs_filesystem $device
 	;;
     *)
-	_check_generic_filesystem $SCRATCH_DEV
+	_check_generic_filesystem $device
 	;;
     esac
 }
diff --git a/tests/generic/311 b/tests/generic/311
index 2b3b569..675d927 100644
--- a/tests/generic/311
+++ b/tests/generic/311
@@ -125,7 +125,7 @@ _run_test()
 
 	#Unmount and fsck to make sure we got a valid fs after replay
 	_unmount_flakey
-	_check_scratch_fs
+	_check_scratch_fs $FLAKEY_DEV
 	[ $? -ne 0 ] && _fatal "fsck failed"
 
 	_mount_flakey
-- 
1.7.7.6

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




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux