mbox series

[rdma-core,0/2] Two updates for libhns

Message ID 1532509409-183771-1-git-send-email-oulijun@huawei.com (mailing list archive)
Headers show
Series Two updates for libhns | expand

Message

Lijun Ou July 25, 2018, 9:03 a.m. UTC
This patch series add a verbs of modifying cq for libhns
as well as fix an immediate data problem.

Lijun Ou (2):
  libhns: Update the data type of immediate data
  libhns: Modify cq support for libhns

 providers/hns/hns_roce_u.c       | 1 +
 providers/hns/hns_roce_u.h       | 1 +
 providers/hns/hns_roce_u_hw_v2.c | 6 +++---
 providers/hns/hns_roce_u_hw_v2.h | 4 ++--
 providers/hns/hns_roce_u_verbs.c | 7 +++++++
 5 files changed, 14 insertions(+), 5 deletions(-)

Comments

Jason Gunthorpe Aug. 3, 2018, 2:55 a.m. UTC | #1
On Wed, Jul 25, 2018 at 05:03:27PM +0800, Lijun Ou wrote:
> This patch series add a verbs of modifying cq for libhns
> as well as fix an immediate data problem.
> 
> Lijun Ou (2):
>   libhns: Update the data type of immediate data
>   libhns: Modify cq support for libhns
> 
>  providers/hns/hns_roce_u.c       | 1 +
>  providers/hns/hns_roce_u.h       | 1 +
>  providers/hns/hns_roce_u_hw_v2.c | 6 +++---
>  providers/hns/hns_roce_u_hw_v2.h | 4 ++--
>  providers/hns/hns_roce_u_verbs.c | 7 +++++++
>  5 files changed, 14 insertions(+), 5 deletions(-)

Can you send this and the other one as a pull request to rdma-core's
github? It is easier to see that the travis passes/etc

Thanks,
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Lijun Ou Aug. 8, 2018, 7:49 a.m. UTC | #2
在 2018/8/3 10:55, Jason Gunthorpe 写道:
> On Wed, Jul 25, 2018 at 05:03:27PM +0800, Lijun Ou wrote:
>> This patch series add a verbs of modifying cq for libhns
>> as well as fix an immediate data problem.
>>
>> Lijun Ou (2):
>>   libhns: Update the data type of immediate data
>>   libhns: Modify cq support for libhns
>>
>>  providers/hns/hns_roce_u.c       | 1 +
>>  providers/hns/hns_roce_u.h       | 1 +
>>  providers/hns/hns_roce_u_hw_v2.c | 6 +++---
>>  providers/hns/hns_roce_u_hw_v2.h | 4 ++--
>>  providers/hns/hns_roce_u_verbs.c | 7 +++++++
>>  5 files changed, 14 insertions(+), 5 deletions(-)
> 
> Can you send this and the other one as a pull request to rdma-core's
> github? It is easier to see that the travis passes/etc
> 
> Thanks,
> Jason
> 
> 
Hi, Jason
  I am trying to send the pull reqeust according to your advice and it is fail.
the patch series is generated based on rdma-core.I need to the permission to
create a new branch based on the rdma-core's github and use git pull-reqeust cmd to
generate the pull reqeust. is ok?

Thanks
Lijun Ou

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Leon Romanovsky Aug. 8, 2018, 10:50 a.m. UTC | #3
On Wed, Aug 08, 2018 at 03:49:52PM +0800, oulijun wrote:
> 在 2018/8/3 10:55, Jason Gunthorpe 写道:
> > On Wed, Jul 25, 2018 at 05:03:27PM +0800, Lijun Ou wrote:
> >> This patch series add a verbs of modifying cq for libhns
> >> as well as fix an immediate data problem.
> >>
> >> Lijun Ou (2):
> >>   libhns: Update the data type of immediate data
> >>   libhns: Modify cq support for libhns
> >>
> >>  providers/hns/hns_roce_u.c       | 1 +
> >>  providers/hns/hns_roce_u.h       | 1 +
> >>  providers/hns/hns_roce_u_hw_v2.c | 6 +++---
> >>  providers/hns/hns_roce_u_hw_v2.h | 4 ++--
> >>  providers/hns/hns_roce_u_verbs.c | 7 +++++++
> >>  5 files changed, 14 insertions(+), 5 deletions(-)
> >
> > Can you send this and the other one as a pull request to rdma-core's
> > github? It is easier to see that the travis passes/etc
> >
> > Thanks,
> > Jason
> >
> >
> Hi, Jason
>   I am trying to send the pull reqeust according to your advice and it is fail.
> the patch series is generated based on rdma-core.I need to the permission to
> create a new branch based on the rdma-core's github and use git pull-reqeust cmd to
> generate the pull reqeust. is ok?

No, you need to fork https://github.com/linux-rdma/rdma-core/ to your
local github place, create new branch there and send pull request from
that repository to the main one.

Thanks

>
> Thanks
> Lijun Ou
>