[openib-general] Fwd: a bug

[Michael S. Tsirkin]

Roland, the following makes sense, does it not?
The spec says:

14.2.5.5 GUIDInfo
The AttributeModifier is a pointer to a block of 8 GUIDs to which this attribute
applies.

and each guid is 64 bit.


----- Forwarded message from Leonid Keller <leonid at mellanox.co.il> -----

Date: Tue, 18 Apr 2006 10:49:06 +0300
From: "Leonid Keller" <leonid at mellanox.co.il>

   look at mthca_query_gid() at the end.
The statement 
    memcpy(gid->raw + 8, out_mad->data + (index % 8) * 16, 8);
is buggy, because the mad contains the block of GUIDs, not GIDs.
Must be
    memcpy(gid->raw + 8, out_mad->data + (index % 8) * 8, 8);


----- End forwarded message -----

-- 
MST