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 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



[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