From b9b5a4dd590e1b30d92dc73b3d1f22d3303e7e41 Mon Sep 17 00:00:00 2001 From: Li-Wen Hsu Date: Wed, 29 Sep 2021 02:38:34 +0800 Subject: [PATCH] gmultipath failloop test: Put the dtrace sanity checker in right place Check if dtrace excution is successful or not right after execution. Sponsored by: The FreeBSD Foundation --- tests/sys/geom/class/multipath/failloop.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/sys/geom/class/multipath/failloop.sh b/tests/sys/geom/class/multipath/failloop.sh index 632458e1f0b1..48006d6b10da 100755 --- a/tests/sys/geom/class/multipath/failloop.sh +++ b/tests/sys/geom/class/multipath/failloop.sh @@ -60,12 +60,12 @@ failloop_body() -i 'geom:multipath:config:restore {@restore = count()}' \ -c "dd if=/dev/zero of=/dev/multipath/"$name" bs=4096 count=1" \ 2>&1 | awk '/exited with status/ {print $NF}'` - # The dd command should've failed ... - atf_check_equal 1 $dd_status - # and triggered 1 or 2 path restores if [ ! -f restore_count ]; then atf_fail "dtrace didn't execute successfully" fi + # The dd command should've failed ... + atf_check_equal 1 $dd_status + # and triggered 1 or 2 path restores if [ `cat restore_count` -gt 2 ]; then atf_fail "gmultipath restored paths too many times" fi