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 > > Jason > Perhaps what is missing is the patch created by 'kernel-headers/update --not-final'? Should that be included in this series? I apologize for 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. Steve. -- 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