From 14e1d0c747565f3175db7ddcf733c29282958dd7 Mon Sep 17 00:00:00 2001 From: Ziye Yang Date: Tue, 19 Feb 2019 04:16:24 +0800 Subject: [PATCH] nvme/tcp: call nvme_ctrlr_add_process in construct function. Purpose: to make the timeout work for NVMe TCP transport, we miss this for TCP transport. Change-Id: Iab4af988cc4796b4d6d98430453f3dbce1fcf313 Signed-off-by: Ziye Yang Reviewed-on: https://review.gerrithub.io/c/445117 Tested-by: SPDK CI Jenkins Reviewed-by: Changpeng Liu Reviewed-by: Shuhei Matsumoto Reviewed-by: Seth Howell Reviewed-by: Ben Walker --- lib/nvme/nvme_tcp.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/lib/nvme/nvme_tcp.c b/lib/nvme/nvme_tcp.c index c4cd8e9bd..acaf79a38 100644 --- a/lib/nvme/nvme_tcp.c +++ b/lib/nvme/nvme_tcp.c @@ -1856,6 +1856,12 @@ struct spdk_nvme_ctrlr *nvme_tcp_ctrlr_construct(const struct spdk_nvme_transpor return NULL; } + if (nvme_ctrlr_add_process(&tctrlr->ctrlr, 0) != 0) { + SPDK_ERRLOG("nvme_ctrlr_add_process() failed\n"); + nvme_ctrlr_destruct(&tctrlr->ctrlr); + return NULL; + } + nvme_ctrlr_init_cap(&tctrlr->ctrlr, &cap, &vs); return &tctrlr->ctrlr;