From: Christoph Hellwig <hch@lst.de>
Date: Tue, 20 Sep 2022 15:38:58 +0200
Subject: nvme: improve the NVME_CONNECT_AUTHREQ* definitions
Patch-mainline: v6.1-rc1
Git-commit: 1c32a8012b7fabe469b6af826edfd4ae2a6201d3
References: git-fixes
Mark them as unsigned so that we don't need extra casts, and define
them relative to cdword0 instead of requiring extra shifts.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Acked-by: Daniel Wagner <dwagner@suse.de>
---
drivers/nvme/target/fabrics-cmd.c | 6 ++----
include/linux/nvme.h | 4 ++--
2 files changed, 4 insertions(+), 6 deletions(-)
--- a/drivers/nvme/target/fabrics-cmd.c
+++ b/drivers/nvme/target/fabrics-cmd.c
@@ -255,8 +255,7 @@ static void nvmet_execute_admin_connect(
req->cqe->result.u16 = cpu_to_le16(ctrl->cntlid);
if (nvmet_has_auth(ctrl))
- req->cqe->result.u32 |=
- cpu_to_le32((u32)NVME_CONNECT_AUTHREQ_ATR << 16);
+ req->cqe->result.u32 |= cpu_to_le32(NVME_CONNECT_AUTHREQ_ATR);
out:
kfree(d);
complete:
@@ -318,8 +317,7 @@ static void nvmet_execute_io_connect(str
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))
- req->cqe->result.u32 |=
- cpu_to_le32((u32)NVME_CONNECT_AUTHREQ_ATR << 16);
+ req->cqe->result.u32 |= cpu_to_le32(NVME_CONNECT_AUTHREQ_ATR);
out:
kfree(d);
--- a/include/linux/nvme.h
+++ b/include/linux/nvme.h
@@ -1394,8 +1394,8 @@ struct nvmf_connect_command {
};
enum {
- NVME_CONNECT_AUTHREQ_ASCR = (1 << 2),
- NVME_CONNECT_AUTHREQ_ATR = (1 << 1),
+ NVME_CONNECT_AUTHREQ_ASCR = (1U << 18),
+ NVME_CONNECT_AUTHREQ_ATR = (1U << 17),
};
struct nvmf_connect_data {