Fix to return negative error code -EINVAL if mandatory sgls are not supported. Fixes: 5aadd5f9311e ("nvme-tcp-offload: Add controller level implementation") Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> --- drivers/nvme/host/tcp-offload.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/nvme/host/tcp-offload.c b/drivers/nvme/host/tcp-offload.c index c76822e5ada7..3879d5ed7497 100644 --- a/drivers/nvme/host/tcp-offload.c +++ b/drivers/nvme/host/tcp-offload.c @@ -622,6 +622,7 @@ static int nvme_tcp_ofld_setup_ctrl(struct nvme_ctrl *nctrl, bool new) if (!(nctrl->sgls & ((1 << 0) | (1 << 1)))) { dev_err(nctrl->device, "Mandatory sgls are not supported!\n"); + rc = -EINVAL; goto destroy_admin; }