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 <[email protected]>
Suggested-by: Dave Chinner <[email protected]>
Signed-off-by: Eryu Guan <[email protected]>
---
 tests/ext4/305 | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/tests/ext4/305 b/tests/ext4/305
index 860d0a6..1711aae 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
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to