On Sat, Oct 23, 2010 at 12:03:18AM -0700, Hefty, Sean wrote:
> +struct umad_smp {
> + uint8_t base_version;
> + uint8_t mgmt_class;
> + uint8_t class_version;
> + uint8_t method;
> + uint16_t status;
> + uint8_t hop_ptr;
> + uint8_t hop_cnt;
> + uint64_t tid;
> + uint16_t attr_id;
> + uint16_t resv;
> + uint32_t attr_mod;
> + uint64_t mkey;
These uint64_t's should also be the special 64 bit type I mentioned
before. The problem here is that uint64_t increases the alignment
requirement of the struct to 8, while its natural place, in say, an IB
packet (LRH+BTH+DETH+SMP) only has an alignment of 4.
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html