[PATCH] nvme-fc: fail new connections to a deleted host or remote port

[Himanshu Madhani]

> On Sep 17, 2020, at 3:33 PM, James Smart <james.smart at broadcom.com> wrote:
> 
> The lldd may have made calls to delete a remote port or local port and
> the delete is in progress when the cli then attempts to create a new
> controller. Currently, this proceeds without error although it can't be
> very successful.
> 
> Fix this by validating that both the host port and remote port are
> present when a new controller is to be created.
> 
> Signed-off-by: James Smart <james.smart at broadcom.com>
> ---
> drivers/nvme/host/fc.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c
> index bc4e1af587cc..3f9eb37fe779 100644
> --- a/drivers/nvme/host/fc.c
> +++ b/drivers/nvme/host/fc.c
> @@ -3671,12 +3671,14 @@ nvme_fc_create_ctrl(struct device *dev, struct nvmf_ctrl_options *opts)
> 	spin_lock_irqsave(&nvme_fc_lock, flags);
> 	list_for_each_entry(lport, &nvme_fc_lport_list, port_list) {
> 		if (lport->localport.node_name != laddr.nn ||
> -		    lport->localport.port_name != laddr.pn)
> +		    lport->localport.port_name != laddr.pn ||
> +		    lport->localport.port_state != FC_OBJSTATE_ONLINE)
> 			continue;
> 
> 		list_for_each_entry(rport, &lport->endp_list, endp_list) {
> 			if (rport->remoteport.node_name != raddr.nn ||
> -			    rport->remoteport.port_name != raddr.pn)
> +			    rport->remoteport.port_name != raddr.pn ||
> +			    rport->remoteport.port_state != FC_OBJSTATE_ONLINE)
> 				continue;
> 
> 			/* if fail to get reference fall through. Will error */
> -- 
> 2.26.2
> 
> _______________________________________________
> Linux-nvme mailing list
> Linux-nvme at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-nvme

Looks good.

Reviewed-by: Himanshu Madhani <himanshu.madhani at oracle.com>

--
Himanshu Madhani	 Oracle Linux Engineering