mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect
@ 2026-09-22 20:07 Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 1/3] nvme-tcp: store and use the caller's network namespace Surabhi Gogte
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Surabhi Gogte @ 2026-09-22 20:07 UTC (permalink / raw)
  To: Keith Busch, Jens Axboe, Christoph Hellwig, Sagi Grimberg
  Cc: linux-nvme, linux-kernel, mkhalfella, randyj, adailey, Surabhi Gogte

Similar to commit 2a8513091d2f ("nvme-rdma: parallelize I/O queue
allocation and startup"), I/O queue allocation and start steps
(including authentication) can be parallelized for nvme-tcp as well.

Patch 1 fixes a pre-existing bug: nvme_tcp_alloc_queue() used
current->nsproxy->net_ns to create sockets during initial connect
while it used kernel's init_net for reconnect and error recovery
paths. The fix captures the caller's network namespace at controller
allocation time.

Patch 2 is a preparatory refactor: it splits the queue-count
negotiation and TLS PSK checks into helpers so that the individual
steps can be called directly from nvme_tcp_configure_io_queues(). No
functional change.

Patch 3 has the async implementation: it fans the per-queue alloc and
start out over an async domain, serializes the per-cpu queue-count in
nvme_tcp_set_queue_io_cpu() that can now run concurrently, and uses
ctrl->net for all socket creation.

Testing on a 64-core host with 64 IO-queues shows nvme-tcp connection
time reduced from 61ms to 11ms.

blktests - nvme suite, TCP transport (nvme_trtype=tcp)

Tested with:
- Ubuntu 22.04 LTS (Jammy Jellyfish), x86_64, KubeVirt VM
- nvme-cli v2.13 / libnvme v1.14 (built with libkeyutils)
- ktls-utils tag ktls-utils-1.4.0 (c63c04b) (tlshd) linked
against gnutls 3.8.5

---
v4:
  - Moved conditional from nvme_tcp_configure_io_queues() to
    nvme_tcp_setup_io_queues().

v3:
  - Added patch 1 fixing a network namespace bug: reconnect and error
    recovery paths were creating sockets in init_net instead of the
    controller's namespace. Patch 3 now uses ctrl->net (from patch 1)
    instead of hardcoding init_net.
  - Patches renumbered accordingly (v2 patch 1->2, patch 2->3).

v2:
  - Resending the entire series to fix missing numbering in the
    original patch 2 subject line. No functional code changes.

v1: https://lore.kernel.org/20260824222939.301887-1-sgogte@purestorage.com/
v2: https://lore.kernel.org/20260824225648.316962-1-sgogte@purestorage.com/
v3: https://lore.kernel.org/20260910202812.1642832-1-sgogte@purestorage.com/
---

Surabhi Gogte (3):
  nvme-tcp: store and use the caller's network namespace
  nvme-tcp: refactor I/O queue setup path
  nvme-tcp: parallelize I/O queue allocation and startup

 drivers/nvme/host/tcp.c | 139 +++++++++++++++++++++++++++-------------
 1 file changed, 96 insertions(+), 43 deletions(-)

-- 
2.55.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v4 1/3] nvme-tcp: store and use the caller's network namespace
  2026-09-22 20:07 [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Surabhi Gogte
@ 2026-09-22 20:07 ` Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 2/3] nvme-tcp: refactor I/O queue setup path Surabhi Gogte
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Surabhi Gogte @ 2026-09-22 20:07 UTC (permalink / raw)
  To: Keith Busch, Jens Axboe, Christoph Hellwig, Sagi Grimberg
  Cc: linux-nvme, linux-kernel, mkhalfella, randyj, adailey,
	Surabhi Gogte, Hannes Reinecke

nvme_tcp_alloc_queue() creates each socket using
current->nsproxy->net_ns, which is correct for the initial connect that
runs in userspace context. However, the reconnect and error recovery
paths run from a workqueue where current netns is a kernel thread in
init_net, so sockets created during reconnect end up in a different
namespace than the one the controller was originally connected from.

Fix this by capturing the caller's network namespace at controller
allocation time. Use ctrl->net for all socket creation so that every
queue, on initial connect, reset, or reconnect, is created in the same
namespace.

Fixes: 3f2304f8c6d6 ("nvme-tcp: add NVMe over TCP host driver")
Signed-off-by: Surabhi Gogte <sgogte@purestorage.com>
Reviewed-by: Hannes Reinecke <hare@kernel.org>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
---
 drivers/nvme/host/tcp.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
index 921934028e0b..f916bcca304f 100644
--- a/drivers/nvme/host/tcp.c
+++ b/drivers/nvme/host/tcp.c
@@ -171,6 +171,7 @@ struct nvme_tcp_ctrl {
 	struct delayed_work	connect_work;
 	struct nvme_tcp_request async_req;
 	u32			io_queues[HCTX_MAX_TYPES];
+	struct net		*net;
 };
 
 static struct workqueue_struct *nvme_tcp_wq;
@@ -1858,7 +1859,7 @@ static int nvme_tcp_alloc_queue(struct nvme_ctrl *nctrl, int qid,
 		queue->cmnd_capsule_len = sizeof(struct nvme_command) +
 						NVME_TCP_ADMIN_CCSZ;
 
-	ret = sock_create_kern(current->nsproxy->net_ns,
+	ret = sock_create_kern(ctrl->net,
 			ctrl->addr.ss_family, SOCK_STREAM,
 			IPPROTO_TCP, &queue->sock);
 	if (ret) {
@@ -2650,6 +2651,7 @@ static void nvme_tcp_free_ctrl(struct nvme_ctrl *nctrl)
 
 	nvmf_free_options(nctrl->opts);
 free_ctrl:
+	put_net(ctrl->net);
 	kfree(ctrl->queues);
 	kfree(ctrl);
 }
@@ -3048,12 +3050,15 @@ static struct nvme_tcp_ctrl *nvme_tcp_alloc_ctrl(struct device *dev,
 		goto out_free_ctrl;
 	}
 
+	ctrl->net = get_net(current->nsproxy->net_ns);
+
 	ret = nvme_init_ctrl(&ctrl->ctrl, dev, &nvme_tcp_ctrl_ops, 0);
 	if (ret)
 		goto out_kfree_queues;
 
 	return ctrl;
 out_kfree_queues:
+	put_net(ctrl->net);
 	kfree(ctrl->queues);
 out_free_ctrl:
 	kfree(ctrl);
-- 
2.55.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v4 2/3] nvme-tcp: refactor I/O queue setup path
  2026-09-22 20:07 [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 1/3] nvme-tcp: store and use the caller's network namespace Surabhi Gogte
@ 2026-09-22 20:07 ` Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup Surabhi Gogte
  2026-10-13 15:33 ` [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Keith Busch
  3 siblings, 0 replies; 6+ messages in thread
From: Surabhi Gogte @ 2026-09-22 20:07 UTC (permalink / raw)
  To: Keith Busch, Jens Axboe, Christoph Hellwig, Sagi Grimberg
  Cc: linux-nvme, linux-kernel, mkhalfella, randyj, adailey,
	Surabhi Gogte, Hannes Reinecke

Split the I/O queue setup helpers apart so that the individual steps can
be called directly from nvme_tcp_configure_io_queues():

- Queue count negotiation moves out of nvme_tcp_alloc_io_queues() into a
  new nvme_tcp_set_io_queue_count(), leaving the allocator with just the
  per-queue allocation loop.
- TLS PSK validation moves out of __nvme_tcp_alloc_io_queues() into a
  new nvme_tcp_tls_check_psk().
- nvme_tcp_configure_io_queues() now calls the three steps directly in
  the same order as before.

Signed-off-by: Surabhi Gogte <sgogte@purestorage.com>
Reviewed-by: Hannes Reinecke <hare@kernel.org>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
---
 drivers/nvme/host/tcp.c | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
index f916bcca304f..9308feba8a74 100644
--- a/drivers/nvme/host/tcp.c
+++ b/drivers/nvme/host/tcp.c
@@ -2184,10 +2184,8 @@ static int nvme_tcp_alloc_admin_queue(struct nvme_ctrl *ctrl)
 	return ret;
 }
 
-static int __nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
+static int nvme_tcp_tls_check_psk(struct nvme_ctrl *ctrl)
 {
-	int i, ret;
-
 	if (nvme_tcp_tls_configured(ctrl)) {
 		if (ctrl->opts->concat) {
 			/*
@@ -2209,6 +2207,13 @@ static int __nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
 		}
 	}
 
+	return 0;
+}
+
+static int __nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
+{
+	int i, ret;
+
 	for (i = 1; i < ctrl->queue_count; i++) {
 		ret = nvme_tcp_alloc_queue(ctrl, i,
 				ctrl->tls_pskid);
@@ -2225,7 +2230,7 @@ static int __nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
 	return ret;
 }
 
-static int nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
+static int nvme_tcp_set_io_queue_count(struct nvme_ctrl *ctrl)
 {
 	unsigned int nr_io_queues;
 	int ret;
@@ -2247,14 +2252,22 @@ static int nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
 
 	nvmf_set_io_queues(ctrl->opts, nr_io_queues,
 			   to_tcp_ctrl(ctrl)->io_queues);
-	return __nvme_tcp_alloc_io_queues(ctrl);
+	return 0;
 }
 
 static int nvme_tcp_configure_io_queues(struct nvme_ctrl *ctrl, bool new)
 {
 	int ret, nr_queues;
 
-	ret = nvme_tcp_alloc_io_queues(ctrl);
+	ret = nvme_tcp_set_io_queue_count(ctrl);
+	if (ret)
+		return ret;
+
+	ret = nvme_tcp_tls_check_psk(ctrl);
+	if (ret)
+		return ret;
+
+	ret = __nvme_tcp_alloc_io_queues(ctrl);
 	if (ret)
 		return ret;
 
-- 
2.55.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup
  2026-09-22 20:07 [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 1/3] nvme-tcp: store and use the caller's network namespace Surabhi Gogte
  2026-09-22 20:07 ` [PATCH v4 2/3] nvme-tcp: refactor I/O queue setup path Surabhi Gogte
@ 2026-09-22 20:07 ` Surabhi Gogte
  2026-10-18 12:25   ` Shin'ichiro Kawasaki
  2026-10-13 15:33 ` [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Keith Busch
  3 siblings, 1 reply; 6+ messages in thread
From: Surabhi Gogte @ 2026-09-22 20:07 UTC (permalink / raw)
  To: Keith Busch, Jens Axboe, Christoph Hellwig, Sagi Grimberg
  Cc: linux-nvme, linux-kernel, mkhalfella, randyj, adailey,
	Surabhi Gogte, Hannes Reinecke

Similar to commit 2a8513091d2f ("nvme-rdma: parallelize I/O queue
allocation and startup"), refactor nvme tcp I/O queue setup to use async
API, combining allocation and startup into a single parallel operation
per queue. This reduces connection and reconnection setup time when
there are delays in establishing connections, which is especially
important for high-core-count hosts.

Key changes:
- Use async API to facilitate parallel calls for io queue setup.
- Add nvme_tcp_queue_setup_ctx for propagating errors from async
  workers.
- Remove nvme_tcp_start_io_queues() and __nvme_tcp_alloc_io_queues();
  their logic is folded into nvme_tcp_setup_io_queues() and
  nvme_tcp_configure_io_queues().
- Allocate the io tag set before the queues so that the queue range is
  known, and only set up the reconnect grow case if the queue count
  actually increased.
- Serialize the cpu scan and claim in nvme_tcp_set_queue_io_cpu() with a
  spinlock, as concurrent callers would otherwise select the same cpu.
  The per-cpu counters no longer need to be atomics.

Testing on a 64-core host with 64 IO-queues shows nvme-tcp connection
time reduced from 61ms to 11ms.

Signed-off-by: Surabhi Gogte <sgogte@purestorage.com>
Reviewed-by: Hannes Reinecke <hare@kernel.org>
---
 drivers/nvme/host/tcp.c | 119 ++++++++++++++++++++++++++--------------
 1 file changed, 77 insertions(+), 42 deletions(-)

diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
index 9308feba8a74..ec678ed07af8 100644
--- a/drivers/nvme/host/tcp.c
+++ b/drivers/nvme/host/tcp.c
@@ -7,6 +7,7 @@
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/slab.h>
+#include <linux/async.h>
 #include <linux/err.h>
 #include <linux/crc32.h>
 #include <linux/nvme-tcp.h>
@@ -54,7 +55,8 @@ MODULE_PARM_DESC(tls_handshake_timeout,
 		 "nvme TLS handshake timeout in seconds (default 10)");
 #endif
 
-static atomic_t nvme_tcp_cpu_queues[NR_CPUS];
+static int nvme_tcp_cpu_queues[NR_CPUS];
+static DEFINE_SPINLOCK(nvme_tcp_cpu_queues_lock);
 
 enum nvme_tcp_send_state {
 	NVME_TCP_SEND_CMD_PDU = 0,
@@ -154,6 +156,12 @@ struct nvme_tcp_queue {
 static DEFINE_MUTEX(nvme_tcp_ctrl_mutex);
 static LIST_HEAD_GUARDED(nvme_tcp_ctrl_list, nvme_tcp_ctrl_mutex);
 
+struct nvme_tcp_queue_setup_ctx {
+	struct nvme_ctrl	*ctrl;
+	int			qid;
+	int			*err;
+};
+
 struct nvme_tcp_ctrl {
 	/* read only in the hot path */
 	struct nvme_tcp_queue	*queues;
@@ -1731,9 +1739,10 @@ static void nvme_tcp_set_queue_io_cpu(struct nvme_tcp_queue *queue)
 		goto out;
 
 	/* Search for the least used cpu from the mq_map */
+	spin_lock(&nvme_tcp_cpu_queues_lock);
 	io_cpu = WORK_CPU_UNBOUND;
 	for_each_online_cpu(cpu) {
-		int num_queues = atomic_read(&nvme_tcp_cpu_queues[cpu]);
+		int num_queues = nvme_tcp_cpu_queues[cpu];
 
 		if (mq_map[cpu] != qid)
 			continue;
@@ -1744,9 +1753,10 @@ static void nvme_tcp_set_queue_io_cpu(struct nvme_tcp_queue *queue)
 	}
 	if (io_cpu != WORK_CPU_UNBOUND) {
 		queue->io_cpu = io_cpu;
-		atomic_inc(&nvme_tcp_cpu_queues[io_cpu]);
+		nvme_tcp_cpu_queues[io_cpu]++;
 		set_bit(NVME_TCP_Q_IO_CPU_SET, &queue->flags);
 	}
+	spin_unlock(&nvme_tcp_cpu_queues_lock);
 out:
 	dev_dbg(ctrl->ctrl.device, "queue %d: using cpu %d\n",
 		qid, queue->io_cpu);
@@ -2023,8 +2033,11 @@ static void nvme_tcp_stop_queue_nowait(struct nvme_ctrl *nctrl, int qid)
 	if (!test_bit(NVME_TCP_Q_ALLOCATED, &queue->flags))
 		return;
 
-	if (test_and_clear_bit(NVME_TCP_Q_IO_CPU_SET, &queue->flags))
-		atomic_dec(&nvme_tcp_cpu_queues[queue->io_cpu]);
+	if (test_and_clear_bit(NVME_TCP_Q_IO_CPU_SET, &queue->flags)) {
+		spin_lock(&nvme_tcp_cpu_queues_lock);
+		nvme_tcp_cpu_queues[queue->io_cpu]--;
+		spin_unlock(&nvme_tcp_cpu_queues_lock);
+	}
 
 	mutex_lock(&queue->queue_lock);
 	if (test_and_clear_bit(NVME_TCP_Q_LIVE, &queue->flags))
@@ -2131,25 +2144,6 @@ static void nvme_tcp_stop_io_queues(struct nvme_ctrl *ctrl)
 		nvme_tcp_wait_queue(ctrl, i);
 }
 
-static int nvme_tcp_start_io_queues(struct nvme_ctrl *ctrl,
-				    int first, int last)
-{
-	int i, ret;
-
-	for (i = first; i < last; i++) {
-		ret = nvme_tcp_start_queue(ctrl, i);
-		if (ret)
-			goto out_stop_queues;
-	}
-
-	return 0;
-
-out_stop_queues:
-	for (i--; i >= first; i--)
-		nvme_tcp_stop_queue(ctrl, i);
-	return ret;
-}
-
 static int nvme_tcp_alloc_admin_queue(struct nvme_ctrl *ctrl)
 {
 	int ret;
@@ -2210,22 +2204,67 @@ static int nvme_tcp_tls_check_psk(struct nvme_ctrl *ctrl)
 	return 0;
 }
 
-static int __nvme_tcp_alloc_io_queues(struct nvme_ctrl *ctrl)
+static void nvme_tcp_setup_queue_async(void *data, async_cookie_t cookie)
 {
-	int i, ret;
+	struct nvme_tcp_queue_setup_ctx *ctx = data;
+	struct nvme_ctrl *ctrl = ctx->ctrl;
+	int ret;
 
-	for (i = 1; i < ctrl->queue_count; i++) {
-		ret = nvme_tcp_alloc_queue(ctrl, i,
-				ctrl->tls_pskid);
-		if (ret)
-			goto out_free_queues;
+	ret = nvme_tcp_alloc_queue(ctrl, ctx->qid, ctrl->tls_pskid);
+	if (ret)
+		goto out_err;
+
+	ret = nvme_tcp_start_queue(ctrl, ctx->qid);
+	if (ret)
+		goto out_err;
+
+	return;
+
+out_err:
+	WRITE_ONCE(*ctx->err, ret);
+}
+
+static int nvme_tcp_setup_io_queues(struct nvme_ctrl *ctrl, unsigned int first,
+		unsigned int last)
+{
+	ASYNC_DOMAIN_EXCLUSIVE(queue_domain);
+	struct nvme_tcp_queue_setup_ctx *ctxs;
+	int nr_queues = last - first;
+	int err = 0, i, ret;
+
+	if (first >= last)
+		return 0;
+
+	ctxs = kmalloc_objs(*ctxs, nr_queues);
+	if (!ctxs)
+		return -ENOMEM;
+
+	for (i = 0; i < nr_queues; i++) {
+		ctxs[i].ctrl = ctrl;
+		ctxs[i].qid = first + i;
+		ctxs[i].err = &err;
+		async_schedule_domain(nvme_tcp_setup_queue_async, &ctxs[i],
+				&queue_domain);
 	}
 
+	async_synchronize_full_domain(&queue_domain);
+	kfree(ctxs);
+
+	ret = READ_ONCE(err);
+	if (ret)
+		goto out_free_queues;
+
 	return 0;
 
 out_free_queues:
-	for (i--; i >= 1; i--)
-		nvme_tcp_free_queue(ctrl, i);
+	for (i = first; i < last; i++) {
+		struct nvme_tcp_queue *queue = &to_tcp_ctrl(ctrl)->queues[i];
+
+		if (test_bit(NVME_TCP_Q_LIVE, &queue->flags))
+			nvme_tcp_stop_queue(ctrl, i);
+		if (test_bit(NVME_TCP_Q_ALLOCATED, &queue->flags))
+			nvme_tcp_free_queue(ctrl, i);
+	}
 
 	return ret;
 }
@@ -2267,10 +2306,6 @@ static int nvme_tcp_configure_io_queues(struct nvme_ctrl *ctrl, bool new)
 	if (ret)
 		return ret;
 
-	ret = __nvme_tcp_alloc_io_queues(ctrl);
-	if (ret)
-		return ret;
-
 	if (new) {
 		ret = nvme_alloc_io_tag_set(ctrl, &to_tcp_ctrl(ctrl)->tag_set,
 				&nvme_tcp_mq_ops,
@@ -2281,12 +2316,12 @@ static int nvme_tcp_configure_io_queues(struct nvme_ctrl *ctrl, bool new)
 	}
 
 	/*
-	 * Only start IO queues for which we have allocated the tagset
+	 * Only setup IO queues for which we have allocated the tagset
 	 * and limited it to the available queues. On reconnects, the
 	 * queue number might have changed.
 	 */
 	nr_queues = min(ctrl->tagset->nr_hw_queues + 1, ctrl->queue_count);
-	ret = nvme_tcp_start_io_queues(ctrl, 1, nr_queues);
+	ret = nvme_tcp_setup_io_queues(ctrl, 1, nr_queues);
 	if (ret)
 		goto out_cleanup_connect_q;
 
@@ -2310,9 +2345,9 @@ static int nvme_tcp_configure_io_queues(struct nvme_ctrl *ctrl, bool new)
 
 	/*
 	 * If the number of queues has increased (reconnect case)
-	 * start all new queues now.
+	 * setup all new queues now.
 	 */
-	ret = nvme_tcp_start_io_queues(ctrl, nr_queues,
+	ret = nvme_tcp_setup_io_queues(ctrl, nr_queues,
 				       ctrl->tagset->nr_hw_queues + 1);
 	if (ret)
 		goto out_wait_freeze_timed_out;
@@ -3157,7 +3192,7 @@ static int __init nvme_tcp_init_module(void)
 		return -ENOMEM;
 
 	for_each_possible_cpu(cpu)
-		atomic_set(&nvme_tcp_cpu_queues[cpu], 0);
+		nvme_tcp_cpu_queues[cpu] = 0;
 
 	nvmf_register_transport(&nvme_tcp_transport);
 	return 0;
-- 
2.55.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect
  2026-09-22 20:07 [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Surabhi Gogte
                   ` (2 preceding siblings ...)
  2026-09-22 20:07 ` [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup Surabhi Gogte
@ 2026-10-13 15:33 ` Keith Busch
  3 siblings, 0 replies; 6+ messages in thread
From: Keith Busch @ 2026-10-13 15:33 UTC (permalink / raw)
  To: Surabhi Gogte
  Cc: Jens Axboe, Christoph Hellwig, Sagi Grimberg, linux-nvme,
	linux-kernel, mkhalfella, randyj, adailey

On Tue, Sep 22, 2026 at 02:07:15PM -0600, Surabhi Gogte wrote:
> Similar to commit 2a8513091d2f ("nvme-rdma: parallelize I/O queue
> allocation and startup"), I/O queue allocation and start steps
> (including authentication) can be parallelized for nvme-tcp as well.
> 
> Patch 1 fixes a pre-existing bug: nvme_tcp_alloc_queue() used
> current->nsproxy->net_ns to create sockets during initial connect
> while it used kernel's init_net for reconnect and error recovery
> paths. The fix captures the caller's network namespace at controller
> allocation time.
> 
> Patch 2 is a preparatory refactor: it splits the queue-count
> negotiation and TLS PSK checks into helpers so that the individual
> steps can be called directly from nvme_tcp_configure_io_queues(). No
> functional change.
> 
> Patch 3 has the async implementation: it fans the per-queue alloc and
> start out over an async domain, serializes the per-cpu queue-count in
> nvme_tcp_set_queue_io_cpu() that can now run concurrently, and uses
> ctrl->net for all socket creation.
> 
> Testing on a 64-core host with 64 IO-queues shows nvme-tcp connection
> time reduced from 61ms to 11ms.

Thanks, applied to nvme-7.4.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup
  2026-09-22 20:07 ` [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup Surabhi Gogte
@ 2026-10-18 12:25   ` Shin'ichiro Kawasaki
  0 siblings, 0 replies; 6+ messages in thread
From: Shin'ichiro Kawasaki @ 2026-10-18 12:25 UTC (permalink / raw)
  To: Surabhi Gogte
  Cc: Keith Busch, Jens Axboe, Christoph Hellwig, Sagi Grimberg,
	linux-nvme, linux-kernel, mkhalfella, randyj, adailey,
	Hannes Reinecke

On Sep 22, 2026 / 14:07, Surabhi Gogte wrote:
> Similar to commit 2a8513091d2f ("nvme-rdma: parallelize I/O queue
> allocation and startup"), refactor nvme tcp I/O queue setup to use async
> API, combining allocation and startup into a single parallel operation
> per queue. This reduces connection and reconnection setup time when
> there are delays in establishing connections, which is especially
> important for high-core-count hosts.
> 
> Key changes:
> - Use async API to facilitate parallel calls for io queue setup.
> - Add nvme_tcp_queue_setup_ctx for propagating errors from async
>   workers.
> - Remove nvme_tcp_start_io_queues() and __nvme_tcp_alloc_io_queues();
>   their logic is folded into nvme_tcp_setup_io_queues() and
>   nvme_tcp_configure_io_queues().
> - Allocate the io tag set before the queues so that the queue range is
>   known, and only set up the reconnect grow case if the queue count
>   actually increased.
> - Serialize the cpu scan and claim in nvme_tcp_set_queue_io_cpu() with a
>   spinlock, as concurrent callers would otherwise select the same cpu.
>   The per-cpu counters no longer need to be atomics.
> 
> Testing on a 64-core host with 64 IO-queues shows nvme-tcp connection
> time reduced from 61ms to 11ms.

Recently, blktests-CI runs for the kernel at block/for-next branch started
reporting failures of the test case nvme/004 (with tcp transport) and md/001.
The kernel reported a WARN related to lock->magic, and it caused the failures.
FYI, here I share the failure run log at md/001 [1] and dmesg at the failure.

  The md/001 confirms a bug fix in md driver. The bug was triggered when a
  nvme device with tcp transport is used as its backend. Then the test case
  exercises the nvme-tcp driver, which this patch touches.

I bisected and found that this commit is the trigger of the failures. When I
revert this commit from the block/for-next branch tip with git hash
34fbfb9d8d1f, the failure goes away.

Surabhi, could you take a look in the failure?


[1] failure of md/001

md/001 (Raid with bitmap on tcp nvmet with opt-io-size over bitmap size) [failed]
    runtime  3.021s  ...  2.970s
    something found in dmesg:
    [   79.786635] [    T966] run blktests md/001 at 2026-10-18 20:35:29.594227680
    [   80.157789] [    T984] brd: module loaded
    [   80.566933] [   T1008] nvmet: adding nsid 1 to subsystem blktests-subsystem-1
    [   80.637888] [   T1011] nvmet_tcp: enabling port 0 (127.0.0.1:4420)
    [   80.925883] [    T312] nvmet: Created nvm controller 1 for subsystem blktests-subsystem-1 for NQN nqn.2014-08.org.nvmexpress:uuid:0f01fb42-9f7f-4856-b0b3-51e60b8de349.
    [   80.940866] [   T1018] nvme nvme5: creating 4 I/O queues.
    [   80.942243] [   T1018] nvme nvme5: mapped 4/0/0 default/read/poll queues.
    [   80.942689] [   T1018] ------------[ cut here ]------------
    [   80.942879] [   T1018] DEBUG_LOCKS_WARN_ON(lock->magic != lock)
    [   80.942882] [   T1018] WARNING: kernel/locking/mutex.c:625 at __mutex_lock+0x1008/0x2510, CPU#0: nvme/1018
    ...
    (See '/home/shin/Blktests/blktests/results/nodev/md/001.dmesg' for the entire message)


[2] dmesg during md/001

2026-10-17T22:40:06Z {detected_level="unknown"} [ 2568.828164] run blktests md/001 at 2026-10-17 22:40:06.674885632
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.162185] brd: module loaded
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.587993] nvmet: adding nsid 1 to subsystem blktests-subsystem-1
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.629198] nvmet_tcp: enabling port 0 (127.0.0.1:4420)
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.848587] nvmet: Created nvm controller 1 for subsystem blktests-subsystem-1 for NQN nqn.2014-08.org.nvmexpress:uuid:0f01fb42-9f7f-4856-b0b3-51e60b8de349.
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.875225] nvme nvme2: creating 4 I/O queues.
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.878288] nvme nvme2: mapped 4/0/0 default/read/poll queues.
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.883211] ------------[ cut here ]------------
2026-10-17T22:40:07Z {detected_level="debug"}   [ 2569.883660] DEBUG_LOCKS_WARN_ON(lock->magic != lock)
2026-10-17T22:40:07Z {detected_level="warn"}    [ 2569.883666] WARNING: kernel/locking/mutex.c:625 at __mutex_lock+0xb73/0x2350, CPU#2: nvme/55686
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.887533] Modules linked in: nvme_tcp(E) nvme_fabrics(E) nvmet_tcp(E) nvmet(E) brd(E) raid1(E) dm_dust(E) loop(E) xfs(E) tun(E) nvme(E) iTCO_wdt(E) intel_rapl_msr(E) intel_pmc_bxt(E) intel_rapl_common(E) pcspkr(E) i2c_i801(E) i2c_smbus(E) nvme_core(E) nvme_keyring(E) virtio_balloon(E) nvme_auth(E) virtio_scsi(E) lpc_ich(E) bochs(E) serio_raw(E) qemu_fw_cfg(E) vmw_vsock_virtio_transport(E) vmw_vsock_virtio_transport_common(E) vsock(E)
2026-10-17T22:40:07Z {detected_level="debug"}   [ 2569.887693] Unloaded tainted modules: nvme_tcp(E):1 nvmet(E):63 nvme_fabrics(E):63 nvme_loop(E):63 loop(E):14 ublk_drv(E):2 null_blk(E):37 scsi_debug(E):53 amd_atl(E):1 edac_mce_amd(E):1 bpf_preload(E):1 [last unloaded: scsi_debug(E)]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.899624] CPU: 2 UID: 0 PID: 55686 Comm: nvme Tainted: G            E       7.3.0-rc7 #1 PREEMPT(lazy)
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.902229] Tainted: [E]=UNSIGNED_MODULE
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.903507] Hardware name: KubeVirt None/RHEL, BIOS 1.16.3-4.el9 04/01/2014
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.905532] RIP: 0010:__mutex_lock+0xb7a/0x2350
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.907336] Code: 38 d0 7c 08 84 d2 0f 85 52 17 00 00 44 8b 05 8d af 8d 02 45 85 c0 0f 85 0a f6 ff ff 48 8d 3d ad cc 8e 02 48 c7 c6 c0 fd 2f 85 <67> 48 0f b9 3a e9 f2 f5 ff ff 4d 8d a7 e0 0e 00 00 4c 89 e7 e8 3d
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.912320] RSP: 0018:ffff8881438c7370 EFLAGS: 00010246
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.914185] RAX: 0000000000000003 RBX: ffff8881face6460 RCX: 1ffffffff0f35008
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.916368] RDX: 0000000000000000 RSI: ffffffff852ffdc0 RDI: ffffffff879b9d70
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.918675] RBP: ffff8881438c74e0 R08: 0000000000000000 R09: fffffbfff0f35008
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.920803] R10: ffffffff879a8047 R11: ffff88812d120008 R12: 0000000000000000
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.923039] R13: 0000000000000000 R14: dffffc0000000000 R15: ffff88812d3a0000
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.925258] FS:  00007fd431b6b640(0000) GS:ffff8882ac2ea000(0000) knlGS:0000000000000000
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.927640] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.929481] CR2: 00007f50088d0c1c CR3: 000000010d0d4006 CR4: 0000000000770ef0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.932053] PKRU: 55555554
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.933507] Call Trace:
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.934813]  <TASK>
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.936625]  ? nvme_tcp_init_request+0xc8/0x300 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.938805]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.940675]  ? __alloc_frozen_pages_noprof+0x543/0x7c0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.942464]  ? __pfx___mutex_lock+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.944123]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.945814]  ? alloc_pages_node_noprof+0xae/0x150
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.947474]  ? nvme_tcp_init_request+0xc8/0x300 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.949268]  nvme_tcp_init_request+0xc8/0x300 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.951141]  blk_mq_alloc_rqs+0x3fc/0x6f0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.952747]  blk_mq_alloc_map_and_rqs+0x168/0x320
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.954450]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.956260]  blk_mq_alloc_set_map_and_rqs+0x1be/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.958088]  blk_mq_alloc_tag_set+0x59a/0xc40
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.959710]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.961321]  ? dev_printk_emit+0xbe/0xf3
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.962736]  nvme_alloc_io_tag_set+0x37d/0x6a0 [nvme_core]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.964446]  ? __pfx_nvme_alloc_io_tag_set+0x10/0x10 [nvme_core]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.966345]  ? percpu_ref_put_many.constprop.0+0x7f/0x1b0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.968109]  ? __nvme_submit_sync_cmd+0x1e2/0x320 [nvme_core]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.970040]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.971714]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="info"}    [ 2569.973339]  ? _dev_info+0xe6/0x11a
2026-10-17T22:40:07Z {detected_level="info"}    [ 2569.974760]  ? __pfx__dev_info+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.976153]  ? __pfx_nvme_set_queue_count+0x10/0x10 [nvme_core]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.977938]  ? lock_acquire.part.0+0xc7/0x240
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.979582]  nvme_tcp_configure_io_queues.cold+0x27f/0x8ee [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.981436]  ? __pfx_nvme_enable_ctrl+0x10/0x10 [nvme_core]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.983132]  ? lockdep_hardirqs_on_prepare.part.0+0x9b/0x140
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.984762]  ? __pfx_nvme_tcp_configure_io_queues+0x10/0x10 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.986525]  nvme_tcp_setup_ctrl+0x281/0x5f0 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.988113]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.989665]  nvme_tcp_create_ctrl+0xf6/0x120 [nvme_tcp]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.991236]  nvmf_create_ctrl+0x34e/0x920 [nvme_fabrics]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.992832]  nvmf_dev_write+0xd5/0x180 [nvme_fabrics]
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.994357]  vfs_write+0x1c7/0x1060
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.995622]  ? __pfx_vfs_write+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.996900]  ksys_write+0xf9/0x1d0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.998093]  ? __pfx_ksys_write+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2569.999349]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.000692]  ? lockdep_hardirqs_on_prepare.part.0+0x9b/0x140
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.002249]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.003675]  ? trace_hardirqs_on+0x18/0x160
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.005046]  do_syscall_64+0xef/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.006137]  ? kmem_cache_free+0x143/0x600
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.007405]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.008722]  ? do_sys_openat2+0x107/0x180
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.009946]  ? __pfx_do_sys_openat2+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.011180]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.012539]  ? fput_close_sync+0xe5/0x1a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.013752]  ? __pfx_fput_close_sync+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.015062]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.016455]  ? __x64_sys_openat+0x108/0x1d0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.017722]  ? __pfx___x64_sys_openat+0x10/0x10
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.019017]  ? trace_hardirqs_on_prepare+0x119/0x150
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.020393]  ? do_syscall_64+0x1d7/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.021623]  ? do_syscall_64+0x1b9/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.022876]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.024209]  ? trace_hardirqs_on_prepare+0x119/0x150
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.025581]  ? do_syscall_64+0x1b9/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.026867]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.028225]  ? trace_hardirqs_on_prepare+0x119/0x150
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.029718]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.031169]  ? do_syscall_64+0x1d7/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.032408]  ? do_syscall_64+0x1b9/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.033704]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.035099]  ? trace_hardirqs_on_prepare+0x119/0x150
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.036605]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.037956]  ? do_syscall_64+0x1d7/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.039181]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.040575]  ? __lock_release.isra.0+0x5b/0x180
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.041947]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.043389]  ? do_user_addr_fault+0x8d3/0xe80
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.044575]  ? irqentry_exit+0xb8/0x810
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.045690]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.046990]  ? do_syscall_64+0x2d/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.048067]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.049302]  ? srso_alias_return_thunk+0x5/0xfbef5
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.050545]  ? do_syscall_64+0x5c/0x5a0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.051649]  entry_SYSCALL_64_after_hwframe+0x76/0x7e
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.052916] RIP: 0033:0x7fd431c3d56e
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.053948] Code: 4d 89 d8 e8 d4 bd 00 00 4c 8b 5d f8 41 8b 93 08 03 00 00 59 5e 48 83 f8 fc 74 11 c9 c3 0f 1f 80 00 00 00 00 48 8b 45 10 0f 05 <c9> c3 83 e2 39 83 fa 08 75 e7 e8 03 ff ff ff 0f 1f 00 f3 0f 1e fa
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.057937] RSP: 002b:00007ffdaa469710 EFLAGS: 00000202 ORIG_RAX: 0000000000000001
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.059733] RAX: ffffffffffffffda RBX: 00005609fa418740 RCX: 00007fd431c3d56e
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.061425] RDX: 00000000000000cf RSI: 00005609fa418740 RDI: 0000000000000003
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.063121] RBP: 00007ffdaa469720 R08: 0000000000000000 R09: 0000000000000000
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.064826] R10: 0000000000000000 R11: 0000000000000202 R12: 0000000000000003
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.066523] R13: 00005609fa4175b0 R14: 00000000000000cf R15: 00005609fa4175b0
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.068235]  </TASK>
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.069130] irq event stamp: 11409
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.070225] hardirqs last  enabled at (11409): [<ffffffff850e58f5>] _raw_spin_unlock_irqrestore+0x45/0x60
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.072350] hardirqs last disabled at (11408): [<ffffffff817db728>] __raw_spin_lock_irqsave+0x58/0x60
2026-10-17T22:40:07Z {detected_level="unknown"} [ 2570.074391] softirqs last  enabled at (11372): [<ffffffff815ffc88>] handle_softirqs+0x638/0x880
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.076357] softirqs last disabled at (11365): [<ffffffff8160003f>] __irq_exit_rcu+0x15f/0x290
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.078305] ---[ end trace 0000000000000000 ]---
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.084017] nvme nvme2: new ctrl: NQN "blktests-subsystem-1", addr 127.0.0.1:4420, hostnqn: nqn.2014-08.org.nvmexpress:uuid:0f01fb42-9f7f-4856-b0b3-51e60b8de349
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.400433] md127: array will not be assembled in old kernels that lack configurable LBS support (<= 6.18)
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.401014] md/raid1:md127: active with 1 out of 2 mirrors
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.404015] md127: detected capacity change from 0 to 2093056
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.559964] md127: detected capacity change from 2093056 to 0
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.560710] md: md127 stopped.
2026-10-17T22:40:08Z {detected_level="unknown"} [ 2570.632174] nvme nvme2: Removing ctrl: NQN "blktests-subsystem-1"
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2571.118121] brd: module unloaded
2026-10-17T22:40:09Z {detected_level="debug"}   [ 2572.039779] scsi_debug:sdebug_driver_probe: scsi_debug: trim poll_queues to 0. poll_q/nr_hw = (0/1)
2026-10-17T22:40:09Z {detected_level="debug"}   [ 2572.040354] scsi host66: scsi_debug: version 0191 [20210520]
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.040354]   dev_size_mb=8, opts=0x0, submit_queues=1, statistics=0
2026-10-17T22:40:09Z {detected_level="debug"}   [ 2572.046424] scsi 66:0:0:0: Direct-Access     Linux    scsi_debug       0191 PQ: 0 ANSI: 7
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.053271] scsi 66:0:0:0: Power-on or device reset occurred
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.063350] sd 66:0:0:0: [sda] 16384 512-byte logical blocks: (8.39 MB/8.00 MiB)
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.063603] sd 66:0:0:0: Attached scsi generic sg1 type 0
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.064965] sd 66:0:0:0: [sda] Write Protect is off
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.065470] sd 66:0:0:0: [sda] Mode Sense: 73 00 10 08
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.068116] sd 66:0:0:0: [sda] Write cache: enabled, read cache: enabled, supports DPO and FUA
2026-10-17T22:40:09Z {detected_level="unknown"} [ 2572.075669] sd 66:0:0:0: [sda] permanent stream count = 5
2026-10-17T22:40:10Z {detected_level="unknown"} [ 2572.080471] sd 66:0:0:0: [sda] Preferred minimum I/O size 512 bytes
2026-10-17T22:40:10Z {detected_level="unknown"} [ 2572.082032] sd 66:0:0:0: [sda] Optimal transfer size 524288 bytes
2026-10-17T22:40:10Z {detected_level="unknown"} [ 2572.131009] sd 66:0:0:0: [sda] Attached SCSI disk

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-10-18 12:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-22 20:07 [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Surabhi Gogte
2026-09-22 20:07 ` [PATCH v4 1/3] nvme-tcp: store and use the caller's network namespace Surabhi Gogte
2026-09-22 20:07 ` [PATCH v4 2/3] nvme-tcp: refactor I/O queue setup path Surabhi Gogte
2026-09-22 20:07 ` [PATCH v4 3/3] nvme-tcp: parallelize I/O queue allocation and startup Surabhi Gogte
2026-10-18 12:25   ` Shin'ichiro Kawasaki
2026-10-13 15:33 ` [PATCH v4 0/3] nvme-tcp: parallelize I/O setup queue in connect Keith Busch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®