The branch main has been updated by lwhsu:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=b9b5a4dd590e1b30d92dc73b3d1f22d3303e7e41

commit b9b5a4dd590e1b30d92dc73b3d1f22d3303e7e41
Author:     Li-Wen Hsu <lw...@freebsd.org>
AuthorDate: 2021-09-28 18:38:34 +0000
Commit:     Li-Wen Hsu <lw...@freebsd.org>
CommitDate: 2021-09-28 18:38:34 +0000

    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
_______________________________________________
dev-commits-src-main@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
To unsubscribe, send any mail to "dev-commits-src-main-unsubscr...@freebsd.org"

Reply via email to