* [PATCH blktests] nvme/033: Remove volatile output
@ 2022-06-04 16:06 Xiao Yang
2022-06-05 23:22 ` Chaitanya Kulkarni
2022-06-08 4:21 ` Shinichiro Kawasaki
0 siblings, 2 replies; 3+ messages in thread
From: Xiao Yang @ 2022-06-04 16:06 UTC (permalink / raw)
To: shinichiro.kawasaki; +Cc: linux-block, logang, Xiao Yang
The output of _nvme_discover() will be changed according to
${nvme_trtype} and the number of NICs. For example: nvme/033
with nvme_trtype=tcp and two NICs got the following error:
------------------------------------------------
Running nvme/033
-Discovery Log Number of Records 1, Generation counter X
+Discovery Log Number of Records 2, Generation counter X
=====Discovery Log Entry 0======
-trtype: loop
+trtype: tcp
+subnqn: nqn.2014-08.org.nvmexpress.discovery
+=====Discovery Log Entry 1======
+trtype: tcp
subnqn: blktests-subsystem-1
NQN:blktests-subsystem-1 disconnected 1 controller(s)
Test complete
------------------------------------------------
Remove volatile output to fix the issue.
Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
---
tests/nvme/033 | 2 --
tests/nvme/033.out | 4 ----
2 files changed, 6 deletions(-)
diff --git a/tests/nvme/033 b/tests/nvme/033
index c6a3f7f..90aee81 100755
--- a/tests/nvme/033
+++ b/tests/nvme/033
@@ -54,8 +54,6 @@ test_device() {
_setup_nvmet
port=$(_nvmet_passthru_target_setup "${subsys}")
- _nvme_discover "${nvme_trtype}" | _filter_discovery
-
nsdev=$(_nvmet_passthru_target_connect "${nvme_trtype}" "${subsys}")
compare_dev_info "${nsdev}"
diff --git a/tests/nvme/033.out b/tests/nvme/033.out
index 6f45a1d..eb508be 100644
--- a/tests/nvme/033.out
+++ b/tests/nvme/033.out
@@ -1,7 +1,3 @@
Running nvme/033
-Discovery Log Number of Records 1, Generation counter X
-=====Discovery Log Entry 0======
-trtype: loop
-subnqn: blktests-subsystem-1
NQN:blktests-subsystem-1 disconnected 1 controller(s)
Test complete
--
2.23.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH blktests] nvme/033: Remove volatile output
2022-06-04 16:06 [PATCH blktests] nvme/033: Remove volatile output Xiao Yang
@ 2022-06-05 23:22 ` Chaitanya Kulkarni
2022-06-08 4:21 ` Shinichiro Kawasaki
1 sibling, 0 replies; 3+ messages in thread
From: Chaitanya Kulkarni @ 2022-06-05 23:22 UTC (permalink / raw)
To: Xiao Yang, shinichiro.kawasaki; +Cc: linux-block, logang
On 6/4/22 09:06, Xiao Yang wrote:
> The output of _nvme_discover() will be changed according to
> ${nvme_trtype} and the number of NICs. For example: nvme/033
> with nvme_trtype=tcp and two NICs got the following error:
> ------------------------------------------------
> Running nvme/033
> -Discovery Log Number of Records 1, Generation counter X
> +Discovery Log Number of Records 2, Generation counter X
> =====Discovery Log Entry 0======
> -trtype: loop
> +trtype: tcp
> +subnqn: nqn.2014-08.org.nvmexpress.discovery
> +=====Discovery Log Entry 1======
> +trtype: tcp
> subnqn: blktests-subsystem-1
> NQN:blktests-subsystem-1 disconnected 1 controller(s)
> Test complete
> ------------------------------------------------
>
> Remove volatile output to fix the issue.
>
> Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
> ---
> tests/nvme/033 | 2 --
> tests/nvme/033.out | 4 ----
> 2 files changed, 6 deletions(-)
>
Looks good.
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
-ck
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH blktests] nvme/033: Remove volatile output
2022-06-04 16:06 [PATCH blktests] nvme/033: Remove volatile output Xiao Yang
2022-06-05 23:22 ` Chaitanya Kulkarni
@ 2022-06-08 4:21 ` Shinichiro Kawasaki
1 sibling, 0 replies; 3+ messages in thread
From: Shinichiro Kawasaki @ 2022-06-08 4:21 UTC (permalink / raw)
To: Xiao Yang; +Cc: linux-block, logang
On Jun 05, 2022 / 00:06, Xiao Yang wrote:
> The output of _nvme_discover() will be changed according to
> ${nvme_trtype} and the number of NICs. For example: nvme/033
> with nvme_trtype=tcp and two NICs got the following error:
> ------------------------------------------------
> Running nvme/033
> -Discovery Log Number of Records 1, Generation counter X
> +Discovery Log Number of Records 2, Generation counter X
> =====Discovery Log Entry 0======
> -trtype: loop
> +trtype: tcp
> +subnqn: nqn.2014-08.org.nvmexpress.discovery
> +=====Discovery Log Entry 1======
> +trtype: tcp
> subnqn: blktests-subsystem-1
> NQN:blktests-subsystem-1 disconnected 1 controller(s)
> Test complete
> ------------------------------------------------
>
> Remove volatile output to fix the issue.
>
> Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com>
Thanks, applied!
--
Shin'ichiro Kawasaki
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-06-08 6:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-04 16:06 [PATCH blktests] nvme/033: Remove volatile output Xiao Yang
2022-06-05 23:22 ` Chaitanya Kulkarni
2022-06-08 4:21 ` Shinichiro Kawasaki
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.