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

[Ira Weiny]

On Tue, 5 May 2009 22:05:46 +0300
Sasha Khapyorsky <sashak at voltaire.com> wrote:

> 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.

Ah, ok, I see now.  I mixed up my bits...  ;-)

Ira

> 
> Sasha


-- 
Ira Weiny
Math Programmer/Computer Scientist
Lawrence Livermore National Lab
weiny2 at llnl.gov