Thread (33 messages) 33 messages, 3 authors, 2021-11-16

Re: [PATCH 06/12] nvme-fabrics: decode 'authentication required' connect error

From: Himanshu Madhani <hidden>
Date: 2021-11-15 15:26:33
Also in: linux-nvme


On 11/12/21 06:59, Hannes Reinecke wrote:
quoted hunk ↗ jump to hunk
The 'connect' command might fail with NVME_SC_AUTH_REQUIRED, so we
should be decoding this error, too.

Signed-off-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
---
  drivers/nvme/host/fabrics.c | 4 ++++
  1 file changed, 4 insertions(+)
diff --git a/drivers/nvme/host/fabrics.c b/drivers/nvme/host/fabrics.c
index c5a2b71c5268..a1343a0790f6 100644
--- a/drivers/nvme/host/fabrics.c
+++ b/drivers/nvme/host/fabrics.c
@@ -332,6 +332,10 @@ static void nvmf_log_connect_error(struct nvme_ctrl *ctrl,
  		dev_err(ctrl->device,
  			"Connect command failed: host path error\n");
  		break;
+	case NVME_SC_AUTH_REQUIRED:
+		dev_err(ctrl->device,
+			"Connect command failed: authentication required\n");
+		break;
  	default:
  		dev_err(ctrl->device,
  			"Connect command failed, error wo/DNR bit: %d\n",
Reviewed-by: Himanshu Madhani <redacted>

-- 
Himanshu Madhani                                Oracle Linux Engineering
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help