On 11/13/12 22:41, Or Gerlitz wrote:
On Tue, Nov 13, 2012 at 11:30 PM, Bart Van Assche <[email protected]> wrote:
These attributes have been added to the SRP transport layer. Since the
ib_srp driver registers itself with the SRP transport layer the SRP
transport layer creates these two attributes for the ib_srp driver. This is
similar to how the FC transport layer creates these attributes for FC
initiator drivers.

thanks for the heads up... can you provide some pointer please 2-3
relevant commits and hopefully documentation?  also I recall an
earlier version of this patch set where you attempted to add to to SRP
or even on some higher level some probing mechanism which mimics the
NOPs used by iSCSI, where/how was this ended? does the fact  that
iscsi has NOPs makes these parameters un-needed where in SRP/FC they
are?

Hello Or,

Via patch number 16 of this series not only the fast_io_fail_tmo and dev_loss_tmo attributes were added to the SRP transport layer but also documentation for these attributes was added to Documentation/ABI/stable/sysfs-transport-srp. And if you would like to have a closer look at the implementation of these attributes for the FC initiator transport layer, you can find it in drivers/scsi/scsi_transport_fc.c.

The code for sending a TUR periodically that was present in a previous version of this patch series has been dropped because it's not a proper transport layer test. As far as I know there is no equivalent for iSCSI NOPs in the SRP protocol. There might be an appropriate equivalent in the IB RC layer but I haven't started looking into that yet.

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to