From: Eryu Guan <eguan@redhat.com>
Date: Mon, 21 Sep 2015 03:06:17 +0000 (+1000)
Subject: ext4/305: reduce runtime by limiting mount/umount cycles
X-Git-Tag: v2022.05.01~2802
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=6c5493bd58655e65e82007bbe30eb82a3344ddd9;p=xfstests-dev.git

ext4/305: reduce runtime by limiting mount/umount cycles

ext4/305 sleeps 3 minutes and does mount/umount loop in background,
which produces lots of logs in dmesg and 3 minutes is not necessary.

Ted pointed out that 30 mount/umount cycles is enough to crash a buggy
kernel, so just limit the mount/umount loop to reduce the runtime. And
now the runtime is about 2s.

Reported-by: Theodore Ts'o <tytso@mit.edu>
Suggested-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Eryu Guan <eguan@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
---

diff --git a/tests/ext4/305 b/tests/ext4/305
index 860d0a65..1711aae8 100755
--- a/tests/ext4/305
+++ b/tests/ext4/305
@@ -53,19 +53,18 @@ DEV_BASENAME=$(_short_dev $SCRATCH_DEV)
 echo "Start test on device $SCRATCH_DEV, basename $DEV_BASENAME" >$seqres.full
 _scratch_mkfs >>$seqres.full 2>&1
 
-while true;do
-	mount $SCRATCH_DEV $SCRATCH_MNT
-	umount $SCRATCH_DEV
-done &
-PIDS=$!
-
-while true;do
+while true; do
 	cat /proc/fs/ext4/$DEV_BASENAME/mb_groups > /dev/null 2>&1
 done &
-PIDS="$PIDS $!"
+PIDS=$!
 
-# sleep for 180s, in most cases a buggy kernel could hit BUG_ON within 3 minutes
-sleep 180
+# 30 loops is enough to crash a buggy kernel
+i=0
+while [ $i -lt 30 ]; do
+	mount $SCRATCH_DEV $SCRATCH_MNT
+	umount $SCRATCH_DEV
+	let i=i+1
+done
 
 # no BUG_ON, all done
 kill $PIDS >/dev/null 2>&1