From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Lijun Ou <oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linuxarm-hv44wF8Li93QT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH v3 rdma-core 0/7] libhns: userspace library for hns
Date: Thu, 10 Nov 2016 18:15:50 +0200 [thread overview]
Message-ID: <20161110161550.GD28957@leon.nu> (raw)
In-Reply-To: <1478781977-116803-1-git-send-email-oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 2435 bytes --]
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
>
> 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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-11-10 16:15 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-10 12:46 [PATCH v3 rdma-core 0/7] libhns: userspace library for hns Lijun Ou
[not found] ` <1478781977-116803-1-git-send-email-oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2016-11-10 12:46 ` [PATCH v3 rdma-core 1/7] libhns: Add initial main frame Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 2/7] libhns: Add verbs of querying device and querying port Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 3/7] libhns: Add verbs of pd and mr support Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 4/7] libhns: Add verbs of cq support Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 5/7] libhns: Add verbs of qp support Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 6/7] libhns: Add verbs of post_send and post_recv support Lijun Ou
2016-11-10 12:46 ` [PATCH v3 rdma-core 7/7] libhns: Add consolidated repo for userspace library of hns Lijun Ou
2016-11-10 16:15 ` Leon Romanovsky [this message]
[not found] ` <20161110161550.GD28957-2ukJVAZIZ/Y@public.gmane.org>
2016-11-11 6:37 ` [PATCH v3 rdma-core 0/7] libhns: userspace library for hns oulijun
[not found] ` <5825673A.1050109-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2016-11-11 12:42 ` Leon Romanovsky
[not found] ` <20161111124240.GG28957-2ukJVAZIZ/Y@public.gmane.org>
2016-11-11 16:02 ` Jason Gunthorpe
[not found] ` <20161111160202.GA22935-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-11-13 10:21 ` oulijun
2016-11-13 10:22 ` oulijun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161110161550.GD28957@leon.nu \
--to=leon-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linuxarm-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
--cc=oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).