From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bart Van Assche Subject: Re: [PATCH for-next V1] IB/usnic: Fix userspace/kernel ABI alignment discrepancy Date: Sat, 21 Dec 2013 10:52:00 +0100 Message-ID: <52B564C0.2090606@acm.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Upinder Malhi (umalhi)" , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" Cc: " Dreier" List-Id: linux-rdma@vger.kernel.org On 12/21/13 03:05, Upinder Malhi (umalhi) wrote: > Change from V0: > Make reserved field 9 bytes instead of 8 bytes. > > This patch depends on http://www.spinics.net/lists/linux-rdma/msg18193.html. > > Change the type of the reserve field to u32 from u64 in > usnic_ib_create_qp_resp bc u64 will align differently for > 32 and 64 bit archs. And reserve even more space for future > expansion. > > Signed-off-by: Upinder Malhi > --- > drivers/infiniband/hw/usnic/usnic_abi.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/infiniband/hw/usnic/usnic_abi.h b/drivers/infiniband/hw/usnic/usnic_abi.h > index 730a371..6c56fbf 100644 > --- a/drivers/infiniband/hw/usnic/usnic_abi.h > +++ b/drivers/infiniband/hw/usnic/usnic_abi.h > @@ -21,7 +21,7 @@ > #define USNIC_ABI_H > > /* ABI between userspace and kernel */ > -#define USNIC_UVERBS_ABI_VERSION 3 > +#define USNIC_UVERBS_ABI_VERSION 4 > > #define USNIC_QP_GRP_MAX_WQS 8 > #define USNIC_QP_GRP_MAX_RQS 8 > @@ -67,7 +67,7 @@ struct usnic_ib_create_qp_resp { > u32 rq_idx[USNIC_QP_GRP_MAX_RQS]; > u32 cq_idx[USNIC_QP_GRP_MAX_CQS]; > u32 transport; > - u64 reserved; > + u32 reserved[9]; > }; > > #endif /* USNIC_ABI_H */ Sorry but to me it seems like the patch description does not match the patch itself. In the patch I see an increase of the size of "reserved" from 8 to 36 bytes while the description says that the size of that member has been increased from 8 to 9 bytes ? Bart. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html