Re: [PATCH v3 rdma-core 0/7] libhns: userspace library for hns

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

 



在 2016/11/11 0:15, Leon Romanovsky 写道:
> On Thu, Nov 10, 2016 at 08:46:10PM +0800, Lijun Ou wrote:
>> This patch series introduces userspace library for hns RoCE driver.
>>
>> changes v2 -> v3:
>> 1. Fix the code style, for example, if (addr == NULL)
>> 2. Fix the bug for hns_roce_u_reg_mr
>>
>> changes v1 -> v2:
>> 1. Delete the min() definition and instead of ccan header
>> 2. Delete the CHECK_C_SOURCE_COMPILES
>> 3. sort the c file in rdma_provider()
>> 4. Delete the unused code in hns_roce_u_db.h
>>
>> Lijun Ou (7):
>>   libhns: Add initial main frame
>>   libhns: Add verbs of querying device and querying port
>>   libhns: Add verbs of pd and mr support
>>   libhns: Add verbs of cq support
>>   libhns: Add verbs of qp support
>>   libhns: Add verbs of post_send and post_recv support
>>   libhns: Add consolidated repo for userspace library of hns
> 
> Hi Lijun,
> 
> I tried to take your patch set, but it fails to pass our Travis CI
> tests.
> 
> https://github.com/linux-rdma/rdma-core/pull/38
> https://travis-ci.org/linux-rdma/rdma-core/builds/174815897
> 
> You need to fix it prior to our acceptance.
> 
> Thanks
> 
Hi, Leon
  I have checked the patches and fixed it and send the patchv4. I think that
the patchv4 will not have errors and warnings for travis CI testing, but
I don't have the Travis CI environment for testing it.
I have tried to install the test environment according to your test log and it
is fail.
In addtion that, I have tested it for  patchv1/patchv2/patch3 by use the following way and it
is ok
 1. directly use the default script file build.sh
    ./build.sh
 2. use CC=aarch64-linux-gnu-gcc build
    mkdir build
    cd build
    CC=aarch64-linux-gnu-gcc cmake -Gninja -DENABLE_RESOLVE_NEIGH=0 ..

lijun Ou
> 
>>
>>  CMakeLists.txt                   |   1 +
>>  MAINTAINERS                      |   6 +
>>  README.md                        |   1 +
>>  providers/hns/CMakeLists.txt     |   6 +
>>  providers/hns/hns_roce_u.c       | 228 +++++++++++
>>  providers/hns/hns_roce_u.h       | 255 ++++++++++++
>>  providers/hns/hns_roce_u_abi.h   |  69 ++++
>>  providers/hns/hns_roce_u_buf.c   |  61 +++
>>  providers/hns/hns_roce_u_db.h    |  54 +++
>>  providers/hns/hns_roce_u_hw_v1.c | 839 +++++++++++++++++++++++++++++++++++++++
>>  providers/hns/hns_roce_u_hw_v1.h | 242 +++++++++++
>>  providers/hns/hns_roce_u_verbs.c | 525 ++++++++++++++++++++++++
>>  12 files changed, 2287 insertions(+)
>>  create mode 100644 providers/hns/CMakeLists.txt
>>  create mode 100644 providers/hns/hns_roce_u.c
>>  create mode 100644 providers/hns/hns_roce_u.h
>>  create mode 100644 providers/hns/hns_roce_u_abi.h
>>  create mode 100644 providers/hns/hns_roce_u_buf.c
>>  create mode 100644 providers/hns/hns_roce_u_db.h
>>  create mode 100644 providers/hns/hns_roce_u_hw_v1.c
>>  create mode 100644 providers/hns/hns_roce_u_hw_v1.h
>>  create mode 100644 providers/hns/hns_roce_u_verbs.c
>>
>> --
>> 1.9.1
>>
>> --
>> 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


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