From a5dfbc4daf8d510e88ea3d66ec605f565eee2e98 Mon Sep 17 00:00:00 2001 From: lorneli Date: Thu, 30 May 2019 18:34:12 +0800 Subject: [PATCH] nvme: zero request->submit_tick in allocation Request may be submitted several times via nvme_qpair_submit_request function, such as request in queued_req queue being re-submitted. With enabling timeout feature, nvme_qpair_submit_request compares request->submit_tick to zero to check if this is the first submission for this request. If true, record submit_tick for this reuqest. So request->submit_tick needs to be set zero in allocation. Change-Id: Ie3f420aa337802c5ad3962c3fdcd680dec1ccdcb Signed-off-by: lorneli Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/456328 Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Ben Walker --- lib/nvme/nvme_internal.h | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/nvme/nvme_internal.h b/lib/nvme/nvme_internal.h index 9262dcd1b..2736fdbf8 100644 --- a/lib/nvme/nvme_internal.h +++ b/lib/nvme/nvme_internal.h @@ -888,6 +888,7 @@ nvme_allocate_request(struct spdk_nvme_qpair *qpair, req->payload_size = payload_size; req->qpair = qpair; req->pid = g_spdk_nvme_pid; + req->submit_tick = 0; return req; }