On Wed, Dec 12, 2018 at 07:45:41AM -0800, Kamenee Arumugam wrote: > hfi1_wc struct is the same struct as ib_uverbs_wc and hfi1_cq_wc > is not in the uapi header. Remove hfi1_wc and hfi1_cq_wc struct > from providers. Include rvt-abi.h header that have rvt_cq_wc > struct definition into kernel-headers. > > The member ib_uverbs_wc arrray in rvt_cq_wc is replaced with > one in rvt-abi. Therefore,plus one to the number of cq entries. > > Reviewed-by: Dennis Dalessandro <dennis.dalessandro@xxxxxxxxx> > Reviewed-by: Mike Marciniszyn <mike.marciniszyn@xxxxxxxxx> > Signed-off-by: Kamenee Arumugam <kamenee.arumugam@xxxxxxxxx> > kernel-headers/CMakeLists.txt | 1 + > providers/hfi1verbs/hfiverbs.h | 37 +++++-------------------------------- > providers/hfi1verbs/verbs.c | 23 +++++++++++++---------- > 3 files changed, 19 insertions(+), 42 deletions(-) > > diff --git a/kernel-headers/CMakeLists.txt b/kernel-headers/CMakeLists.txt > index e00eadb..e6b4d6e 100644 > +++ b/kernel-headers/CMakeLists.txt > @@ -25,6 +25,7 @@ publish_internal_headers(rdma > rdma/rdma_user_rxe.h > rdma/rvt-abi.h > rdma/vmw_pvrdma-abi.h > + rdma/rvt-abi.h Duplicate file These patch seem fine, but the process still not right, please work with Dennis.. - only one 'update kernel headers' patch - Send to the mailing list as a *series* not a random set of loose patches - Use a github PR if the kernel patches are not accepted yet Please resend them when the kernel side is accepted. Jason
