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-corequoted
-----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, ferruhHi 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%2Fdpdquoted
quoted
kquoted
.org%2Fdev%2Fpatchwork%2Fpatch%2F28380%2F&data=02%7C01%7Cshachquoted
quoted
arbe%40mquoted
ellanox.com%7Cc79d7690cb454f8dd41508d4fb771dea%7Ca652971c7d2e4d9bquoted
quoted
quoted
a6a4dquoted
149256f461b%7C0%7C0%7C636409936407431479&sdata=T5wgSUeCx1tzM9Zquoted
quoted
quoted
UnBAauNtquoted
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><...>