On Tue, Jul 10, 2018 at 12:48:38PM -0500, Steve Wise wrote: > > On 7/10/2018 12:01 PM, Jason Gunthorpe wrote: > > On Tue, Jul 10, 2018 at 11:36:00AM -0500, Steve Wise wrote: > >> > >> On 7/10/2018 11:18 AM, Jason Gunthorpe wrote: > >>> On Tue, Jul 10, 2018 at 09:40:05PM +0530, Raju Rangoju wrote: > >>>> On Tuesday, July 07/10/18, 2018 at 09:44:10 -0600, Jason Gunthorpe wrote: > >>>>> On Tue, Jul 10, 2018 at 12:52:07PM +0530, Raju Rangoju wrote: > >>>>>> This patch adds user mode t4_srq structures and support functions > >>>>>> - Also adds srq resp structures and helper functions > >>>>>> > >>>>>> Signed-off-by: Raju Rangoju <rajur@xxxxxxxxxxx> > >>>>>> Reviewed-by: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx> > >>>>>> providers/cxgb4/cxgb4-abi.h | 2 + > >>>>>> providers/cxgb4/libcxgb4.h | 22 ++++++ > >>>>>> providers/cxgb4/t4.h | 155 +++++++++++++++++++++++++++++++++++++++++- > >>>>>> providers/cxgb4/t4_regs.h | 4 ++ > >>>>>> providers/cxgb4/t4fw_api.h | 2 + > >>>>>> providers/cxgb4/t4fw_ri_api.h | 19 ++++++ > >>>>>> 6 files changed, 201 insertions(+), 3 deletions(-) > >>>>>> > >>>>>> diff --git a/providers/cxgb4/cxgb4-abi.h b/providers/cxgb4/cxgb4-abi.h > >>>>>> index 67d44512..d5140660 100644 > >>>>>> +++ b/providers/cxgb4/cxgb4-abi.h > >>>>>> @@ -59,6 +59,8 @@ DECLARE_DRV_CMD(uc4iw_alloc_pd, IB_USER_VERBS_CMD_ALLOC_PD, > >>>>>> empty, c4iw_alloc_pd_resp); > >>>>>> DECLARE_DRV_CMD(uc4iw_create_cq, IB_USER_VERBS_CMD_CREATE_CQ, > >>>>>> c4iw_create_cq, c4iw_create_cq_resp); > >>>>>> +DECLARE_DRV_CMD(uc4iw_create_srq, IB_USER_VERBS_CMD_CREATE_SRQ, > >>>>>> + empty, c4iw_create_srq_resp); > >>>>> Where did c4iw_create_srq_resp come from? It is not in this series? > >>>>> > >>>> Yes, it is in the same series. Patch 2/2 > >>> ? Where? > >> Raju, I don't see it in the 2nd patch either. > >> > >>> Please send these larger things to github so we can see the travis > >>> results. > >> Hey Jason, are you asking for Raju to clone the github repo via the > >> github gui, and then create a pull request? > > Yes > > Perhaps what is missing is the patch created by 'kernel-headers/update > the ignorance, but I haven't added new ABI stuff on the user side with > the new bldenv. This is also needed for the prerequisite 64BCQE patch. Yes, those need to be included. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html