[openib-general] tavor quirks etc (opensm compliance etc)

[Or Gerlitz]

Eitan Zahavi wrote:
> According to the IBTA spec:
> 1. In order for MTU and MTUSelector to have any effect their component 
> mask bits MUST be set to 1 in the query
> 2. Behavior of the SM is defined with small "freedom" to choose between 
> multiple matching MTU values if they exist.
> 3. The table below summarizes all options:
> 
> Assuming the value M represents the lowest MTU on the path
> We denote by M-1 the MTU value one level below M (e.g. 1K if M=2K)
> R represents the MTU value in the request. Similarly R-1 is one below R 
> and R+1 is one above R.
> 
> Query-MTU | Query-Sel | Resp by Spec     | OpenSM Should  | OpenSM Quirk 
> w. Tavor End Port
> ----------------------------------------------------------------------------------------- 
> 
> UNDEFINED | UNDEFINED | <= M             | M              | min(M,1K)
> R         | <         | <= min(R-1, M)   | min(R-1, M)    | min(R-1, M, 1K)
> R         | =         | R if M>=R /ERR   | R if M>=R /ERR | R if M>=R /ERR
> R         | >         | R < <= M         | R+1 if M>R /ERR| R+1 if M>R /ERR
> 
> I have built some test code for making sure OpenSM does what is required.
> Apparently it does not. In any case the M is not identical to R it fails 
> the request.
> 
> I am working on fixing OpenSM.
> 
> Any comments are welcome.

OK Eitan, thanks for putting the time on this, we will be able to 
provide feedback tomorrow or later this week.

Is the direction suggested by this matrix and patch that follows 
orthogonal to the open-sm tavor quirk patch?

Or.