zbd: Improve robustness of unit tests
authorBart Van Assche <bvanassche@acm.org>
Wed, 14 Aug 2019 20:10:07 +0000 (13:10 -0700)
committerJens Axboe <axboe@kernel.dk>
Wed, 14 Aug 2019 21:01:29 +0000 (15:01 -0600)
Give up if creation of the null_blk instance fails.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
t/zbd/run-tests-against-zoned-nullb
t/zbd/test-zbd-support

index 9336716d47626da7523d77d17156c7bb3dfed3f0..0952011c576e04af1d5ffedaf4db5a58b40b36f7 100755 (executable)
@@ -24,6 +24,6 @@ modprobe null_blk nr_devices=0 &&
     echo 4096 > blocksize &&
     echo 1024 > size &&
     echo 1 > memory_backed &&
-    echo 1 > power
+    echo 1 > power || exit $?
 
 "${scriptdir}"/test-zbd-support "$@" /dev/nullb0
index 6fb48ef091cffad42c28e2330b01e206b46485fb..ed54a0aa21fd61702da0eeb73605d8520ceb8929 100755 (executable)
@@ -772,8 +772,8 @@ source "$(dirname "$0")/functions" || exit $?
 dev=$1
 realdev=$(readlink -f "$dev")
 basename=$(basename "$realdev")
-major=$((0x$(stat -L -c '%t' "$realdev")))
-minor=$((0x$(stat -L -c '%T' "$realdev")))
+major=$((0x$(stat -L -c '%t' "$realdev"))) || exit $?
+minor=$((0x$(stat -L -c '%T' "$realdev"))) || exit $?
 disk_size=$(($(<"/sys/dev/block/$major:$minor/size")*512))
 # When the target is a partition device, get basename of its holder device to
 # access sysfs path of the holder device