[PATCH 2/2] nvme-auth: remove the redundant req->cqe->result.u16 assignment operation

Jackie Liu liu.yun at linux.dev
Thu Aug 11 20:12:31 PDT 2022


From: Jackie Liu <liuyun01 at kylinos.cn>

req->cqe->result.u16 has already been assigned in the previous line, no
need to do it again.

Signed-off-by: Jackie Liu <liuyun01 at kylinos.cn>
---
 drivers/nvme/target/fabrics-cmd.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/nvme/target/fabrics-cmd.c b/drivers/nvme/target/fabrics-cmd.c
index f91a56180d3d..c1dfdfb92ebf 100644
--- a/drivers/nvme/target/fabrics-cmd.c
+++ b/drivers/nvme/target/fabrics-cmd.c
@@ -332,7 +332,6 @@ static void nvmet_execute_io_connect(struct nvmet_req *req)
 	req->cqe->result.u16 = cpu_to_le16(ctrl->cntlid);
 
 	pr_debug("adding queue %d to ctrl %d.\n", qid, ctrl->cntlid);
-	req->cqe->result.u16 = cpu_to_le16(ctrl->cntlid);
 	if (nvmet_has_auth(ctrl))
 		nvmet_init_auth(ctrl, req);
 
-- 
2.25.1




More information about the Linux-nvme mailing list