From: Daniel Wagner <dwagner@suse.de>
To: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Cc: linux-block@vger.kernel.org, linux-nvme@lists.infradead.org,
Chaitanya Kulkarni <chaitanyak@nvidia.com>,
Hannes Reinecke <hare@suse.de>, Daniel Wagner <dwagner@suse.de>
Subject: [PATCH blktests v3 15/20] nvme: drop default subsysnqn argument from _nvme_{connect|disconnect}_subsys
Date: Tue, 26 Mar 2024 14:13:57 +0100 [thread overview]
Message-ID: <20240326131402.5092-16-dwagner@suse.de> (raw)
In-Reply-To: <20240326131402.5092-1-dwagner@suse.de>
Remove the last positional argument for
_nvme_{connect|disconnect}_subsys which most test pass in the default
subsysnqn anyway. There is little point in cluttering all the test
textual noise.
Signed-off-by: Daniel Wagner <dwagner@suse.de>
---
tests/nvme/003 | 4 ++--
tests/nvme/004 | 4 ++--
tests/nvme/005 | 2 +-
tests/nvme/008 | 4 ++--
tests/nvme/009 | 4 ++--
tests/nvme/010 | 4 ++--
tests/nvme/011 | 4 ++--
tests/nvme/012 | 4 ++--
tests/nvme/013 | 4 ++--
tests/nvme/014 | 4 ++--
tests/nvme/015 | 4 ++--
tests/nvme/018 | 4 ++--
tests/nvme/019 | 4 ++--
tests/nvme/020 | 4 ++--
tests/nvme/021 | 4 ++--
tests/nvme/022 | 4 ++--
tests/nvme/023 | 4 ++--
tests/nvme/024 | 4 ++--
tests/nvme/025 | 4 ++--
tests/nvme/026 | 4 ++--
tests/nvme/027 | 4 ++--
tests/nvme/028 | 4 ++--
tests/nvme/029 | 4 ++--
tests/nvme/031 | 4 ++--
tests/nvme/033 | 2 +-
tests/nvme/034 | 2 +-
tests/nvme/035 | 2 +-
tests/nvme/036 | 2 +-
tests/nvme/037 | 3 ++-
tests/nvme/040 | 2 +-
tests/nvme/041 | 10 ++++------
tests/nvme/042 | 10 ++++------
tests/nvme/043 | 10 ++++------
tests/nvme/044 | 20 ++++++++------------
tests/nvme/045 | 6 ++----
tests/nvme/047 | 6 +++---
tests/nvme/048 | 11 +++++------
tests/nvme/rc | 30 +++++++++++++++++++++---------
38 files changed, 105 insertions(+), 105 deletions(-)
diff --git a/tests/nvme/003 b/tests/nvme/003
index 9a7c41f0856b..be6b4e18ec30 100755
--- a/tests/nvme/003
+++ b/tests/nvme/003
@@ -25,7 +25,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys nqn.2014-08.org.nvmexpress.discovery
+ _nvme_connect_subsys --subsysnqn "nqn.2014-08.org.nvmexpress.discovery"
# This is ugly but checking for the absence of error messages is ...
sleep 10
@@ -38,7 +38,7 @@ test() {
echo "Fail"
fi
- _nvme_disconnect_subsys nqn.2014-08.org.nvmexpress.discovery
+ _nvme_disconnect_subsys --subsysnqn "nqn.2014-08.org.nvmexpress.discovery"
_nvmet_target_cleanup
diff --git a/tests/nvme/004 b/tests/nvme/004
index 024ac986e5c1..b751746c6c91 100755
--- a/tests/nvme/004
+++ b/tests/nvme/004
@@ -24,13 +24,13 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
local nvmedev
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
- _nvme_disconnect_subsys ${def_subsysnqn}
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/005 b/tests/nvme/005
index 80a5359e862e..f17174d40dfc 100755
--- a/tests/nvme/005
+++ b/tests/nvme/005
@@ -25,7 +25,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
diff --git a/tests/nvme/008 b/tests/nvme/008
index fb1726723d43..a5d06818c6e4 100755
--- a/tests/nvme/008
+++ b/tests/nvme/008
@@ -24,12 +24,12 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/009 b/tests/nvme/009
index a9d83b675ba2..a1655d43e56f 100755
--- a/tests/nvme/009
+++ b/tests/nvme/009
@@ -23,12 +23,12 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/010 b/tests/nvme/010
index 496f6e5c6a52..7d875989a01c 100755
--- a/tests/nvme/010
+++ b/tests/nvme/010
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -32,7 +32,7 @@ test() {
_run_fio_verify_io --size="${nvme_img_size}" \
--filename="/dev/${nvmedev}n1"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/011 b/tests/nvme/011
index 14a17a774d5f..0acc8b1dbaed 100755
--- a/tests/nvme/011
+++ b/tests/nvme/011
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -32,7 +32,7 @@ test() {
_run_fio_verify_io --size="${nvme_img_size}" \
--filename="/dev/${nvmedev}n1"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/012 b/tests/nvme/012
index 8dbf8eb1a9f0..a0bff298ab29 100755
--- a/tests/nvme/012
+++ b/tests/nvme/012
@@ -28,7 +28,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -37,7 +37,7 @@ test() {
echo "FAIL: fio verify failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/013 b/tests/nvme/013
index eb22933fdec6..5e5026eabe17 100755
--- a/tests/nvme/013
+++ b/tests/nvme/013
@@ -27,7 +27,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -36,7 +36,7 @@ test() {
echo "FAIL: fio verify failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/014 b/tests/nvme/014
index 20ff69176231..da4b4c13b347 100755
--- a/tests/nvme/014
+++ b/tests/nvme/014
@@ -27,7 +27,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -41,7 +41,7 @@ test() {
nvme flush "/dev/${nvmedev}" --namespace-id 1
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/015 b/tests/nvme/015
index 634c42c07a86..b82f2253c011 100755
--- a/tests/nvme/015
+++ b/tests/nvme/015
@@ -27,7 +27,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -41,7 +41,7 @@ test() {
nvme flush "/dev/${nvmedev}n1" --namespace-id 1
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/018 b/tests/nvme/018
index 9225f7d58377..bd6e5e930e35 100755
--- a/tests/nvme/018
+++ b/tests/nvme/018
@@ -25,7 +25,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -39,7 +39,7 @@ test() {
--block-count 0 --data-size "$bs" &>"$FULL" \
&& echo "ERROR: nvme read for out of range LBA was not rejected"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/019 b/tests/nvme/019
index 9cf9f5299305..4d7de8191da7 100755
--- a/tests/nvme/019
+++ b/tests/nvme/019
@@ -26,7 +26,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -34,7 +34,7 @@ test() {
nvme dsm "/dev/${nvmedev}" --namespace-id 1 --ad \
--slbs "${sblk_range}" --blocks "${nblk_range}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/020 b/tests/nvme/020
index f6a204e6e417..c734210c5bd9 100755
--- a/tests/nvme/020
+++ b/tests/nvme/020
@@ -25,7 +25,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
nvme dsm "/dev/${nvmedev}" --namespace-id 1 --ad \
--slbs "${sblk_range}" --blocks "${nblk_range}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/021 b/tests/nvme/021
index 73e414e9db47..358e209d0458 100755
--- a/tests/nvme/021
+++ b/tests/nvme/021
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
echo "ERROR: device not listed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/022 b/tests/nvme/022
index 31435cd2e9c7..7ce33ddbf006 100755
--- a/tests/nvme/022
+++ b/tests/nvme/022
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
echo "ERROR: reset failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/023 b/tests/nvme/023
index c4c292899f32..abbb35a0b580 100755
--- a/tests/nvme/023
+++ b/tests/nvme/023
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -34,7 +34,7 @@ test() {
echo "ERROR: smart-log bdev-ns failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/024 b/tests/nvme/024
index b0d6f5a3c0b9..3ee18822e26b 100755
--- a/tests/nvme/024
+++ b/tests/nvme/024
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
>> "$FULL" 2>&1; then
echo "ERROR: smart-log file-ns failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/025 b/tests/nvme/025
index 107eb38ba787..3cd62d0b4e5e 100755
--- a/tests/nvme/025
+++ b/tests/nvme/025
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
echo "ERROR: effects-log failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/026 b/tests/nvme/026
index ff4ff91d5d4f..5d54b0ff3d28 100755
--- a/tests/nvme/026
+++ b/tests/nvme/026
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -34,7 +34,7 @@ test() {
echo "ERROR: ns-desc failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/027 b/tests/nvme/027
index a15e6d6e92d0..053fd58f9369 100755
--- a/tests/nvme/027
+++ b/tests/nvme/027
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
echo "ERROR: ns-rescan failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/028 b/tests/nvme/028
index 41dcb6ef7a4f..4710bba1f416 100755
--- a/tests/nvme/028
+++ b/tests/nvme/028
@@ -24,7 +24,7 @@ test() {
_nvmet_target_setup --blkdev file
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -33,7 +33,7 @@ test() {
echo "ERROR: list-subsys"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/029 b/tests/nvme/029
index 7bde5565b020..8dbe49ba15cc 100755
--- a/tests/nvme/029
+++ b/tests/nvme/029
@@ -58,7 +58,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
_check_uuid "${nvmedev}"
@@ -84,7 +84,7 @@ test() {
echo 0 > /proc/sys/vm/nr_hugepages
fi
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/031 b/tests/nvme/031
index 19854ccb903f..0bf823dbbb0d 100755
--- a/tests/nvme/031
+++ b/tests/nvme/031
@@ -43,8 +43,8 @@ test() {
_nvmet_target_setup --subsysnqn "${subsys}$i" \
--blkdev "${loop_dev}"
- _nvme_connect_subsys "${subsys}$i"
- _nvme_disconnect_subsys "${subsys}$i" >> "${FULL}" 2>&1
+ _nvme_connect_subsys --subsysnqn "${subsys}$i"
+ _nvme_disconnect_subsys --subsysnqn "${subsys}$i" >> "${FULL}" 2>&1
_nvmet_target_cleanup --subsysnqn "${subsys}$i" \
--blkdev "${loop_dev}"
diff --git a/tests/nvme/033 b/tests/nvme/033
index cb120e20b573..c0482c808d43 100755
--- a/tests/nvme/033
+++ b/tests/nvme/033
@@ -57,7 +57,7 @@ test_device() {
compare_dev_info "${nsdev}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_passthru_target_cleanup "${def_subsysnqn}"
echo "Test complete"
diff --git a/tests/nvme/034 b/tests/nvme/034
index 98a7db859b36..8bb2d8cd8d62 100755
--- a/tests/nvme/034
+++ b/tests/nvme/034
@@ -27,7 +27,7 @@ test_device() {
_run_fio_verify_io --size="${nvme_img_size}" --filename="${nsdev}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_passthru_target_cleanup "${def_subsysnqn}"
echo "Test complete"
diff --git a/tests/nvme/035 b/tests/nvme/035
index c17e8be6ce46..1d1b5e4ac46e 100755
--- a/tests/nvme/035
+++ b/tests/nvme/035
@@ -35,7 +35,7 @@ test_device() {
echo "FAIL: fio verify failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_passthru_target_cleanup "${def_subsysnqn}"
echo "Test complete"
diff --git a/tests/nvme/036 b/tests/nvme/036
index a1ae74fa95ea..0e9c3bf3486c 100755
--- a/tests/nvme/036
+++ b/tests/nvme/036
@@ -29,7 +29,7 @@ test_device() {
echo "ERROR: reset failed"
fi
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_passthru_target_cleanup "${def_subsysnqn}"
echo "Test complete"
diff --git a/tests/nvme/037 b/tests/nvme/037
index eb47839a4289..159d9d990bb7 100755
--- a/tests/nvme/037
+++ b/tests/nvme/037
@@ -25,7 +25,8 @@ test_device() {
_nvmet_passthru_target_setup "${subsys}${i}"
nsdev=$(_nvmet_passthru_target_connect "${subsys}${i}")
- _nvme_disconnect_subsys "${subsys}${i}" >>"${FULL}" 2>&1
+ _nvme_disconnect_subsys \
+ --subsysnqn "${subsys}${i}" >>"${FULL}" 2>&1
_nvmet_passthru_target_cleanup "${subsys}${i}"
done
diff --git a/tests/nvme/040 b/tests/nvme/040
index 06d0d0d47368..f00fc16b643f 100755
--- a/tests/nvme/040
+++ b/tests/nvme/040
@@ -26,7 +26,7 @@ test() {
_nvmet_target_setup
- _nvme_connect_subsys "${def_subsysnqn}"
+ _nvme_connect_subsys
nvmedev=$(_find_nvme_dev "${def_subsysnqn}")
# start fio job
diff --git a/tests/nvme/041 b/tests/nvme/041
index 02452fd52628..d6845628726e 100755
--- a/tests/nvme/041
+++ b/tests/nvme/041
@@ -38,20 +38,18 @@ test() {
# Test unauthenticated connection (should fail)
echo "Test unauthenticated connection (should fail)"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
# Test authenticated connection
echo "Test authenticated connection"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/042 b/tests/nvme/042
index 961301ff6993..480e1cba9d19 100755
--- a/tests/nvme/042
+++ b/tests/nvme/042
@@ -41,12 +41,11 @@ test() {
fi
_set_nvmet_hostkey "${def_hostnqn}" "${hostkey}"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
done
for key_len in 32 48 64; do
@@ -58,12 +57,11 @@ test() {
fi
_set_nvmet_hostkey "${def_hostnqn}" "${hostkey}"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
done
_nvmet_target_cleanup
diff --git a/tests/nvme/043 b/tests/nvme/043
index ed18869a5977..dd48035dd967 100755
--- a/tests/nvme/043
+++ b/tests/nvme/043
@@ -45,12 +45,11 @@ test() {
_set_nvmet_hash "${def_hostnqn}" "${hash}"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
done
for dhgroup in "ffdhe2048" "ffdhe3072" "ffdhe4096" "ffdhe6144" "ffdhe8192" ; do
@@ -59,12 +58,11 @@ test() {
_set_nvmet_dhgroup "${def_hostnqn}" "${dhgroup}"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
done
_nvmet_target_cleanup
diff --git a/tests/nvme/044 b/tests/nvme/044
index 8e2b4131b969..624b832c0cc6 100755
--- a/tests/nvme/044
+++ b/tests/nvme/044
@@ -49,46 +49,42 @@ test() {
# Step 1: Connect with host authentication only
echo "Test host authentication"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
# Step 2: Connect with host authentication
# and invalid ctrl authentication
echo "Test invalid ctrl authentication (should fail)"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}" \
--dhchap-ctrl-secret "${hostkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
# Step 3: Connect with host authentication
# and valid ctrl authentication
echo "Test valid ctrl authentication"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}" \
--dhchap-ctrl-secret "${ctrlkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
# Step 4: Connect with host authentication
# and invalid ctrl key
echo "Test invalid ctrl key (should fail)"
invkey="DHHC-1:00:Jc/My1o0qtLCWRp+sHhAVafdfaS7YQOMYhk9zSmlatobqB8C:"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}" \
--dhchap-ctrl-secret "${invkey}"
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
diff --git a/tests/nvme/045 b/tests/nvme/045
index f89378836e67..3a5df7c0946c 100755
--- a/tests/nvme/045
+++ b/tests/nvme/045
@@ -51,8 +51,7 @@ test() {
_set_nvmet_dhgroup "${def_hostnqn}" "ffdhe2048"
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
--hostid "${def_hostid}" \
--dhchap-secret "${hostkey}" \
--dhchap-ctrl-secret "${ctrlkey}"
@@ -108,8 +107,7 @@ test() {
rand_io_size="$(_nvme_calc_rand_io_size 4m)"
_run_fio_rand_io --size="${rand_io_size}" --filename="/dev/${nvmedev}n1"
- _nvme_disconnect_subsys "${def_subsysnqn}"
-
+ _nvme_disconnect_subsys
_nvmet_target_cleanup
echo "Test complete"
diff --git a/tests/nvme/047 b/tests/nvme/047
index 162bd3bf70fa..75aad7d06a36 100755
--- a/tests/nvme/047
+++ b/tests/nvme/047
@@ -35,15 +35,15 @@ test() {
rand_io_size="$(_nvme_calc_rand_io_size 4M)"
_run_fio_rand_io --filename="/dev/${nvmedev}n1" --size="${rand_io_size}"
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
- _nvme_connect_subsys "${def_subsysnqn}" \
+ _nvme_connect_subsys \
--nr-write-queues 1 \
--nr-poll-queues 1 || echo FAIL
_run_fio_rand_io --filename="/dev/${nvmedev}n1" --size="${rand_io_size}"
- _nvme_disconnect_subsys "${def_subsysnqn}" >> "$FULL" 2>&1
+ _nvme_disconnect_subsys >> "$FULL" 2>&1
_nvmet_target_cleanup
diff --git a/tests/nvme/048 b/tests/nvme/048
index 44fdffe287c8..79620e0d0879 100755
--- a/tests/nvme/048
+++ b/tests/nvme/048
@@ -99,11 +99,10 @@ test() {
_nvmet_target_setup --blkdev file
if [[ -f "${cfs_path}/attr_qid_max" ]] ; then
- _nvme_connect_subsys "${def_subsysnqn}" \
- --hostnqn "${def_hostnqn}" \
- --hostid "${def_hostid}" \
- --keep-alive-tmo 1 \
- --reconnect-delay 2
+ _nvme_connect_subsys --hostnqn "${def_hostnqn}" \
+ --hostid "${def_hostid}" \
+ --keep-alive-tmo 1 \
+ --reconnect-delay 2
if ! nvmf_wait_for_state "${def_subsysnqn}" "live" ; then
echo FAIL
@@ -112,7 +111,7 @@ test() {
set_qid_max "${def_subsysnqn}" 2 || echo FAIL
fi
- _nvme_disconnect_subsys "${def_subsysnqn}"
+ _nvme_disconnect_subsys
else
SKIP_REASONS+=("missing attr_qid_max feature")
skipped=true
diff --git a/tests/nvme/rc b/tests/nvme/rc
index 1dd1072f9ffb..6493aa30adef 100644
--- a/tests/nvme/rc
+++ b/tests/nvme/rc
@@ -400,15 +400,27 @@ _nvme_disconnect_ctrl() {
}
_nvme_disconnect_subsys() {
- local subsysnqn="$1"
+ local subsysnqn="$def_subsysnqn"
+
+ while [[ $# -gt 0 ]]; do
+ case $1 in
+ --subsysnqn)
+ subsysnqn="$2"
+ shift 2
+ ;;
+ *)
+ echo "WARNING: unknown argument: $1"
+ shift
+ ;;
+ esac
+ done
nvme disconnect --nqn "${subsysnqn}" |& tee -a "$FULL" |
grep -o "disconnected.*"
}
_nvme_connect_subsys() {
- local positional_args=()
- local subsysnqn=""
+ local subsysnqn="$def_subsysnqn"
local hostnqn="$def_hostnqn"
local hostid="$def_hostid"
local hostkey=""
@@ -424,6 +436,10 @@ _nvme_connect_subsys() {
while [[ $# -gt 0 ]]; do
case $1 in
+ --subsysnqn)
+ subsysnqn="$2"
+ shift 2
+ ;;
--hostnqn)
hostnqn="$2"
shift 2
@@ -469,16 +485,12 @@ _nvme_connect_subsys() {
shift 1
;;
*)
- positional_args+=("$1")
+ echo "WARNING: unknown argument: $1"
shift
;;
esac
done
- set -- "${positional_args[@]}"
-
- subsysnqn="$1"
-
ARGS=(--transport "${nvme_trtype}" --nqn "${subsysnqn}")
if [[ "${nvme_trtype}" == "fc" ]] ; then
ARGS+=(--traddr "${def_traddr}" --host-traddr "${def_host_traddr}")
@@ -899,7 +911,7 @@ _nvmet_passthru_target_setup() {
_nvmet_passthru_target_connect() {
local subsys_name=$1
- _nvme_connect_subsys "${subsys_name}" --no-wait || return
+ _nvme_connect_subsys --subsysnqn "${subsys_name}" --no-wait || return
nsdev=$(_find_nvme_passthru_loop_dev "${subsys_name}")
# The following tests can race with the creation
--
2.44.0
next prev parent reply other threads:[~2024-03-26 13:14 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-26 13:13 [PATCH blktests v3 00/20] refactoring and various cleanups/fixes Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 01/20] nvme/rc: silence error on module unload for fc Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 02/20] nvme/rc: silence fcloop cleanup failures Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 03/20] nvme/rc: log error if stale configuration is found Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 04/20] common/xfs: propagate errors from _xfs_run_fio_verify_io Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 05/20] nvme/{012,013,035}: check return value of _xfs_run_fio_verify_io Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 06/20] nvme/rc: use long command line option for nvme Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 07/20] nvme/{014,015,018,019,020,023,024,026,045,046}: " Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 08/20] nvme/rc: connect subsys only support long options Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 09/20] nvme/rc: add nqn/uuid args to target setup/cleanup helper Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 10/20] nvme/rc: remove unused connect options Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 11/20] nvme/rc: do not cleanup external managed loop device Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 12/20] nvme/031: do not open code target setup/cleanup Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 13/20] nvme: drop default trtype argument for _nvmet_connect_subsys Daniel Wagner
2024-03-26 13:13 ` [PATCH blktests v3 14/20] nvme: drop default trtype argument for _nvmet_passthru_target_connect Daniel Wagner
2024-03-26 13:13 ` Daniel Wagner [this message]
2024-03-26 13:13 ` [PATCH blktests v3 16/20] nvme: drop default subsysnqn argument from _nvme_passthru_target_{setup|cleanup} Daniel Wagner
2024-03-29 8:05 ` Shinichiro Kawasaki
2024-03-26 13:13 ` [PATCH blktests v3 17/20] nvme: drop default subsysnqn argument from _nvmet_passthru_target_connect Daniel Wagner
2024-03-26 13:14 ` [PATCH blktests v3 18/20] nvme/{041,042,043,044,045,048}: do not pass default host{nqn|id} to _nvme_connect_subsys Daniel Wagner
2024-03-26 13:14 ` [PATCH blktests v3 19/20] nvme: don't assume namespace id Daniel Wagner
2024-03-26 13:14 ` [PATCH blktests v3 20/20] nvme/028: drop unused nvmedev Daniel Wagner
2024-03-29 8:05 ` [PATCH blktests v3 00/20] refactoring and various cleanups/fixes Shinichiro Kawasaki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240326131402.5092-16-dwagner@suse.de \
--to=dwagner@suse.de \
--cc=chaitanyak@nvidia.com \
--cc=hare@suse.de \
--cc=linux-block@vger.kernel.org \
--cc=linux-nvme@lists.infradead.org \
--cc=shinichiro.kawasaki@wdc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.