Re: [PATCH rdma-core 1/2] rdma-core/cxgb4: Add support for srq functions & structs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux