2022-08-23 02:39:46

by Genjian

[permalink] [raw]
Subject: [PATCH v2] nvmet-auth: remove redundant parameters req

From: Genjian Zhang <[email protected]>

The parameter is not used in this function, so remove it.

Signed-off-by: Genjian Zhang <[email protected]>
---
drivers/nvme/target/fabrics-cmd-auth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/nvme/target/fabrics-cmd-auth.c b/drivers/nvme/target/fabrics-cmd-auth.c
index ebdf9aa81041..e467be86db78 100644
--- a/drivers/nvme/target/fabrics-cmd-auth.c
+++ b/drivers/nvme/target/fabrics-cmd-auth.c
@@ -177,7 +177,7 @@ static u16 nvmet_auth_reply(struct nvmet_req *req, void *d)
return 0;
}

-static u16 nvmet_auth_failure2(struct nvmet_req *req, void *d)
+static u16 nvmet_auth_failure2(void *d)
{
struct nvmf_auth_dhchap_failure_data *data = d;

@@ -310,7 +310,7 @@ void nvmet_execute_auth_send(struct nvmet_req *req)
goto done_kfree;
break;
case NVME_AUTH_DHCHAP_MESSAGE_FAILURE2:
- status = nvmet_auth_failure2(req, d);
+ status = nvmet_auth_failure2(d);
if (status) {
pr_warn("ctrl %d qid %d: authentication failed (%d)\n",
ctrl->cntlid, req->sq->qid, status);
--
2.25.1


2022-09-05 11:14:13

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH v2] nvmet-auth: remove redundant parameters req

Thanks,

applied to nvme-6.1.