Re: [PATCH V2 rdma-core 0/5] Userspace library matching for hip08 RoCE

[Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 2668 bytes --]

On Tue, Oct 24, 2017 at 04:41:57PM +0800, oulijun wrote:
> 在 2017/10/24 12:37, Leon Romanovsky 写道:
> > On Tue, Oct 24, 2017 at 10:35:45AM +0800, Lijun Ou wrote:
> >> This patch series add userspace library for hip08 RoCE driver. It will
> >> work with the kernel driver of hip08 RoCE(hns-roce-hw-v2.ko).
> >>
> >> Change from V1:
> >> - Fix build warning due to the strange placement of fprintf
> >>
> >
> > It still fails to build cleanly.
> > https://travis-ci.org/linux-rdma/rdma-core/builds/291891845?utm_source=github_status&utm_medium=notification
> >
> > [74/190] Building C object providers/h...s/hns-rdmav16.dir/hns_roce_u_hw_v2.c.o
> > FAILED: /usr/bin/clang-4.0 -Dhns_rdmav16_EXPORTS -Werror -Wall -Wextra -Wno-sign-compare -Wno-unused-parameter -Wmissing-prototypes -Wmissing-declarations -Wwrite-strings -Wformat=2 -Wshadow -Wstrict-prototypes -Wold-style-definition -Wredundant-decls -O2 -g -fPIC -Iinclude -I/usr/include/libnl3 -std=gnu11 -MMD -MT providers/hns/CMakeFiles/hns-rdmav16.dir/hns_roce_u_hw_v2.c.o -MF providers/hns/CMakeFiles/hns-rdmav16.dir/hns_roce_u_hw_v2.c.o.d -o providers/hns/CMakeFiles/hns-rdmav16.dir/hns_roce_u_hw_v2.c.o -c ../providers/hns/hns_roce_u_hw_v2.c
> > ../providers/hns/hns_roce_u_hw_v2.c:40:20: error: unused function 'set_raddr_seg_v2' [-Werror,-Wunused-function]
> > static inline void set_raddr_seg_v2(struct hns_roce_v2_wqe_raddr_seg *rseg,
> >  ^
> > 1 error generated.
> > [74/190] Building C object providers/h...s/hns-rdmav16.dir/hns_roce_u_hw_v1.c.o
> >
> > Please use buildlib/cbuild script to test your patch series, the option "travis" will run the same checkers as I'm running.
> >
> > Thanks
> >
> Hi， leon
>  sorry, I am fail to run cbuild tools.
>  the cmd as follows:
>  oulijun@Turing-Arch-b:~/rdma/rdma-core/buildlib$ ./cbuild pkg travis
> HEAD is now at 5d410f5 libhns: Update consolidate repo for userspace library of hip08 RoCE
>
> Can I use the cbuild tools quickly? Can I use buildlib/cbuild scripts for check?

It was our intention.

You are supposed to run cbuild script directly from source root directory.

There is extensive help in the beginning of cbuild script, but it can be
narrowed to the following three commands:

1) Create and download docker images (you are supposed to run this once
only, per requested operating system)
➜  rdma-core git:(master) ./buildlib/cbuild build-images travis

2) Compile for the target
➜  rdma-core git:(master) ./buildlib/cbuild make travis

3) Actually run all scripts specific to that target
➜  rdma-core git:(master) ./buildlib/cbuild pkg travis

Thanks

>
> thanks
> Lijun Ou
>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]