Thread (25 messages) 25 messages, 5 authors, 2017-09-28

Re: [PATCH v5] net/mlx5: support upstream rdma-core

From: Yigit, Ferruh <hidden>
Date: 2017-09-18 16:05:35

Thanks Shachar,

I have seen v7 [1], I will wait this and depended two patches to be acked before getting them.

Thanks,
ferruh

[1]
http://dpdk.org/dev/patchwork/patch/28852/
-----Original Message-----
From: Shachar Beiser [mailto:shacharbe@mellanox.com]
Sent: Monday, September 18, 2017 2:43 PM
To: Yigit, Ferruh <redacted>; dev@dpdk.org
Cc: Adrien Mazarguil <redacted>; Nélio Laranjeiro
[off-list ref]
Subject: RE: [dpdk-dev] [PATCH v5] net/mlx5: support upstream rdma-core


quoted
-----Original Message-----
From: Ferruh Yigit [mailto:ferruh.yigit@intel.com]
Sent: Monday, September 18, 2017 12:07 PM
To: Shachar Beiser <redacted>; dev@dpdk.org
Cc: Adrien Mazarguil <redacted>; Nélio Laranjeiro
[off-list ref]
Subject: Re: [dpdk-dev] [PATCH v5] net/mlx5: support upstream
rdma-core

On 9/17/2017 8:31 AM, Shachar Beiser wrote:
quoted
quoted
-----Original Message-----
From: Ferruh Yigit [mailto:ferruh.yigit@intel.com]
Sent: Thursday, September 14, 2017 4:47 PM
To: Shachar Beiser <redacted>; dev@dpdk.org
Cc: Adrien Mazarguil <redacted>; Nélio Laranjeiro
[off-list ref]
Subject: Re: [dpdk-dev] [PATCH v5] net/mlx5: support upstream
rdma-core

On 9/14/2017 2:34 PM, Shachar Beiser wrote:
quoted
This removes the dependency on specific Mellanox OFED libraries by
using the upstream rdma-core and linux upstream community code.
Would you mind mentioning what is the difference between "specific
Mellanox OFED libraries" and "upstream rdma-core"? If not in the
documentation, at least in mail list for interested people?

Does it make sense to put this update into release notes?

Thanks,
ferruh
 Hi Ferruh,

Both packages, rdma-core upstream &  Mellanox OFED are Linux user-
space packages :
quoted
 	1. Rdma-core is the Linux upstream user-space package.
 	2. Mellanox OFED  is the Mellanox's Linux user-space package.
 The difference is the APIs . We shall explain that difference in
the release notes .
The difference can be too much detail for release notes, I believe it
is good to mention in release notes the dependency requirement change,
as above commit log.

But this commit log can have the detail.
I will send a new PATCHv6 with an updated commit log . That contains this
explanation above. The release notes will updated by Shahaf Shuler separately.
quoted
quoted
             -Shachar Beiser.
quoted
quoted
---
I have squashed :
[PATCH v2 3/3] “net/mlx5: fix interrupt enable return”
https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpd
quoted
quoted
k
quoted
.org%2Fdev%2Fpatchwork%2Fpatch%2F28380%2F&data=02%7C01%7Cshach
quoted
quoted
arbe%40m
quoted
ellanox.com%7Cc79d7690cb454f8dd41508d4fb771dea%7Ca652971c7d2e4d9b
quoted
quoted
quoted
a6a4d
quoted
149256f461b%7C0%7C0%7C636409936407431479&sdata=T5wgSUeCx1tzM9Z
quoted
quoted
quoted
UnBAauNt
quoted
iG4djfIpOJZOj%2FWM6v4Y%3D&reserved=0
into this patch

a. Compile with rdma-core commit f11292efd541 ("Merge pull request
#202") b. Tested with linux kernel 4.13-rc4 c. For performance
testing recommended to wait till kernel 4.14

Signed-off-by: Shachar Beiser <redacted>
<...>
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help