* [PATCH blktests] nvme/031: Add test to check controller deletion after setup
@ 2019-09-05 17:43 Logan Gunthorpe
2019-09-05 20:54 ` Sagi Grimberg
2019-09-07 18:19 ` Chaitanya Kulkarni
0 siblings, 2 replies; 6+ messages in thread
From: Logan Gunthorpe @ 2019-09-05 17:43 UTC (permalink / raw)
To: linux-block, linux-nvme, Omar Sandoval; +Cc: Logan Gunthorpe
A number of bug fixes have been submitted to the kernel to
fix bugs when a controller is removed immediately after it is
set up. This new test ensures this doesn't regress.
Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
---
This is reallly just a resend. The patches this tests for are all in
5.3-rc7 or earlier and it passes on said kernel version.
I've rebased this patch onto the latest blktests as of today with no
changes required.
Thanks,
Logan
tests/nvme/031 | 55 ++++++++++++++++++++++++++++++++++++++++++++++
tests/nvme/031.out | 2 ++
2 files changed, 57 insertions(+)
create mode 100755 tests/nvme/031
create mode 100644 tests/nvme/031.out
diff --git a/tests/nvme/031 b/tests/nvme/031
new file mode 100755
index 000000000000..16390dcb380e
--- /dev/null
+++ b/tests/nvme/031
@@ -0,0 +1,55 @@
+#!/bin/bash
+# SPDX-License-Identifier: GPL-3.0+
+# Copyright (C) 2019 Logan Gunthorpe
+#
+# Regression test for the following patches:
+# nvme: fix controller removal race with scan work
+# nvme: fix regression upon hot device removal and insertion
+# nvme-core: Fix extra device_put() call on error path
+# nvmet-loop: Flush nvme_delete_wq when removing the port
+# nvmet: Fix use-after-free bug when a port is removed
+#
+# All these patches fix issues related to deleting a controller
+# immediately after setting it up.
+
+. tests/nvme/rc
+
+DESCRIPTION="test deletion of NVMeOF controllers immediately after setup"
+QUICK=1
+
+requires() {
+ _have_program nvme &&
+ _have_modules loop nvme-loop nvmet &&
+ _have_configfs
+}
+
+test() {
+ local subsys="blktests-subsystem-"
+ local iterations=10
+ local loop_dev
+ local port
+
+ echo "Running ${TEST_NAME}"
+
+ _setup_nvmet
+
+ truncate -s 1G "$TMPDIR/img"
+
+ local loop_dev
+ loop_dev="$(losetup -f --show "$TMPDIR/img")"
+
+ port="$(_create_nvmet_port "loop")"
+
+ for ((i = 0; i < iterations; i++)); do
+ _create_nvmet_subsystem "${subsys}$i" "${loop_dev}"
+ _add_nvmet_subsys_to_port "${port}" "${subsys}$i"
+ nvme connect -t loop -n "${subsys}$i"
+ nvme disconnect -n "${subsys}$i" >> "${FULL}" 2>&1
+ _remove_nvmet_subsystem_from_port "${port}" "${subsys}$i"
+ _remove_nvmet_subsystem "${subsys}$i"
+ done
+
+ _remove_nvmet_port "${port}"
+
+ echo "Test complete"
+}
diff --git a/tests/nvme/031.out b/tests/nvme/031.out
new file mode 100644
index 000000000000..ae902bdd36d4
--- /dev/null
+++ b/tests/nvme/031.out
@@ -0,0 +1,2 @@
+Running nvme/031
+Test complete
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH blktests] nvme/031: Add test to check controller deletion after setup
2019-09-05 17:43 [PATCH blktests] nvme/031: Add test to check controller deletion after setup Logan Gunthorpe
@ 2019-09-05 20:54 ` Sagi Grimberg
2019-09-07 18:19 ` Chaitanya Kulkarni
1 sibling, 0 replies; 6+ messages in thread
From: Sagi Grimberg @ 2019-09-05 20:54 UTC (permalink / raw)
To: Logan Gunthorpe, linux-block, linux-nvme, Omar Sandoval
Looks good Logan,
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH blktests] nvme/031: Add test to check controller deletion after setup
2019-09-05 17:43 [PATCH blktests] nvme/031: Add test to check controller deletion after setup Logan Gunthorpe
2019-09-05 20:54 ` Sagi Grimberg
@ 2019-09-07 18:19 ` Chaitanya Kulkarni
2019-09-09 16:13 ` Logan Gunthorpe
1 sibling, 1 reply; 6+ messages in thread
From: Chaitanya Kulkarni @ 2019-09-07 18:19 UTC (permalink / raw)
To: Logan Gunthorpe, linux-block, linux-nvme, Omar Sandoval
On 09/05/2019 10:44 AM, Logan Gunthorpe wrote:
> A number of bug fixes have been submitted to the kernel to
> fix bugs when a controller is removed immediately after it is
> set up. This new test ensures this doesn't regress.
>
> Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
>
> ---
>
> This is reallly just a resend. The patches this tests for are all in
> 5.3-rc7 or earlier and it passes on said kernel version.
>
> I've rebased this patch onto the latest blktests as of today with no
> changes required.
>
> Thanks,
>
> Logan
>
> tests/nvme/031 | 55 ++++++++++++++++++++++++++++++++++++++++++++++
> tests/nvme/031.out | 2 ++
> 2 files changed, 57 insertions(+)
> create mode 100755 tests/nvme/031
> create mode 100644 tests/nvme/031.out
>
> diff --git a/tests/nvme/031 b/tests/nvme/031
> new file mode 100755
> index 000000000000..16390dcb380e
> --- /dev/null
> +++ b/tests/nvme/031
> @@ -0,0 +1,55 @@
> +#!/bin/bash
> +# SPDX-License-Identifier: GPL-3.0+
> +# Copyright (C) 2019 Logan Gunthorpe
> +#
> +# Regression test for the following patches:
> +# nvme: fix controller removal race with scan work
> +# nvme: fix regression upon hot device removal and insertion
> +# nvme-core: Fix extra device_put() call on error path
> +# nvmet-loop: Flush nvme_delete_wq when removing the port
> +# nvmet: Fix use-after-free bug when a port is removed
> +#
> +# All these patches fix issues related to deleting a controller
> +# immediately after setting it up.
> +
> +. tests/nvme/rc
> +
> +DESCRIPTION="test deletion of NVMeOF controllers immediately after setup"
> +QUICK=1
> +
> +requires() {
> + _have_program nvme &&
> + _have_modules loop nvme-loop nvmet &&
> + _have_configfs
> +}
> +
> +test() {
> + local subsys="blktests-subsystem-"
> + local iterations=10
> + local loop_dev
> + local port
> +
> + echo "Running ${TEST_NAME}"
> +
> + _setup_nvmet
> +
> + truncate -s 1G "$TMPDIR/img"
> +
> + local loop_dev
Duplicate declaration of the local variable ?
> + loop_dev="$(losetup -f --show "$TMPDIR/img")"
> +
> + port="$(_create_nvmet_port "loop")"
> +
> + for ((i = 0; i < iterations; i++)); do
> + _create_nvmet_subsystem "${subsys}$i" "${loop_dev}"
> + _add_nvmet_subsys_to_port "${port}" "${subsys}$i"
> + nvme connect -t loop -n "${subsys}$i"
> + nvme disconnect -n "${subsys}$i" >> "${FULL}" 2>&1
> + _remove_nvmet_subsystem_from_port "${port}" "${subsys}$i"
> + _remove_nvmet_subsystem "${subsys}$i"
> + done
> +
> + _remove_nvmet_port "${port}"
> +
> + echo "Test complete"
> +}
> diff --git a/tests/nvme/031.out b/tests/nvme/031.out
> new file mode 100644
> index 000000000000..ae902bdd36d4
> --- /dev/null
> +++ b/tests/nvme/031.out
> @@ -0,0 +1,2 @@
> +Running nvme/031
> +Test complete
> --
> 2.20.1
>
> _______________________________________________
> Linux-nvme mailing list
> Linux-nvme@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-nvme
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH blktests] nvme/031: Add test to check controller deletion after setup
2019-09-07 18:19 ` Chaitanya Kulkarni
@ 2019-09-09 16:13 ` Logan Gunthorpe
2019-09-09 17:06 ` Chaitanya Kulkarni
0 siblings, 1 reply; 6+ messages in thread
From: Logan Gunthorpe @ 2019-09-09 16:13 UTC (permalink / raw)
To: Chaitanya Kulkarni, linux-block, linux-nvme, Omar Sandoval
On 2019-09-07 12:19 p.m., Chaitanya Kulkarni wrote:
> On 09/05/2019 10:44 AM, Logan Gunthorpe wrote:
>> A number of bug fixes have been submitted to the kernel to
>> fix bugs when a controller is removed immediately after it is
>> set up. This new test ensures this doesn't regress.
>>
>> Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
>>
>> ---
>>
>> This is reallly just a resend. The patches this tests for are all in
>> 5.3-rc7 or earlier and it passes on said kernel version.
>>
>> I've rebased this patch onto the latest blktests as of today with no
>> changes required.
>>
>> Thanks,
>>
>> Logan
>>
>> tests/nvme/031 | 55 ++++++++++++++++++++++++++++++++++++++++++++++
>> tests/nvme/031.out | 2 ++
>> 2 files changed, 57 insertions(+)
>> create mode 100755 tests/nvme/031
>> create mode 100644 tests/nvme/031.out
>>
>> diff --git a/tests/nvme/031 b/tests/nvme/031
>> new file mode 100755
>> index 000000000000..16390dcb380e
>> --- /dev/null
>> +++ b/tests/nvme/031
>> @@ -0,0 +1,55 @@
>> +#!/bin/bash
>> +# SPDX-License-Identifier: GPL-3.0+
>> +# Copyright (C) 2019 Logan Gunthorpe
>> +#
>> +# Regression test for the following patches:
>> +# nvme: fix controller removal race with scan work
>> +# nvme: fix regression upon hot device removal and insertion
>> +# nvme-core: Fix extra device_put() call on error path
>> +# nvmet-loop: Flush nvme_delete_wq when removing the port
>> +# nvmet: Fix use-after-free bug when a port is removed
>> +#
>> +# All these patches fix issues related to deleting a controller
>> +# immediately after setting it up.
>> +
>> +. tests/nvme/rc
>> +
>> +DESCRIPTION="test deletion of NVMeOF controllers immediately after setup"
>> +QUICK=1
>> +
>> +requires() {
>> + _have_program nvme &&
>> + _have_modules loop nvme-loop nvmet &&
>> + _have_configfs
>> +}
>> +
>> +test() {
>> + local subsys="blktests-subsystem-"
>> + local iterations=10
>> + local loop_dev
>> + local port
>> +
>> + echo "Running ${TEST_NAME}"
>> +
>> + _setup_nvmet
>> +
>> + truncate -s 1G "$TMPDIR/img"
>> +
>> + local loop_dev
> Duplicate declaration of the local variable ?
Oops, yes, nice catch. I'll send an updated patch later this week.
Logan
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH blktests] nvme/031: Add test to check controller deletion after setup
2019-09-09 16:13 ` Logan Gunthorpe
@ 2019-09-09 17:06 ` Chaitanya Kulkarni
0 siblings, 0 replies; 6+ messages in thread
From: Chaitanya Kulkarni @ 2019-09-09 17:06 UTC (permalink / raw)
To: Logan Gunthorpe, linux-block, linux-nvme, Omar Sandoval
Hi,
Thanks for the testcase Logan, once passthru patches are applied
feel free to post the blktests for the passthru on the mainling list,
I'll be happy review.
On 09/09/2019 09:14 AM, Logan Gunthorpe wrote:
>
> On 2019-09-07 12:19 p.m., Chaitanya Kulkarni wrote:
>> >On 09/05/2019 10:44 AM, Logan Gunthorpe wrote:
>>> >>A number of bug fixes have been submitted to the kernel to
>>> >>fix bugs when a controller is removed immediately after it is
>>> >>set up. This new test ensures this doesn't regress.
>>> >>
>>> >>Signed-off-by: Logan Gunthorpe<logang@deltatee.com>
>>> >>
>>> >>---
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH blktests] nvme/031: Add test to check controller deletion after setup
@ 2019-07-31 23:47 Logan Gunthorpe
0 siblings, 0 replies; 6+ messages in thread
From: Logan Gunthorpe @ 2019-07-31 23:47 UTC (permalink / raw)
To: linux-block, linux-nvme, Omar Sandoval; +Cc: Logan Gunthorpe
A number of bug fixes have been submitted to the kernel to
fix bugs when a controller is removed immediately after it is
set up. This new test ensures this doesn't regress.
Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
---
All of the fixes required for this test to pass have been on the
list for a couple weeks, but not all of them have been merged.
I'm not sure what the policy is for blktests for patches that
aren't yet upstream but this patch is here for when it makes
sense to pick it up.
tests/nvme/031 | 55 ++++++++++++++++++++++++++++++++++++++++++++++
tests/nvme/031.out | 2 ++
2 files changed, 57 insertions(+)
create mode 100755 tests/nvme/031
create mode 100644 tests/nvme/031.out
diff --git a/tests/nvme/031 b/tests/nvme/031
new file mode 100755
index 000000000000..16390dcb380e
--- /dev/null
+++ b/tests/nvme/031
@@ -0,0 +1,55 @@
+#!/bin/bash
+# SPDX-License-Identifier: GPL-3.0+
+# Copyright (C) 2019 Logan Gunthorpe
+#
+# Regression test for the following patches:
+# nvme: fix controller removal race with scan work
+# nvme: fix regression upon hot device removal and insertion
+# nvme-core: Fix extra device_put() call on error path
+# nvmet-loop: Flush nvme_delete_wq when removing the port
+# nvmet: Fix use-after-free bug when a port is removed
+#
+# All these patches fix issues related to deleting a controller
+# immediately after setting it up.
+
+. tests/nvme/rc
+
+DESCRIPTION="test deletion of NVMeOF controllers immediately after setup"
+QUICK=1
+
+requires() {
+ _have_program nvme &&
+ _have_modules loop nvme-loop nvmet &&
+ _have_configfs
+}
+
+test() {
+ local subsys="blktests-subsystem-"
+ local iterations=10
+ local loop_dev
+ local port
+
+ echo "Running ${TEST_NAME}"
+
+ _setup_nvmet
+
+ truncate -s 1G "$TMPDIR/img"
+
+ local loop_dev
+ loop_dev="$(losetup -f --show "$TMPDIR/img")"
+
+ port="$(_create_nvmet_port "loop")"
+
+ for ((i = 0; i < iterations; i++)); do
+ _create_nvmet_subsystem "${subsys}$i" "${loop_dev}"
+ _add_nvmet_subsys_to_port "${port}" "${subsys}$i"
+ nvme connect -t loop -n "${subsys}$i"
+ nvme disconnect -n "${subsys}$i" >> "${FULL}" 2>&1
+ _remove_nvmet_subsystem_from_port "${port}" "${subsys}$i"
+ _remove_nvmet_subsystem "${subsys}$i"
+ done
+
+ _remove_nvmet_port "${port}"
+
+ echo "Test complete"
+}
diff --git a/tests/nvme/031.out b/tests/nvme/031.out
new file mode 100644
index 000000000000..ae902bdd36d4
--- /dev/null
+++ b/tests/nvme/031.out
@@ -0,0 +1,2 @@
+Running nvme/031
+Test complete
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-09-09 17:07 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-05 17:43 [PATCH blktests] nvme/031: Add test to check controller deletion after setup Logan Gunthorpe
2019-09-05 20:54 ` Sagi Grimberg
2019-09-07 18:19 ` Chaitanya Kulkarni
2019-09-09 16:13 ` Logan Gunthorpe
2019-09-09 17:06 ` Chaitanya Kulkarni
-- strict thread matches above, loose matches on Subject: below --
2019-07-31 23:47 Logan Gunthorpe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).