[dpdk-dev] [PATCH v3 3/3] doc: update mlx guides

2017-07-25 Thread Shahaf Shuler
Update the guides with: * New supported features. * Supported OFED and FW versions. * Quick start guide. * Performance tunning guide. Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil --- On v3: - Changed Limitiation and Known issues to Limitation o

[dpdk-dev] [PATCH v3 2/3] doc: update release notes for mlx driver

2017-07-25 Thread Shahaf Shuler
Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil --- doc/guides/rel_notes/release_17_08.rst | 126 + 1 file changed, 126 insertions(+) diff --git a/doc/guides/rel_notes/release_17_08.rst b/doc/guides/rel_notes/release_17_08.rst

[dpdk-dev] [PATCH v3 1/3] doc: update mlx5 supported features

2017-07-25 Thread Shahaf Shuler
Supported features which were not included: * ARMv8 * Extended stats Not supported features which were wrongly included: * Inner L3 checksum * Inner L4 checksum Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil --- doc/guides/nics/features/mlx5.ini | 4

Re: [dpdk-dev] [PATCH] net/i40e: fix PF notify issue when VF not up

2017-07-25 Thread Yang, Qiming
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Xiaoyun Li > Sent: Tuesday, July 25, 2017 2:43 AM > To: dev@dpdk.org > Cc: Wu, Jingjing ; Li, Xiaoyun ; > sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] net/i40e: fix PF notify issue when VF not up > > This patch

[dpdk-dev] [PATCH] net/mlx5: fix verification of device context

2017-07-25 Thread Shachar Beiser
Get interface name function lacks verification of device context. It might lead to segmentation fault when trying to query the name after the device is closed.fixing it by adding the missing verification Fixes: cd89f22a1e9770 ("net/mlx5: remove unused interface name query") Cc: sta...@dpdk.org Si

Re: [dpdk-dev] [pktgen PATCH 0/6] fix compilation

2017-07-25 Thread Thomas Monjalon
26/07/2017 06:54, Wiles, Keith: > > > On Jul 25, 2017, at 5:21 PM, Thomas Monjalon wrote: > > > > Some errors were seen when trying to compile with gcc 7 or clang, > > for 32-bit, and with DPDK 17.08-rc2. > > > > Thomas Monjalon (6): > > remove unused functions > > fix screen type saving > >

Re: [dpdk-dev] [PATCH v5] doc: document NIC features

2017-07-25 Thread Shreyansh Jain
Hello Ferruh, Some trivial comments inline. On Thursday 20 July 2017 02:53 PM, Ferruh Yigit wrote: Document NIC features, add more information about them and add more implementation related support. Signed-off-by: Ferruh Yigit Signed-off-by: John McNamara Reviewed-by: Andrew Rybchenko --- C

Re: [dpdk-dev] [pktgen PATCH 0/6] fix compilation

2017-07-25 Thread Roberts, Lee A.
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Wiles, Keith > Sent: Tuesday, July 25, 2017 9:54 PM > To: Thomas Monjalon > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [pktgen PATCH 0/6] fix compilation > > > > On Jul 25, 2017, at 5:21 PM, Thomas Monjalon wr

Re: [dpdk-dev] [pktgen PATCH 0/6] fix compilation

2017-07-25 Thread Wiles, Keith
> On Jul 25, 2017, at 5:21 PM, Thomas Monjalon wrote: > > Some errors were seen when trying to compile with gcc 7 or clang, > for 32-bit, and with DPDK 17.08-rc2. > > Thomas Monjalon (6): > remove unused functions > fix screen type saving > fix compilation of printf based functions > fix co

[dpdk-dev] [pktgen PATCH 5/6] fix 32-bit build

2017-07-25 Thread Thomas Monjalon
Signed-off-by: Thomas Monjalon --- app/cli-functions.c | 2 +- app/pktgen-cmds.c| 18 +- app/pktgen-latency.c | 26 ++ app/pktgen-port-cfg.h| 2 +- app/pktgen-stats.c | 47 --- lib/cli

[dpdk-dev] [pktgen PATCH 6/6] fix build with dpdk-17.08-rc2

2017-07-25 Thread Thomas Monjalon
The DEVTYPE API was changed in dpdk-17.08-rc1 and was reverted in dpdk-17.08-rc2. Fixes: 5653601bd0f5 ("fix changes for DPDK 17.08") Signed-off-by: Thomas Monjalon --- lib/common/port_config.c | 8 1 file changed, 8 deletions(-) diff --git a/lib/common/port_config.c b/lib/common/port_

[dpdk-dev] [pktgen PATCH 3/6] fix compilation of printf based functions

2017-07-25 Thread Thomas Monjalon
Functions with printf style must be declared with __attribute__((format(printf, Otherwise there is this compilation error: error: format string is not a string literal Other errors fixed: - "%*s" must match an int value - "% " is not a valid printf format string Signed-off-by: Tho

[dpdk-dev] [pktgen PATCH 4/6] fix compilation with gcc 7

2017-07-25 Thread Thomas Monjalon
Signed-off-by: Thomas Monjalon --- lib/common/mbuf.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/common/mbuf.h b/lib/common/mbuf.h index 0191b28..31bfa96 100644 --- a/lib/common/mbuf.h +++ b/lib/common/mbuf.h @@ -90,6 +90,7 @@ pg_pktmbuf_alloc_bulk(struct rte_mempool *pool,

[dpdk-dev] [pktgen PATCH 1/6] remove unused functions

2017-07-25 Thread Thomas Monjalon
clang compilations stops on unused static functions. Some are removed, other one is commented out, according to the context. Signed-off-by: Thomas Monjalon --- app/lpktgenlib.c | 12 ++-- lib/common/l2p.c | 34 -- 2 files changed, 2 insertions(+), 44 delet

[dpdk-dev] [pktgen PATCH 2/6] fix screen type saving

2017-07-25 Thread Thomas Monjalon
Seen with clang: error: explicitly assigning value of variable of type 'int' to itself Signed-off-by: Thomas Monjalon --- lib/cli/cli_scrn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/cli/cli_scrn.c b/lib/cli/cli_scrn.c index be7df4a..b195e92 100644 --- a/lib/cli/cli

[dpdk-dev] [pktgen PATCH 0/6] fix compilation

2017-07-25 Thread Thomas Monjalon
Some errors were seen when trying to compile with gcc 7 or clang, for 32-bit, and with DPDK 17.08-rc2. Thomas Monjalon (6): remove unused functions fix screen type saving fix compilation of printf based functions fix compilation with gcc 7 fix 32-bit build fix build with dpdk-17.08-rc2

[dpdk-dev] [PATCH] doc: add description of the traffic management api

2017-07-25 Thread Jasvinder Singh
This patch adds description of the traffic management api to dpdk programmers guide. Signed-off-by: Jasvinder Singh Signed-off-by: Cristian Dumitrescu Acked-by: Cristian Dumitrescu --- doc/guides/prog_guide/index.rst | 1 + doc/guides/prog_guide/traffic_management_api.rst |

Re: [dpdk-dev] doubts on dpdk qos-sched sample application

2017-07-25 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Mahesh Ishwar > Mathad > Sent: Tuesday, July 25, 2017 7:33 AM > To: users ; dev@dpdk.org > Subject: [dpdk-dev] doubts on dpdk qos-sched sample application > > Hi Team, > > $ ./build/qos_sched -c 0xe -n 4 -- --pf

[dpdk-dev] [PATCH] crypto/openssl: fix typo

2017-07-25 Thread Pablo de Lara
Fixes: b79e4c00af0e ("cryptodev: use AES-GCM/CCM as AEAD algorithms") Signed-off-by: Pablo de Lara --- drivers/crypto/openssl/rte_openssl_pmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/openssl/rte_openssl_pmd.c b/drivers/crypto/openssl/rte_openssl_pmd.c

[dpdk-dev] [PATCH] net/mlx5: fix counting consumed Tx descriptors

2017-07-25 Thread Yongseok Koh
When advancing Tx ring index (txq->wqe_ci) in txq_scatter_v(), the title descriptor of multi-packet send isn't taken into account if it doesn't cross 64B boundary. Fixes: 6cb559d67b83 ("net/mlx5: add vectorized Rx/Tx burst for x86") Signed-off-by: Yongseok Koh Acked-by: Nelio Laranjeiro --- dr

Re: [dpdk-dev] 17.11 Intel Roadmap

2017-07-25 Thread Alejandro Lucero
Hi Tim, I'd like to add this: Netronome NFP PMD with PF support, including PF mutliport suport. Currently NFP PMF just supports VFs. Thanks On Tue, Jul 25, 2017 at 11:18 AM, O'Driscoll, Tim wrote: > Below are the features that we're planning to submit for the 17.11 > release. We'll submit a p

[dpdk-dev] [PATCH] doc: support new ZUC library version

2017-07-25 Thread Pablo de Lara
A new version of the LibSSO ZUC library has been released. This version includes shared library support and bug fixes. This commit extends the instructions to install and initialize the PMD with the new library, enabling also the PMD to be compiled as a shared library. Signed-off-by: Pablo de Lar

[dpdk-dev] [PATCH] mk: use extra cflags when linking libs with a compiler

2017-07-25 Thread Gage Eads
When using the compiler to link libraries, include EXTRA_CFLAGS. This is needed when cross-compiling to pass --sysroot, for example. GCC cross-compilers built with Yocto don't use the --with-sysroot option, making it necessary to pass the --sysroot command-line option. This is the same solution as

[dpdk-dev] [PATCH] crypto/qat: fix SHA384-HMAC block size

2017-07-25 Thread Pablo de Lara
Block size of SHA384-HMAC algorithm is 128 bytes, and not 64 bytes. Fixes: d905ee32d0dc ("crypto/qat: add aes-sha384-hmac capability") Cc: sta...@dpdk.org Signed-off-by: Pablo de Lara --- drivers/crypto/qat/qat_crypto_capabilities.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

Re: [dpdk-dev] [pktgen PATCH] remove Intel from DPDK name

2017-07-25 Thread Wiles, Keith
> On Jul 24, 2017, at 3:55 PM, Thomas Monjalon wrote: > > DPDK is not called Intel DPDK since 2014. > It can be renamed in DPDK pktgen as well. > > Signed-off-by: Thomas Monjalon Acked by: Keith Wiles > --- > README.md | 18 +- > changelog.txt

[dpdk-dev] [PATCH v2] cryptodev: fix session init return value

2017-07-25 Thread Pablo de Lara
When calling rte_cryptodev_sym_session_init(), if there was an error, it returned -1, instead of returning the specific error code, which can be valuable for the application for error handling. Fixes: b3bbd9e5f265 ("cryptodev: support device independent sessions") Signed-off-by: Pablo de Lara Ac

[dpdk-dev] [PATCH v2] net/i40e: new API to add VF MAC address from PF

2017-07-25 Thread Wenzhuo Lu
Currently, on i40e the parameter 'pool' of API rte_eth_dev_mac_addr_add means the VMDq pool, not VF. So, it's wrong to use it to set the VF MAC address. As this API is also used by the VMDq example, ideally we need a parameter to tell the pool is VMDq or VF. But it's hard to change it because of th

Re: [dpdk-dev] [PATCH] net/e1000: fix out of bounds access

2017-07-25 Thread Lu, Wenzhuo
Hi Michal, > -Original Message- > From: Jastrzebski, MichalX K > Sent: Tuesday, July 25, 2017 9:36 AM > To: Lu, Wenzhuo ; Kozak, KubaX > > Cc: dev@dpdk.org; Jain, Deepak K ; > sta...@dpdk.org > Subject: RE: [PATCH] net/e1000: fix out of bounds access > > > -Original Message- > >

Re: [dpdk-dev] [PATCH v4] net/mlx5: fix inconsistent link status query

2017-07-25 Thread Nélio Laranjeiro
On Tue, Jul 25, 2017 at 07:08:07PM +0530, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > ETHTOOL_GLINKSETTINGS ioctl call in mlx5 pmd returns inconsistent > link status due to which any application relying on it would not > function correctly. > > Fixes: 188408719888 ("net/mlx5: fix

Re: [dpdk-dev] 17.11 Intel Roadmap

2017-07-25 Thread O'Driscoll, Tim
Apologies, but I missed a feature: QoS Metering and Policing: A new API will be created to support QoS Metering and Policing. This will include a software implementation using the existing QoS metering library (librte_meter). Further details are available in the RFC at: http://dpdk.org/ml/archi

[dpdk-dev] [PATCH v4] net/mlx5: fix inconsistent link status query

2017-07-25 Thread Gowrishankar
From: Gowrishankar Muthukrishnan ETHTOOL_GLINKSETTINGS ioctl call in mlx5 pmd returns inconsistent link status due to which any application relying on it would not function correctly. Fixes: 188408719888 ("net/mlx5: fix support for newer link speeds") Cc: sta...@dpdk.org Signed-off-by: Gowrisha

Re: [dpdk-dev] [PATCH] net/e1000: fix out of bounds access

2017-07-25 Thread Jastrzebski, MichalX K
> -Original Message- > From: Lu, Wenzhuo > Sent: Tuesday, July 25, 2017 3:25 PM > To: Kozak, KubaX > Cc: dev@dpdk.org; Jain, Deepak K ; Jastrzebski, > MichalX K ; sta...@dpdk.org > Subject: RE: [PATCH] net/e1000: fix out of bounds access > > Hi Kuba, > > > -Original Message- > >

Re: [dpdk-dev] [PATCH] net/e1000: fix out of bounds access

2017-07-25 Thread Lu, Wenzhuo
Hi Kuba, > -Original Message- > From: Kozak, KubaX > Sent: Tuesday, July 25, 2017 8:19 AM > To: Lu, Wenzhuo > Cc: dev@dpdk.org; Jain, Deepak K ; Jastrzebski, > MichalX K ; Kozak, KubaX > ; sta...@dpdk.org > Subject: [PATCH] net/e1000: fix out of bounds access > > Fix wrong structure type

[dpdk-dev] [PATCH 3/3] crypto/qat: fix HMAC supported key sizes

2017-07-25 Thread Pablo de Lara
For HMAC algorithms (MD5-HMAC, SHAx-HMAC), the supported key sizes are not a fixed value, but a range between 1 and the block size. Fixes: 26c2e4ad5ad4 ("cryptodev: add capabilities discovery") Cc: sta...@dpdk.org Signed-off-by: Pablo de Lara --- drivers/crypto/qat/qat_crypto_capabilities.h | 2

[dpdk-dev] [PATCH 2/3] crypto/openssl: fix HMAC supported key sizes

2017-07-25 Thread Pablo de Lara
For HMAC algorithms (MD5-HMAC, SHAx-HMAC), the supported key sizes are not a fixed value, but a range between 1 and the block size. Fixes: d61f70b4c918 ("crypto/libcrypto: add driver for OpenSSL library") Cc: sta...@dpdk.org Signed-off-by: Pablo de Lara --- drivers/crypto/openssl/rte_openssl_pm

[dpdk-dev] [PATCH 1/3] crypto/aesni_mb: fix HMAC supported key sizes

2017-07-25 Thread Pablo de Lara
For HMAC algorithms (MD5-HMAC, SHAx-HMAC), the supported key sizes are not a fixed value, but a range between 1 and the block size. Fixes: 26c2e4ad5ad4 ("cryptodev: add capabilities discovery") Cc: sta...@dpdk.org Signed-off-by: Pablo de Lara --- drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c |

[dpdk-dev] [PATCH 0/3] HMAC supported key size fixes

2017-07-25 Thread Pablo de Lara
For HMAC algorithms (SHAx/MD5), the capabilities were not correct, as they were just a fixed value (block size). Instead, the authentication key size for these algorithms can go between 1 and the block size. Note that these patches are fixing AESNI-MB, OpenSSL and QAT PMDs, but DPAA2 and ARMv8 cry

[dpdk-dev] [PATCH] net/e1000: fix out of bounds access

2017-07-25 Thread Kuba Kozak
Fix wrong structure type used as argument in memset() call. Coverity issue: 147223 Coverity issue: 147227 Fixes: a8600af43738 ("net/igb: parse flow API ethertype filter") Fixes: 22bb13410cb2 ("net/igb: create consistent filter") Cc: sta...@dpdk.org Signed-off-by: Kuba Kozak --- drivers/net/e100

Re: [dpdk-dev] [PATCH] app/testpmd: fix argument cannot be negative

2017-07-25 Thread Van Haaren, Harry
Hi Daniel, > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Daniel Mrzyglod > Sent: Tuesday, July 25, 2017 12:00 PM > To: Xing, Beilei > Cc: dev@dpdk.org; Mrzyglod, DanielX T > Subject: [dpdk-dev] [PATCH] app/testpmd: fix argument cannot be negative Sorry for the nit-pick review, but the

[dpdk-dev] [PATCH v2 3/3] doc: update mlx guides

2017-07-25 Thread Shahaf Shuler
Update the guides with: * New supported features. * Supported OFED and FW versions. * Quick start guide. * Performance tunning guide. Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro --- On v2: - Fixed grammer and spelling. - Changed limitations and Known issues location. --

[dpdk-dev] [PATCH v2 2/3] doc: update release notes for mlx driver

2017-07-25 Thread Shahaf Shuler
Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro --- doc/guides/rel_notes/release_17_08.rst | 126 + 1 file changed, 126 insertions(+) diff --git a/doc/guides/rel_notes/release_17_08.rst b/doc/guides/rel_notes/release_17_08.rst index ef51ebbe8..cdb156962

[dpdk-dev] [PATCH v2 1/3] doc: update mlx5 supported features

2017-07-25 Thread Shahaf Shuler
Supported features which were not included: * ARMv8 * Extended stats Not supported features which were wrongly included: * Inner L3 checksum * Inner L4 checksum Signed-off-by: Shahaf Shuler Acked-by: Nelio Laranjeiro --- doc/guides/nics/features/mlx5.ini | 4 ++-- 1 file changed, 2 ins

Re: [dpdk-dev] [PATCH] cryptodev: fix session init return value

2017-07-25 Thread Trahe, Fiona
Hi Pablo, > -Original Message- > From: De Lara Guarch, Pablo > Sent: Monday, July 24, 2017 9:54 AM > To: zbigniew.bo...@caviumnetworks.com; jerin.ja...@caviumnetworks.com; > akhil.go...@nxp.com; > hemant.agra...@nxp.com; Trahe, Fiona ; Jain, Deepak K > ; Griffin, John > Cc: dev@dpdk.org;

Re: [dpdk-dev] [PATCH 3/3] doc: update mlx guides

2017-07-25 Thread Shahaf Shuler
Adrien, Thanks for the review. sorry for the caps. One issue I not agree with below Tuesday, July 25, 2017 10:17 AM, Adrien Mazarguil: > Hi Shahaf, > [...] > > Currently supported by DPDK: > > > > -- Mellanox OFED **4.0-2.0.0.0**. > > -- Firmware version **2.40.7000**. > > +- Mellanox OFED **4.1

[dpdk-dev] [RFC PATCH 0/1] IPSec Inline and look aside crypto offload

2017-07-25 Thread Akhil Goyal
Below is a counter proposal for the RFC sent by Boris. If we find some consensus, we can have implementation for this proposal in a few weeks. The proposal is largely inline with the thoughts from Declan with a few exceptions. Here we are proposing a security framework which can be used both by

[dpdk-dev] [RFC PATCH 1/1] rte_security: proposal

2017-07-25 Thread Akhil Goyal
Detailed description is added in the coverletter Signed-off-by: Akhil Goyal --- lib/librte_security/rte_security.h | 405 + 1 file changed, 405 insertions(+) create mode 100644 lib/librte_security/rte_security.h diff --git a/lib/librte_security/rte_security.

[dpdk-dev] [PATCH] net/mlx4: fix drop action setting before start

2017-07-25 Thread Matan Azrad
The corrupted code causes segmentation fault when user creates flow with drop action before device starting. For example, failsafe PMD recreates all the flows before calling dev_start in plug-in sequence and mlx4 allocated its flow drop queue in dev_start. Hence, when failsafe created flow with dr

[dpdk-dev] [PATCH] app/testpmd: fix argument cannot be negative

2017-07-25 Thread Daniel Mrzyglod
Coverity issue: 143454 Fixes: a92a5a2cbbff ("app/testpmd: add command for loading DDP") Signed-off-by: Daniel Mrzyglod --- app/test-pmd/config.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index ee6644d10..b77fb96e1 100

Re: [dpdk-dev] [PATCH] doc: add how to enable empty cycles profiling

2017-07-25 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Mcnamara, John > Sent: Tuesday, July 25, 2017 11:05 AM > To: Kurakin, Ilia ; dev@dpdk.org > Cc: jerin.ja...@caviumnetworks.com; Ananyev, Konstantin > ; Wiles, Keith ; > Galanov, Dmitry ; Kurakin, Ilia > > Subject

[dpdk-dev] [PATCH v1] app/testpmd: fix Tx first with LSC interrupt

2017-07-25 Thread David Hunt
The lsc_interrupt flag is enabled by default, and when the --tx-first is used as a command line parameter, the 32 packets can be sent out before the link state is up, resulting in the loss of the packets, and no further forwarding will take place. E.g. ./build/app/testpmd -c f0 -- --tx-first --sta

[dpdk-dev] 17.11 Intel Roadmap

2017-07-25 Thread O'Driscoll, Tim
Below are the features that we're planning to submit for the 17.11 release. We'll submit a patch to update the roadmap page with this info. It would be good if others are also willing to share their plans so that we can build up a complete picture of what's planned for 17.11 and make sure there'

Re: [dpdk-dev] [PATCH] doc: add how to enable empty cycles profiling

2017-07-25 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of > ilia.kura...@intel.com > Sent: Monday, July 24, 2017 6:24 PM > To: dev@dpdk.org > Cc: jerin.ja...@caviumnetworks.com; Ananyev, Konstantin > ; Wiles, Keith ; > Galanov, Dmitry ; Kurakin, Ilia > > Subject: [dpdk-

Re: [dpdk-dev] [PATCH] test: fix the parameter issue of test case

2017-07-25 Thread Declan Doherty
On 13/07/2017 9:00 AM, Herbert Guan wrote: When test case "test_balance_l23_tx_burst_ipv4_toggle_ip_addr" is calling balance_l23_tx_burst(), the ip_addr instead of mac_addr should be toggled according to the test name. Signed-off-by: Herbert Guan --- test/test/test_link_bonding.c | 2 +- 1 fil

Re: [dpdk-dev] [PATCH] net/mlx5: fix inconsistent link status query

2017-07-25 Thread Nélio Laranjeiro
On Tue, Jul 25, 2017 at 01:37:39PM +0530, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > ETHTOOL_GLINKSETTINGS ioctl call in mlx5 pmd returns inconsistent > link status due to which any application relying on it would not > function correctly. > > Fixes: 188408719888 ("net/mlx5: fix

Re: [dpdk-dev] [PATCH] cryptodev: fix session init return value

2017-07-25 Thread Akhil Goyal
On 7/24/2017 2:24 PM, Pablo de Lara wrote: When calling rte_cryptodev_sym_session_init(), if there was an error, it returned -1, regardless the error. Instead, it should return the specific error code, which can be valuable for the application for error handling. Fixes: b3bbd9e5f265 ("cryptodev:

[dpdk-dev] [PATCH] net/mlx5: fix inconsistent link status query

2017-07-25 Thread Gowrishankar
From: Gowrishankar Muthukrishnan ETHTOOL_GLINKSETTINGS ioctl call in mlx5 pmd returns inconsistent link status due to which any application relying on it would not function correctly. Fixes: 188408719888 ("net/mlx5: fix support for newer link speeds") Cc: sta...@dpdk.org Signed-off-by: Gowrisha

Re: [dpdk-dev] [PATCH] net/mlx5: poll completion queue once per a call

2017-07-25 Thread Yongseok Koh
On Sun, Jul 23, 2017 at 12:49:36PM +0300, Sagi Grimberg wrote: > > > > mlx5_tx_complete() polls completion queue multiple times until it > > > > encounters an invalid entry. As Tx completions are suppressed by > > > > MLX5_TX_COMP_THRESH, it is waste of cycles to expect multiple > > > > completion

Re: [dpdk-dev] [PATCH 3/3] doc: update mlx guides

2017-07-25 Thread Adrien Mazarguil
Hi Shahaf, On Mon, Jul 24, 2017 at 03:36:37PM +0300, Shahaf Shuler wrote: > Update the guides with: >* New supported features. >* Supported OFED and FW versions. >* Quick start guide. >* Performance tunning guide. > > Signed-off-by: Shahaf Shuler > Acked-by: Nelio Laranjeiro Th