Commit edec8dfe authored by Mark Rustad's avatar Mark Rustad Committed by Nicholas Bellinger

tcm_fc: Do not report target role when target is not defined

Clear the target role when no target is provided for
the node performing a PRLI.
Signed-off-by: default avatarMark Rustad <mark.d.rustad@intel.com>
Reviewed-by: default avatarBhanu Prakash Gollapudi <bprakash@broadcom.com>
Acked by Robert Love <robert.w.love@intel.com>
Cc: stable@vger.kernel.org
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
parent f2eeba21
...@@ -355,11 +355,11 @@ static int ft_prli_locked(struct fc_rport_priv *rdata, u32 spp_len, ...@@ -355,11 +355,11 @@ static int ft_prli_locked(struct fc_rport_priv *rdata, u32 spp_len,
tport = ft_tport_create(rdata->local_port); tport = ft_tport_create(rdata->local_port);
if (!tport) if (!tport)
return 0; /* not a target for this local port */ goto not_target; /* not a target for this local port */
acl = ft_acl_get(tport->tpg, rdata); acl = ft_acl_get(tport->tpg, rdata);
if (!acl) if (!acl)
return 0; goto not_target; /* no target for this remote */
if (!rspp) if (!rspp)
goto fill; goto fill;
...@@ -402,6 +402,12 @@ static int ft_prli_locked(struct fc_rport_priv *rdata, u32 spp_len, ...@@ -402,6 +402,12 @@ static int ft_prli_locked(struct fc_rport_priv *rdata, u32 spp_len,
fcp_parm &= ~FCP_SPPF_RETRY; fcp_parm &= ~FCP_SPPF_RETRY;
spp->spp_params = htonl(fcp_parm | FCP_SPPF_TARG_FCN); spp->spp_params = htonl(fcp_parm | FCP_SPPF_TARG_FCN);
return FC_SPP_RESP_ACK; return FC_SPP_RESP_ACK;
not_target:
fcp_parm = ntohl(spp->spp_params);
fcp_parm &= ~FCP_SPPF_TARG_FCN;
spp->spp_params = htonl(fcp_parm);
return 0;
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment