[ofa-general] Re: Issues with combined routing in smpquery

[Sasha Khapyorsky]

Hi Ira,

On 16:04 Wed 29 Apr     , Ira Weiny wrote:
> 
> I know what changed but there appears to be a discrepancy between ib_mad_f 
> and the spec.
> 
> Commit 2dbb8b95d9dc27423a6fdb85d88ef385ecee0005
>    "libibmad: remove c99 definitions within the ib_mad_f structure"
> removed the designated initializers from ib_mad_f.  Appling the patch below
> aligns the MAD_FIELDS with ib_mad_f.

Thanks for looking into this.

> However, if you look at the offsets specified in ib_mad_f they are wrong.
> According to 14.2.1.2, DrSLID is at offset 32 bytes (256 bits).  ib_mad_f
> places the offset at 272.  I have verified the bytes using a debugger and byte
> 32 is the DrSLID.  I hesitate to say there is a bug in mad_set_field however
> there does appear to be something amiss.  :-/

I think everything is ok there. 14.2.1.2 says: at offset 32 bytes (256
bits) DrDLID - bits 0-15, DrSLID - bits 16-31.

Sasha