[openib-general] Some CM Comments

[Hal Rosenstock]

Hi Sean,

I've been looking over ib_cm.h and have some comments:

1.IBA 1.2 CM is still class version 2 and is upward compatible with 1.1
CM. The only difference appears to be SRQ component added to REQ and
REP.

2. The following CM states appear to be missing:
peer compare
reject retry
timeout
DREQ timeout (is an event) It is also a state in the spec
RTU timeout (nor is it an event)

3. There is no REJ received CM event type. Shouldn't there be ?

4. Should there be an API to obtain local/remote comm IDs ?

5. REQ message is missing transport service type (connection type).
Also, what about ack timeout ? Also, SRQ. Is starting PSN chosen
internally by the CM ?

6. SRQ should be added for REP.

Is there a danger to have the client set the CM response timeout ? Also,
is the lack of remote response timeout for LAP in inconsistent with this
?

Thanks.

-- Hal