[openib-general] [PATCH] rdma_cm: let SDP control the SDP version in the hello header

[Sean Hefty]

>I think if you want to do it this way you should check only the MajV.
>SDP spec says:
>
>A4.3.2.1.1 MAJOR PROTOCOL VERSION NUMBER (MAJV) - 4 BITS
>The current specification requires MajV to be set to 2. See section A4.5.1
>Connection Setup on page 1218 for additional information.
>CA4-15: The accepting peer shall reject the connection if MajV in the HH
>does not match its local value.
>CA4-16: The accepting peer shall not reject the connection, solely on the
>basis that MinV of the HH does not match its local value.
>This enables future protocol extensions which are upwardly compatible.

I agree, the CMA should only check the major version number then.  If we're in
agreement on this approach, I will update the CMA to check the major version,
and expect that SDP will set the version information.

- Sean