diff options
| author | Hannes Reinecke <[email protected]> | 2024-07-22 14:02:26 +0200 |
|---|---|---|
| committer | Keith Busch <[email protected]> | 2024-08-22 13:25:11 -0700 |
| commit | ff4a0a4088adc6a37293a9cce25bb56ad2f26a16 (patch) | |
| tree | 91a45674c3b24761b034a93a6c674057ebf47b32 | |
| parent | bb2df18958b4287104c38541cf48c4a60c90e721 (diff) | |
nvme-target: do not check authentication status for admin commands twice
nvmet_check_ctrl_status() checks the authentication status, so
we don't need to do that prior to calling it.
Signed-off-by: Hannes Reinecke <[email protected]>
Reviewed-by: Sagi Grimberg <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Signed-off-by: Keith Busch <[email protected]>
| -rw-r--r-- | drivers/nvme/target/admin-cmd.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/nvme/target/admin-cmd.c b/drivers/nvme/target/admin-cmd.c index f7e1156ac7ec..d64480f01f4a 100644 --- a/drivers/nvme/target/admin-cmd.c +++ b/drivers/nvme/target/admin-cmd.c @@ -1005,8 +1005,6 @@ u16 nvmet_parse_admin_cmd(struct nvmet_req *req) if (nvme_is_fabrics(cmd)) return nvmet_parse_fabrics_admin_cmd(req); - if (unlikely(!nvmet_check_auth_status(req))) - return NVME_SC_AUTH_REQUIRED | NVME_STATUS_DNR; if (nvmet_is_disc_subsys(nvmet_req_subsys(req))) return nvmet_parse_discovery_cmd(req); |