All of lore.kernel.org
 help / color / mirror / Atom feed
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 --]

  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.