Sven Schuetz wrote:
Shouldn't we add FC_BSG_HST_CT_NOLOGIN as well?
I didn't add this as FC-GS-6 r9.21, which defines the CT protocol, says:
4.5.2.1 Fabric login and N_Port login
An Nx_Port shall perform Fabric Login, and shall perform N_Port Login
with the WKA or N_Port Identifier where the Service is offered, in the
manner that is specified in FC-FS, before making any requests
of a Server provided by the Service. An Nx_Port that has completed its
registration with a Server shall perform explicit N_Port Logout with
the Service. An Nx_Port that has completed any other requests with a
Server should also perform explicit N_Port Logout with the Service.
So - a login for CT requests is a must (at least as I read it).
That's not the case for ELS's, which can usually survive on an implicit
login - especially things like ADISC/PDISC. Login is also the
distinction in my opinion for whether something should or should not be
exported via an rport in the transport.
As far as I can see from looking over the patch (correct me when I am wrong),
the way to issue a CT request now is to send an FC_BSG_HST_ADD_RPORT for the wka
port in question (if not enumerated by the driver, which zfcp does not do), then
send the FC_BSG_RPT_CT followed by a FC_BSG_HST_DEL_RPORT. I was under the
impression the CT stuff was one of the main reasons to have devices for the
fc_host. What do you think?
Your flow is exactly what I intended...
I also did not involve all the rport build-up, etc in the transport for
the adding of the rport. Just as today, all of this occurs within the
LLDD for normal discovery (deciding when to send the ELS's, when to make
the transport calls, etc) - I'm assuming the same holds true for the
add rport.
Also not addressed - how do we enable CT-receive and ELS-recieve. This,
due to the buffering and routing definitions (what process gets what)
is a fun problem.
Thanks Sven.
-- james s
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
