On Tue, Sep 17, 2019 at 10:19:37AM -0700, Bart Van Assche wrote:
> On 9/15/19 6:36 PM, Honggang LI wrote:
> > From: Honggang Li <honli@xxxxxxxxxx>
> >
> > The 'Send Message Size' field of IOControllerProfile attributes
> > contains the maximum initiator to target IU size.
> >
> > When there is something wrong with SRP login to a third party
> > SRP target, whose ib_srpt parameters can't be collected with
> > ordinary method, dump the 'Send Message Size' may help us to
> > diagnose the problem.
> >
> > Signed-off-by: Honggang Li <honli@xxxxxxxxxx>
> > ---
> > srp_daemon/srp_daemon.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/srp_daemon/srp_daemon.c b/srp_daemon/srp_daemon.c
> > index 337b21c7..90533c77 100644
> > --- a/srp_daemon/srp_daemon.c
> > +++ b/srp_daemon/srp_daemon.c
> > @@ -1022,6 +1022,8 @@ static int do_port(struct resources *res, uint16_t pkey, uint16_t dlid,
> > pr_human(" vendor ID: %06x\n", be32toh(target->ioc_prof.vendor_id) >> 8);
> > pr_human(" device ID: %06x\n", be32toh(target->ioc_prof.device_id));
> > pr_human(" IO class : %04hx\n", be16toh(target->ioc_prof.io_class));
> > + pr_human(" Maximum initiator to target IU size: %d\n",
> > + be32toh(target->ioc_prof.send_size));
> > pr_human(" ID: %s\n", target->ioc_prof.id);
> > pr_human(" service entries: %d\n", target->ioc_prof.service_entries);
>
> How about using the terminology from the InfiniBand Architecture
As this is srp specific, so I suggest to use the terminology from
srp specification 'srp2r06'.
Table B.7 — IOControllerProfile attributes for SRP target ports
----------------------------------------------------------------
| Field | SRP requirement |
----------------------------------------------------------------
|(skip many lines)..... |
----------------------------------------------------------------
|Send Message Size |MAXIMUM INITIATOR TO TARGET IU LENGTH |
----------------------------------------------------------------
> Specification? This is what I found in release 1.3, table 306:
>
> "Maximum size of Send Messages in bytes"
>
> Thanks,
>
> Bart.
