Blame patches.suse/nvme-tcp-fix-error-codes-in-nvme_tcp_setup_ctrl.patch
|
Hannes Reinecke |
6aa7c9 |
From: Dan Carpenter <dan.carpenter@oracle.com>
|
|
Hannes Reinecke |
6aa7c9 |
Date: Sat, 5 Jun 2021 15:48:16 +0300
|
|
Hannes Reinecke |
6aa7c9 |
Subject: [PATCH] nvme-tcp: fix error codes in nvme_tcp_setup_ctrl()
|
|
Hannes Reinecke |
6aa7c9 |
Git-commit: 522af60cb2f8e3658bda1902fb7f200dcf888a5c
|
|
Hannes Reinecke |
6aa7c9 |
Patch-mainline: v5.14-rc1
|
|
Hannes Reinecke |
6aa7c9 |
References: git-fixes
|
|
Hannes Reinecke |
6aa7c9 |
|
|
Hannes Reinecke |
6aa7c9 |
These error paths currently return success but they should return
|
|
Hannes Reinecke |
6aa7c9 |
-EOPNOTSUPP.
|
|
Hannes Reinecke |
6aa7c9 |
|
|
Hannes Reinecke |
6aa7c9 |
Fixes: 73ffcefcfca0 ("nvme-tcp: check sgl supported by target")
|
|
Hannes Reinecke |
6aa7c9 |
Fixes: 3f2304f8c6d6 ("nvme-tcp: add NVMe over TCP host driver")
|
|
Hannes Reinecke |
6aa7c9 |
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
|
|
Hannes Reinecke |
6aa7c9 |
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
|
|
Hannes Reinecke |
6aa7c9 |
Signed-off-by: Christoph Hellwig <hch@lst.de>
|
|
Hannes Reinecke |
6aa7c9 |
Acked-by: Hannes Reinecke <hare@suse.com>
|
|
Hannes Reinecke |
6aa7c9 |
---
|
|
Hannes Reinecke |
6aa7c9 |
drivers/nvme/host/tcp.c | 2 ++
|
|
Hannes Reinecke |
6aa7c9 |
1 file changed, 2 insertions(+)
|
|
Hannes Reinecke |
6aa7c9 |
|
|
Hannes Reinecke |
6aa7c9 |
diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6aa7c9 |
index 5fc6c568c626..6a65b0516180 100644
|
|
Hannes Reinecke |
6aa7c9 |
--- a/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6aa7c9 |
+++ b/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6aa7c9 |
@@ -1988,11 +1988,13 @@ static int nvme_tcp_setup_ctrl(struct nvme_ctrl *ctrl, bool new)
|
|
Hannes Reinecke |
6aa7c9 |
return ret;
|
|
Hannes Reinecke |
6aa7c9 |
|
|
Hannes Reinecke |
6aa7c9 |
if (ctrl->icdoff) {
|
|
Hannes Reinecke |
6aa7c9 |
+ ret = -EOPNOTSUPP;
|
|
Hannes Reinecke |
6aa7c9 |
dev_err(ctrl->device, "icdoff is not supported!\n");
|
|
Hannes Reinecke |
6aa7c9 |
goto destroy_admin;
|
|
Hannes Reinecke |
6aa7c9 |
}
|
|
Hannes Reinecke |
6aa7c9 |
|
|
Hannes Reinecke |
6aa7c9 |
if (!(ctrl->sgls & ((1 << 0) | (1 << 1)))) {
|
|
Hannes Reinecke |
6aa7c9 |
+ ret = -EOPNOTSUPP;
|
|
Hannes Reinecke |
6aa7c9 |
dev_err(ctrl->device, "Mandatory sgls are not supported!\n");
|
|
Hannes Reinecke |
6aa7c9 |
goto destroy_admin;
|
|
Hannes Reinecke |
6aa7c9 |
}
|
|
Hannes Reinecke |
6aa7c9 |
--
|
|
Hannes Reinecke |
6aa7c9 |
2.29.2
|
|
Hannes Reinecke |
6aa7c9 |
|