| Message ID | 20170406223314.GA3893@obsidianresearch.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Thu, Apr 06, 2017 at 4:33:14PM -0600, Jason Gunthorpe wrote: > imm_data is copied directly from the ib_send_wr and ib_wc which have > it marked as __be32, copy that mark into the uapi structures as well. > > Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> > --- > include/uapi/rdma/vmw_pvrdma-abi.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > This fixes sparse errors in user space and kernel space > Thanks! Tested-by: Adit Ranadive <aditr@vmware.com> Acked-by: Adit Ranadive <aditr@vmware.com> -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, 2017-04-06 at 18:17 -0700, Adit Ranadive wrote: > On Thu, Apr 06, 2017 at 4:33:14PM -0600, Jason Gunthorpe wrote: > > > > imm_data is copied directly from the ib_send_wr and ib_wc which > > have > > it marked as __be32, copy that mark into the uapi structures as > > well. > > > > Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> > > --- > > include/uapi/rdma/vmw_pvrdma-abi.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > This fixes sparse errors in user space and kernel space > > > > Thanks! > > Tested-by: Adit Ranadive <aditr@vmware.com> > Acked-by: Adit Ranadive <aditr@vmware.com> Thanks, applied.
diff --git a/include/uapi/rdma/vmw_pvrdma-abi.h b/include/uapi/rdma/vmw_pvrdma-abi.h index 5016abc9ee9712..c8c1d2d6df4d52 100644 --- a/include/uapi/rdma/vmw_pvrdma-abi.h +++ b/include/uapi/rdma/vmw_pvrdma-abi.h @@ -222,7 +222,7 @@ struct pvrdma_sq_wqe_hdr { __u32 opcode; /* operation type */ __u32 send_flags; /* wr flags */ union { - __u32 imm_data; + __be32 imm_data; __u32 invalidate_rkey; } ex; __u32 reserved; @@ -273,7 +273,7 @@ struct pvrdma_cqe { __u32 opcode; __u32 status; __u32 byte_len; - __u32 imm_data; + __be32 imm_data; __u32 src_qp; __u32 wc_flags; __u32 vendor_err;
imm_data is copied directly from the ib_send_wr and ib_wc which have it marked as __be32, copy that mark into the uapi structures as well. Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com> --- include/uapi/rdma/vmw_pvrdma-abi.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) This fixes sparse errors in user space and kernel space