From 94507133eb7fd6087da4cb7d83f0483a9a77ad5c Mon Sep 17 00:00:00 2001 From: Jacek Kalwas Date: Mon, 2 Dec 2019 13:08:55 +0100 Subject: [PATCH] nvmf/tcp: rm set_state in spdk_nvmf_tcp_capsule_cmd_hdr_handle TCP_REQUEST_STATE_NEW is already set in spdk_nvmf_tcp_req_get. Signed-off-by: Jacek Kalwas Change-Id: Ia835f3763cd74ef9b504901c719d9954317f49af Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/476164 Community-CI: Broadcom SPDK FC-NVMe CI Community-CI: SPDK CI Jenkins Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto --- lib/nvmf/tcp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/nvmf/tcp.c b/lib/nvmf/tcp.c index 791ac52f7..5ab54411c 100644 --- a/lib/nvmf/tcp.c +++ b/lib/nvmf/tcp.c @@ -1294,7 +1294,7 @@ spdk_nvmf_tcp_capsule_cmd_hdr_handle(struct spdk_nvmf_tcp_transport *ttransport, } pdu->req = tcp_req; - spdk_nvmf_tcp_req_set_state(tcp_req, TCP_REQUEST_STATE_NEW); + assert(tcp_req->state == TCP_REQUEST_STATE_NEW); spdk_nvmf_tcp_req_process(ttransport, tcp_req); return; }