Re: [PATCH v11 00/22] Add HiSilicon RoCE driver

在 2016/7/2 17:39, Lijun Ou 写道: > The HiSilicon Network Substem is a long term evolution IP which is > supposed to be used in HiSilicon ICT SoCs. HNS (HiSilicon Network > Sybsystem) also has a hardware support of performing RDMA with > RoCEE. > The driver for HiSilicon RoCEE(RoCE Engine) is a

Re: [PATCH v11 00/22] Add HiSilicon RoCE driver

在 2016/7/2 17:39, Lijun Ou 写道: > The HiSilicon Network Substem is a long term evolution IP which is > supposed to be used in HiSilicon ICT SoCs. HNS (HiSilicon Network > Sybsystem) also has a hardware support of performing RDMA with > RoCEE. > The driver for HiSilicon RoCEE(RoCE Engine) is a

Re: [PATCH v11 00/22] Add HiSilicon RoCE driver

在 2016/7/2 20:58, Leon Romanovsky 写道: > On Sat, Jul 02, 2016 at 05:39:02PM +0800, Lijun Ou wrote: > > This v11 >> 28 files changed, 10626 insertions(+), 1 deletion(-) > > First version >> 27 files changed, 11670 insertions(+), 11 deletions(-) > > 1K LOC less, we are moving in right direction.

Re: [PATCH v10 07/22] IB/hns: Add event queue support

Hi, Leon 在 2016/6/24 23:46, Leon Romanovsky 写道: > On Thu, Jun 16, 2016 at 10:35:15PM +0800, Lijun Ou wrote: >> This patch added event queue support for RoCE driver. It is used >> for RoCE interrupt. RoCE includes 32 synchronous event irqs, 1 >> asynchronous event irq and 1 common overflow irq. >>

Re: [PATCH v10 05/22] IB/hns: Add initial profile resource

Hi, leon 在 2016/6/24 23:10, Leon Romanovsky 写道: > On Thu, Jun 16, 2016 at 10:35:13PM +0800, Lijun Ou wrote: >> This patch added the operation for cmd, and added some functions >> for initializing eq table and selecting cmd mode. >> >> Signed-off-by: Wei Hu >>

Re: [PATCH v10 04/22] IB/hns: Add RoCE engine reset function

Hi, Leon 在 2016/6/27 16:01, Leon Romanovsky 写道: > On Sat, Jun 25, 2016 at 06:25:37PM +0800, Wei Hu (Xavier) wrote: >> >> >> On 2016/6/24 22:59, Leon Romanovsky wrote: >>> On Thu, Jun 16, 2016 at 10:35:12PM +0800, Lijun Ou wrote: This patch mainly added reset flow of RoCE engine in RoCE

Re: [PATCH v10 01/22] net: hns: Add reset function support for RoCE driver

Hi, Leon 在 2016/6/24 19:49, Leon Romanovsky 写道: > On Thu, Jun 16, 2016 at 10:35:09PM +0800, Lijun Ou wrote: >> It added reset function for RoCE driver. RoCE is a feature of hns. >> In hip06 SoC, in RoCE reset process, it's needed to configure dsaf >> channel reset, port and sl map info. Reset

Re: [RESEND PATCH v9 00/22] Add HiSilicon RoCE driver

On 2016/6/8 19:55, Doug Ledford wrote: > On 6/8/2016 2:44 AM, Lijun Ou wrote: >> The HiSilicon Network Substem is a long term evolution IP which is >> supposed to be used in HiSilicon ICT SoCs. HNS (HiSilicon Network >> Sybsystem) also has a hardware support of performing RDMA with >> RoCEE. >>

Re: [PATCH v9 05/22] IB/hns: Add initial profile resource

Hi, On 2016/6/9 15:01, Leon Romanovsky wrote: > On Wed, Jun 01, 2016 at 11:37:47PM +0800, Lijun Ou wrote: >> This patch mainly configured some profile resoure. For example, >> vendor_id, hardware version, and some data structure sizes so on. >> >> Signed-off-by: Wei Hu >>

Re: [PATCH v6 00/21] Add HiSilicon RoCE driver

On 2016/5/4 2:48, Leon Romanovsky wrote: > On Thu, Apr 28, 2016 at 08:09:35PM +0800, Lijun Ou wrote: >> The HiSilicon Network Substem is a long term evolution IP which is >> supposed to be used in HiSilicon ICT SoCs. HNS (HiSilicon Network >> Sybsystem) also has a hardware support of performing

Re: [PATCH v5 09/21] IB/hns: Add hca support

On 2016/4/26 22:18, Leon Romanovsky wrote: > On Tue, Apr 26, 2016 at 02:34:44PM +0800, oulijun wrote: >> On 2016/4/24 15:54, Leon Romanovsky wrote: >>> On Sat, Apr 23, 2016 at 06:26:47PM +0800, Lijun Ou wrote: >>>> This patch mainly setup hca for RoCE. it will do

Re: [PATCH v5 09/21] IB/hns: Add hca support

On 2016/4/26 22:25, Jiri Pirko wrote: > Tue, Apr 26, 2016 at 04:18:21PM CEST, l...@kernel.org wrote: >> On Tue, Apr 26, 2016 at 02:34:44PM +0800, oulijun wrote: >>> On 2016/4/24 15:54, Leon Romanovsky wrote: > > > >>>>> +int hns_roce_bitmap_allo

Re: [PATCH v5 09/21] IB/hns: Add hca support

On 2016/4/24 15:54, Leon Romanovsky wrote: > On Sat, Apr 23, 2016 at 06:26:47PM +0800, Lijun Ou wrote: >> This patch mainly setup hca for RoCE. it will do a series of >> initial works as follows: >> 1. init uar table, allocate uar resource >> 2. init pd table >> 3. init cq table

Re: [RESEND PATCH V4 2/3] IB/hns: Add HiSilicon RoCE driver support

Hi, Leon Romanovsky On 2016/4/2 9:58, Leon Romanovsky wrote: > On Fri, Apr 01, 2016 at 05:21:31PM +0800, Lijun Ou wrote: >> The driver for HiSilicon RoCE is a platform driver. >> The driver will support multiple versions of hardware. Currently only "v1" >> for hip06 SoC is supported. >> The

Re: call attention to review

On 2016/3/24 14:17, Leon Romanovsky wrote: > On Thu, Mar 24, 2016 at 01:50:30PM +0800, oulijun wrote: >> Hi, >>I am Lijun Ou. I have sent the PATCH v4 of HiSilicon RoCE driver at March >> 22, 2016. >> if you are convenient, please help to review. Welcome to give yo

call attention to review

Hi, I am Lijun Ou. I have sent the PATCH v4 of HiSilicon RoCE driver at March 22, 2016. if you are convenient, please help to review. Welcome to give your reviewing. thanks Lijun Ou

Re: [PATCH v4 2/3] IB/hns: Add HiSilicon RoCE driver support

On 2016/3/23 2:54, Christoph Hellwig wrote: >> drivers/infiniband/Kconfig |1 + >> drivers/infiniband/hw/Makefile |1 + >> drivers/infiniband/hw/hisilicon/hns/Kconfig| 10 + > > To fit in with the other drivers drop the hisilicon level

Re: [PATCH 3/4] infiniband: hns: add Hisilicon RoCE support(driver code)

On 2016/3/15 2:20, Parav Pandit wrote: >>> >>> Since SRQ is not supported in this driver version, can you keep >>> remaining code base also to not bother about SRQ specifically >>> poll_cq_one, modify_qp, destroy_qp etc? >>> SRQ support can come as complete additional patch along with cmd_mask,

Re: [PATCH 2/3] net: hns: add Hisilicon RoCE support

On 2016/3/14 14:49, Leon Romanovsky wrote: > On Mon, Mar 14, 2016 at 09:12:28AM +0800, Yankejian (Hackim Yim) wrote: >> >> >> On 2016/3/12 18:43, Leon Romanovsky wrote: >>> On Fri, Mar 11, 2016 at 06:37:10PM +0800, Lijun Ou wrote: It added hns_dsaf_roce_reset routine for roce driver.

Re: [PATCH 3/3] infiniband: IB/hns: add Hisilicon RoCE support with bindings

On 2016/3/11 21:39, Sergei Shtylyov wrote: > Hello. > > On 3/11/2016 1:37 PM, Lijun Ou wrote: > >> This submit add binding file and dts file. > >I see no .dts file. > >> >> Signed-off-by: Lijun Ou >> Signed-off-by: Wei Hu(Xavier) >> --- >>

Re: [PATCH 0/3] infiniband: IB/hns: Hisilicon RoCE support

On 2016/3/12 18:20, Leon Romanovsky wrote: > On Fri, Mar 11, 2016 at 06:37:08PM +0800, Lijun Ou wrote: > > 1) It is redundant to write "infiniband" and "IB" in one title to mention > relevant subsystem, since it is the same. > > Please take a look on the other submissions here on the list and

Re: [PATCH 1/3] infiniband: IB/hns: add Hisilicon RoCE support

On 2016/3/17 14:43, Leon Romanovsky wrote: > On Wed, Mar 16, 2016 at 11:36:38AM +0100, Jiri Pirko wrote: >>> so, I continue to have it. >> >> I will continue to bash on your odd codingstyle. Please fix it! > > Jiri, > > Checkpatch errors is an easiest issue with this patch. > > It is full of

Re: [PATCH 1/3] infiniband: IB/hns: add Hisilicon RoCE support

Hi dledford, thanks for reviewing. I have modified according to your reivews. I will send a new patch at soon. thanks Lijun Ou On 2016/3/12 18:39, Leon Romanovsky wrote: > On Fri, Mar 11, 2016 at 06:37:09PM +0800, Lijun Ou wrote: >> The driver for Hisilicon RoCE is a platform driver. >> The

Re: [PATCH 1/3] infiniband: IB/hns: add Hisilicon RoCE support

Hi Jiri Pirko, thanks your reviewing. sorry, I will send a new patch according to your reviews. On 2016/3/11 18:42, Jiri Pirko wrote: > Fri, Mar 11, 2016 at 11:37:09AM CET, ouli...@huawei.com wrote: >> The driver for Hisilicon RoCE is a platform driver. >> The driver will support mulitple

Re: [PATCH 3/4] infiniband: hns: add Hisilicon RoCE support(driver code)

Hi Parav Pandit, thanks your reviewing. On 2016/3/4 17:37, Parav Pandit wrote: > On Fri, Mar 4, 2016 at 2:11 PM, Wei Hu(Xavier) > wrote: >> + >> +int hns_roce_register_device(struct hns_roce_dev *hr_dev) >> +{ >> + int ret; >> + struct hns_roce_ib_iboe *iboe

Re: [PATCH 4/4] infiniband: hns: add Hisilicon RoCE support(Kconfig)

It will be send soon. thanks Lijun Ou > >> >> Signed-off-by: Wei Hu(Xavier) <xavier.hu...@huawei.com> >> Signed-off-by: oulijun <ouli...@huawei.com> > > ^^^.. -- Is this first name or second name? > >> --- >> drivers/inf

Re: [PATCH 1/4] net: hns: add Hisilicon RoCE support(the dependent routine)

for roce driver. >> RoCE is a feature of hns. >> In hip06 SOC, in roce reset process, it's needed to configure >> dsaf channel reset,port and sl map info. >> >> Signed-off-by: Wei Hu(Xavier) <xavier.hu...@huawei.com> >> Signed-off-by: Lisheng <lisheng.

Re: [PATCH 3/4] infiniband: hns: add Hisilicon RoCE support(driver code)

Hi Jiri Pirko, thanks for reviewing On 2016/3/4 17:16, Jiri Pirko wrote: > Fri, Mar 04, 2016 at 09:41:16AM CET, xavier.hu...@huawei.com wrote: > > > >> +int hns_roce_buf_alloc( >> +struct hns_roce_dev *hr_dev, >> +int size, int max_direct, >> +

Re: [PATCH 2/4] infiniband: hns: add Hisilicon RoCE support(binding)

t;> This submit add binding file and dts file. >> >> Signed-off-by: Wei Hu(Xavier) <xavier.hu...@huawei.com> >> Signed-off-by: oulijun <ouli...@huawei.com> >> --- >> .../bindings/infiniband/hisilicon-hns-roce.txt | 68 >> ++