From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3DBFD1CFA9; Tue, 27 Feb 2024 13:31:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709040672; cv=none; b=pAtlcoXLgTGGE6fQycBZ5K31VpmSM6OK/utbuxndOIu3p4/EhCXXyutYzjnZRtJO+Xaytf68DPTdyKxzkAhRvWlg1hsT2gq9AW1N3DfiLrAI8ioozyHvt/hXAOJc8PZggu3uSYb+9ANr6VhyMhc6KWLy7zI11ZFpEzWOm/YMN1Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709040672; c=relaxed/simple; bh=EIfQ2cq+gadbXKPmRdqGNvaGSIqYsGuQsybmCvUMTxI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=d/S6jTPF3zQ7mC8PMTLTA1yoKMuUJGoG6tsxQ2lcFW+z4w0hWYYQP4NzJBBDy4nZHT4Lkwvd4RV8zhxzz9GkpSXjWqGEjO5OvJEnK6L8HdPRXdPEACZ5SiJkM5A6wk+HKNB1eoFz4mgzyC2T93xl49tB276hry4VPWNgGj97lxM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=TBehFgxe; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="TBehFgxe" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7DF1BC433F1; Tue, 27 Feb 2024 13:31:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1709040671; bh=EIfQ2cq+gadbXKPmRdqGNvaGSIqYsGuQsybmCvUMTxI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=TBehFgxeUqRGlFWfHZX+6NJnLolMsbD5tuUXDIm6vH6azJ+YBAv2wR9CN4gC1wNSv 2j9W//YL85Ps6f5dhp9jdPfKAYolrNImCPp0ikhpW/C0mIbKJn86pq2+zksTZojGAq sxtSze4xSk2veoDuC5mNTYgvqbxYipAYak1EjOAk= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Hannes Reinecke , Christoph Hellwig , Daniel Wagner , Keith Busch , Sasha Levin Subject: [PATCH 6.7 069/334] nvmet-fc: abort command when there is no binding Date: Tue, 27 Feb 2024 14:18:47 +0100 Message-ID: <20240227131632.780784620@linuxfoundation.org> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20240227131630.636392135@linuxfoundation.org> References: <20240227131630.636392135@linuxfoundation.org> User-Agent: quilt/0.67 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.7-stable review patch. If anyone has any objections, please let me know. ------------------ From: Daniel Wagner [ Upstream commit 3146345c2e9c2f661527054e402b0cfad80105a4 ] When the target port has not active port binding, there is no point in trying to process the command as it has to fail anyway. Instead adding checks to all commands abort the command early. Reviewed-by: Hannes Reinecke Reviewed-by: Christoph Hellwig Signed-off-by: Daniel Wagner Signed-off-by: Keith Busch Signed-off-by: Sasha Levin --- drivers/nvme/target/fc.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/nvme/target/fc.c b/drivers/nvme/target/fc.c index 6d111b03d3713..686843eb2ebf3 100644 --- a/drivers/nvme/target/fc.c +++ b/drivers/nvme/target/fc.c @@ -1100,6 +1100,9 @@ nvmet_fc_alloc_target_assoc(struct nvmet_fc_tgtport *tgtport, void *hosthandle) int idx; bool needrandom = true; + if (!tgtport->pe) + return NULL; + assoc = kzalloc(sizeof(*assoc), GFP_KERNEL); if (!assoc) return NULL; @@ -2522,8 +2525,9 @@ nvmet_fc_handle_fcp_rqst(struct nvmet_fc_tgtport *tgtport, fod->req.cmd = &fod->cmdiubuf.sqe; fod->req.cqe = &fod->rspiubuf.cqe; - if (tgtport->pe) - fod->req.port = tgtport->pe->port; + if (!tgtport->pe) + goto transport_error; + fod->req.port = tgtport->pe->port; /* clear any response payload */ memset(&fod->rspiubuf, 0, sizeof(fod->rspiubuf)); -- 2.43.0