Skip to content

Commit ff4a0a4

Browse files
Hannes Reineckekeithbusch
authored andcommitted
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]>
1 parent bb2df18 commit ff4a0a4

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

drivers/nvme/target/admin-cmd.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1005,8 +1005,6 @@ u16 nvmet_parse_admin_cmd(struct nvmet_req *req)
10051005

10061006
if (nvme_is_fabrics(cmd))
10071007
return nvmet_parse_fabrics_admin_cmd(req);
1008-
if (unlikely(!nvmet_check_auth_status(req)))
1009-
return NVME_SC_AUTH_REQUIRED | NVME_STATUS_DNR;
10101008
if (nvmet_is_disc_subsys(nvmet_req_subsys(req)))
10111009
return nvmet_parse_discovery_cmd(req);
10121010

0 commit comments

Comments
 (0)