From 6c5493bd58655e65e82007bbe30eb82a3344ddd9 Mon Sep 17 00:00:00 2001 From: Eryu Guan Date: Mon, 21 Sep 2015 13:06:17 +1000 Subject: [PATCH] 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 Suggested-by: Dave Chinner Signed-off-by: Eryu Guan Reviewed-by: Dave Chinner Signed-off-by: Dave Chinner --- tests/ext4/305 | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) 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 -- 2.39.5