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

Reply via email to