Ack. That's a bug. Will fix. Upinder
On Dec 10, 2013, at 1:14 PM, Roland Dreier <rol...@purestorage.com> wrote: > On Mon, Sep 9, 2013 at 8:38 PM, Upinder Malhi (umalhi) <uma...@cisco.com> > wrote: >> +/*TODO: Future - usnic_modify_qp needs to pass in generic filters */ >> +struct usnic_ib_create_qp_resp { >> + u32 vfid; >> + u32 qp_grp_id; >> + u64 bar_bus_addr; >> + u32 bar_len; >> +/* >> + * WQ, RQ, CQ are explicity specified bc exposing a generic resources >> inteface >> + * expands the scope of ABI to many files. >> + */ >> + u32 wq_cnt; >> + u32 rq_cnt; >> + u32 cq_cnt; >> + u32 wq_idx[USNIC_QP_GRP_MAX_WQS]; >> + u32 rq_idx[USNIC_QP_GRP_MAX_RQS]; >> + u32 cq_idx[USNIC_QP_GRP_MAX_CQS]; >> + u32 transport; > > Unless I'm misreading this, the u64 reserved field will have different > alignment on platforms that align u64 to 4 bytes (eg 32-bit x86) and > to 8 bytes (eg 64-bit x86) which means you'll have problems running > 32-bit userspace on a 64-bit kernel. > >> + u64 reserved; >> +}; -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html