From: oulijun <oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
To: Jason Gunthorpe
<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>,
Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@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: Sun, 13 Nov 2016 18:21:35 +0800 [thread overview]
Message-ID: <58283EAF.7020508@huawei.com> (raw)
In-Reply-To: <20161111160202.GA22935-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
在 2016/11/12 0:02, Jason Gunthorpe 写道:
> On Fri, Nov 11, 2016 at 02:42:40PM +0200, Leon Romanovsky wrote:
>
>>> 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 ..
>
> For the 32 bit bugs all you need to do is run on x86 with the 32 bit
> compiler support installed:
>
> $ CFLAGS=-m32 cmake -GNinja ..
> $ ninja
>
>> 4. Send pull request to rdma-core via github interface and it will check
>> automatically.
>
> This is certainly a better option than having Leon test your patches,
> just keep force pushing to your own pull request until travis is
> happy.
>
> Make sure the 32 bit problems are fixed properly, use inttypes.h,
> uintptr, and so forth.
>
> Jason
>
Thanks your guide. I have test it by travis CI successfully and sent tha patchV5
Lijun Ou
> .
>
--
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
next prev parent reply other threads:[~2016-11-13 10:21 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 ` [PATCH v3 rdma-core 0/7] libhns: userspace library for hns Leon Romanovsky
[not found] ` <20161110161550.GD28957-2ukJVAZIZ/Y@public.gmane.org>
2016-11-11 6:37 ` 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 [this message]
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=58283EAF.7020508@huawei.com \
--to=oulijun-hv44wf8li93qt0dzr+alfa@public.gmane.org \
--cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
--cc=leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linuxarm-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).