[dpdk-dev] [PATCH] examples: add a new example for link reset

2016-06-08 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Monday, June 06, 2016 6:48 AM > To: dev at dpdk.org > Cc: Lu, Wenzhuo > Subject: [dpdk-dev] [PATCH] examples: add a new example for link reset > > Add a new example to show when the PF is

[dpdk-dev] [PATCH v2 4/5] testpmd: handle all rxqs in rss setup

2016-06-08 Thread Wang, Zhihong
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Tuesday, June 7, 2016 6:30 PM > To: Wang, Zhihong ; dev at dpdk.org > Cc: Ananyev, Konstantin ; Richardson, Bruce > ; thomas.monjalon at 6wind.com > Subject: RE: [PATCH v2 4/5] testpmd: handle all rxqs in rss setup > > > > >

[dpdk-dev] [PATCH v2 1/5] testpmd: add retry option

2016-06-08 Thread Wang, Zhihong
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Tuesday, June 7, 2016 5:28 PM > To: Wang, Zhihong ; dev at dpdk.org > Cc: Ananyev, Konstantin ; Richardson, Bruce > ; thomas.monjalon at 6wind.com > Subject: RE: [PATCH v2 1/5] testpmd: add retry option > > > > > -Original

[dpdk-dev] [PATCH v2 3/5] testpmd: show throughput in port stats

2016-06-08 Thread Wang, Zhihong
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Tuesday, June 7, 2016 6:03 PM > To: Wang, Zhihong ; dev at dpdk.org > Cc: Ananyev, Konstantin ; Richardson, Bruce > ; thomas.monjalon at 6wind.com > Subject: RE: [PATCH v2 3/5] testpmd: show throughput in port stats > > > > >

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-08 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Tuesday, June 7, 2016 5:59 PM > To: Tao, Zhe; dev at dpdk.org > Cc: Lu, Wenzhuo; Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, > Jingjing; > Zhang, Helin > Subject: RE: [PATCH v4 2/8] lib/librte_ether: defind

[dpdk-dev] [PATCH] examples: add a new example for link reset

2016-06-08 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Wednesday, June 8, 2016 8:25 AM > To: Lu, Wenzhuo; dev at dpdk.org > Cc: Lu, Wenzhuo > Subject: RE: [dpdk-dev] [PATCH] examples: add a new example for link reset > > > > > -Original Message- > > From: dev

[dpdk-dev] [PATCH v4 4/8] ixgbe: implement device reset on VF

2016-06-08 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Tuesday, June 7, 2016 6:03 PM > To: Tao, Zhe; dev at dpdk.org > Cc: Lu, Wenzhuo; Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, > Jingjing; > Zhang, Helin > Subject: RE: [PATCH v4 4/8] ixgbe: implement device

[dpdk-dev] [PATCH 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-08 Thread Lu, Wenzhuo
Hi Stephen, > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Wednesday, June 8, 2016 10:16 AM > To: Lu, Wenzhuo > Cc: dev at dpdk.org; Tao, Zhe > Subject: Re: [dpdk-dev] [PATCH 2/8] lib/librte_ether: defind RX/TX lock mode > > On Mon, 6 Jun

[dpdk-dev] [PATCH v1 1/1] examples/l2fwd-crypto: improve random key generator

2016-06-08 Thread Azarewicz, PiotrX T
> 2016-05-25 15:34, Piotr Azarewicz: > > This patch improve generate_random_key() function by replacing rand() > > function with reading from /dev/urandom. > > > > CID 120136 : Calling risky function (DC.WEAK_CRYPTO) > > dont_call: rand should not be used for security related applications, > > as

[dpdk-dev] about rx checksum flags

2016-06-08 Thread Chandran, Sugesh
Regards _Sugesh > -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Friday, June 3, 2016 1:43 PM > To: Chandran, Sugesh ; Ananyev, Konstantin > ; Stephen Hemminger > > Cc: Yuanhan Liu ; dev at dpdk.org; Richardson, > Bruce ; Adrien Mazarguil > ; Tan,

[dpdk-dev] [PATCH] mbuf: remove inconsistent assert statements

2016-06-08 Thread Adrien Mazarguil
An assertion failure occurs in __rte_mbuf_raw_free() (called by a few PMDs) when compiling DPDK with CONFIG_RTE_LOG_LEVEL=RTE_LOG_DEBUG and starting applications with a log level high enough to trigger it. While rte_mbuf_raw_alloc() sets refcount to 1, __rte_mbuf_raw_free() expects it to be 0.

[dpdk-dev] [PATCH] examples: add a new example for link reset

2016-06-08 Thread Ananyev, Konstantin
Hi Wenzhuo, > Hi Konstantin, > > > -Original Message- > > From: Ananyev, Konstantin > > Sent: Wednesday, June 8, 2016 8:25 AM > > To: Lu, Wenzhuo; dev at dpdk.org > > Cc: Lu, Wenzhuo > > Subject: RE: [dpdk-dev] [PATCH] examples: add a new example for link reset > > > > > > > > >

[dpdk-dev] [PATCH v4 4/8] ixgbe: implement device reset on VF

2016-06-08 Thread Ananyev, Konstantin
> -Original Message- > From: Lu, Wenzhuo > Sent: Wednesday, June 08, 2016 8:24 AM > To: Ananyev, Konstantin; Tao, Zhe; dev at dpdk.org > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > Helin > Subject: RE: [PATCH v4 4/8] ixgbe: implement device reset on VF >

[dpdk-dev] [PATCH] doc: remove reference to MATCH

2016-06-08 Thread Chen Jing D(Mark)
From: "Chen Jing D(Mark)" Intel stopped supporting MATCH, remove reference of MATCH in the document. Signed-off-by: Chen Jing D(Mark) --- doc/guides/nics/fm10k.rst |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/doc/guides/nics/fm10k.rst

[dpdk-dev] [PATCH v3 1/2] ethdev: add callback to get register size in bytes

2016-06-08 Thread Thomas Monjalon
Hi Zyta, 2016-06-01 09:56, zr at semihalf.com: > rte_eth_dev_get_reg_length and rte_eth_dev_get_reg callbacks > do not provide register size to the app in any way. It is > needed to allocate proper number of bytes before retrieving > registers content with rte_eth_dev_get_reg. Yes, register size

[dpdk-dev] [PATCH] examples: add a new example for link reset

2016-06-08 Thread Thomas Monjalon
2016-06-08 08:37, Ananyev, Konstantin: > > From: Ananyev, Konstantin > > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > > > > Add a new example to show when the PF is down and up, VF port can be > > > > reset and recover. > > > > > > Do we really need a totally new example

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-08 Thread Ananyev, Konstantin
> > Hi Konstantin, > > > > -Original Message- > > From: Ananyev, Konstantin > > Sent: Tuesday, June 7, 2016 5:59 PM > > To: Tao, Zhe; dev at dpdk.org > > Cc: Lu, Wenzhuo; Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, > > Jingjing; > > Zhang, Helin > > Subject: RE: [PATCH v4

[dpdk-dev] [PATCH v4 4/8] ixgbe: implement device reset on VF

2016-06-08 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin > Sent: Wednesday, June 08, 2016 9:42 AM > To: Lu, Wenzhuo; Tao, Zhe; dev at dpdk.org > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > Helin > Subject: Re:

[dpdk-dev] [PATCH v3 01/10] rte: change xstats to use integer ids

2016-06-08 Thread Thomas Monjalon
2016-05-30 11:48, Remy Horton: > struct rte_eth_xstats { > + /* FIXME: Remove name[] once remaining drivers converted */ > char name[RTE_ETH_XSTATS_NAME_SIZE]; What is the plan? This field must be deprecated with an attribute. We cannot have 2 different APIs depending of the driver.

[dpdk-dev] [PATCH 0/7] Miscellaneous fixes for mlx4 and mlx5

2016-06-08 Thread Nelio Laranjeiro
Various minor fixes for mlx4 (ConnectX-3) and mlx5 (ConnectX-4). Adrien Mazarguil (4): mlx: ensure MTU update is effective mlx: retrieve mbuf size through proper API mlx5: fix RX VLAN stripping capability check mlx5: cosmetic changes (coding style) Nelio Laranjeiro (3): mlx: remove

[dpdk-dev] [PATCH 1/7] mlx: remove unused memory region property

2016-06-08 Thread Nelio Laranjeiro
Memory regions are always local with raw Ethernet queues, drop the remote property as it adds extra processing on the hardware side. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx4/mlx4.c | 4 ++-- drivers/net/mlx5/mlx5_rxtx.c | 2 +-

[dpdk-dev] [PATCH 2/7] mlx: ensure MTU update is effective

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil There is no guarantee that the new MTU is effective after writing its value to sysfs. Retrieve it to be sure. Signed-off-by: Adrien Mazarguil --- drivers/net/mlx4/mlx4.c| 10 +- drivers/net/mlx5/mlx5_ethdev.c | 10 +-

[dpdk-dev] [PATCH 3/7] mlx: retrieve mbuf size through proper API

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil No need to allocate a mbuf for that. Signed-off-by: Adrien Mazarguil --- drivers/net/mlx4/mlx4.c| 29 - drivers/net/mlx5/mlx5_ethdev.c | 5 - drivers/net/mlx5/mlx5_rxq.c| 20 ++--

[dpdk-dev] [PATCH 4/7] mlx5: fix RX VLAN stripping capability check

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil A hardware capability check is missing before enabling RX VLAN stripping during queue setup. Also, while dev_conf.rxmode.hw_vlan_strip is currently a single bit that can be stored in priv->hw_vlan_strip directly, it should be interpreted as a

[dpdk-dev] [PATCH 5/7] mlx5: cosmetic changes (coding style)

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Add consistency to mlx5_rxtx.h. Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_rxtx.h | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_rxtx.h

[dpdk-dev] [PATCH 6/7] mlx5: enhance SR-IOV detection

2016-06-08 Thread Nelio Laranjeiro
SR-IOV mode is currently set when dealing with VF devices. PF devices must be taken into account as well if they have active VFs. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5.c| 14 -- drivers/net/mlx5/mlx5.h| 3 ++-

[dpdk-dev] [PATCH 7/7] mlx5: update documentation part related to features and limitations

2016-06-08 Thread Nelio Laranjeiro
Signed-off-by: Nelio Laranjeiro --- doc/guides/nics/mlx5.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index b6f91e6..d9196d1 100644 --- a/doc/guides/nics/mlx5.rst +++ b/doc/guides/nics/mlx5.rst @@ -86,11 +86,11 @@

[dpdk-dev] [PATCH 00/24] Refactor mlx5 to improve performance

2016-06-08 Thread Nelio Laranjeiro
Enhance mlx5 with a data path that bypasses Verbs. The first half of this patchset removes support for functionality completely rewritten in the second half (scatter/gather, inline send), while the data path is refactored without Verbs. The PMD remains usable during the transition. This

[dpdk-dev] [PATCH 01/24] mlx5: split memory registration function for better performance

2016-06-08 Thread Nelio Laranjeiro
Except for the first time when memory registration occurs, the lkey is always cached. Since memory registration is slow and performs system calls, performance can be improved by moving that code to its own function outside of the data path so only the lookup code is left in the original inlined

[dpdk-dev] [PATCH 02/24] mlx5: remove TX gather support

2016-06-08 Thread Nelio Laranjeiro
This is done in preparation of bypassing Verbs entirely for the data path as a performance improvement. TX gather cannot be maintained during the transition and will be reimplemented later. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_ethdev.c | 2

[dpdk-dev] [PATCH 03/24] mlx5: remove RX scatter support

2016-06-08 Thread Nelio Laranjeiro
This is done in preparation of bypassing Verbs entirely for the data path as a performance improvement. RX scatter cannot be maintained during the transition and will be reimplemented later. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_ethdev.c |

[dpdk-dev] [PATCH 04/24] mlx5: remove configuration variable for maximum number of segments

2016-06-08 Thread Nelio Laranjeiro
There is no scatter/gather support anymore, CONFIG_RTE_LIBRTE_MLX5_SGE_WR_N has no purpose and can be removed. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- config/common_base | 1 - doc/guides/nics/mlx5.rst | 7 --- drivers/net/mlx5/Makefile| 4

[dpdk-dev] [PATCH 06/24] mlx5: split TX queue structure

2016-06-08 Thread Nelio Laranjeiro
To keep the data path as efficient as possible, move fields only useful to the control path into new structure txq_ctrl. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5.c| 21 +++-- drivers/net/mlx5/mlx5_ethdev.c | 27 +++---

[dpdk-dev] [PATCH 05/24] mlx5: remove inline TX support

2016-06-08 Thread Nelio Laranjeiro
Inline TX will be fully managed by the PMD after Verbs is bypassed in the data path. Remove the current code until then. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- config/common_base | 1 - doc/guides/nics/mlx5.rst | 10 -- drivers/net/mlx5/Makefile

[dpdk-dev] [PATCH 07/24] mlx5: split RX queue structure

2016-06-08 Thread Nelio Laranjeiro
To keep the data path as efficient as possible, move fields only useful to the control path into new structure rxq_ctrl. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5.c | 6 +- drivers/net/mlx5/mlx5_fdir.c | 8 +- drivers/net/mlx5/mlx5_rxq.c

[dpdk-dev] [PATCH 08/24] mlx5: update prerequisites for upcoming enhancements

2016-06-08 Thread Nelio Laranjeiro
The latest version of Mellanox OFED exposes hardware definitions necessary to implement data path operation bypassing Verbs. Update the minimum version requirement to MLNX_OFED >= 3.3 and clean up compatibility checks for previous releases. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien

[dpdk-dev] [PATCH 10/24] mlx5: add support for configuration through kvargs

2016-06-08 Thread Nelio Laranjeiro
The intent is to replace the remaining compile-time options and environment variables with a common mean of runtime configuration. This commit only adds the kvargs handling code, subsequent commits will update the rest. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil ---

[dpdk-dev] [PATCH 09/24] mlx5: add definitions for data path without Verbs

2016-06-08 Thread Nelio Laranjeiro
These structures and macros extend those exposed by libmlx5 (in mlx5_hw.h) to let the PMD manage work queue and completion queue elements directly. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_prm.h | 155

[dpdk-dev] [PATCH 11/24] mlx5: add TX/RX burst function selection wrapper

2016-06-08 Thread Nelio Laranjeiro
These wrappers are meant to prevent code duplication later. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5.h| 2 ++ drivers/net/mlx5/mlx5_ethdev.c | 34 -- drivers/net/mlx5/mlx5_txq.c| 2 +- 3 files

[dpdk-dev] [PATCH 12/24] mlx5: refactor RX data path

2016-06-08 Thread Nelio Laranjeiro
Bypass Verbs to improve RX performance. Signed-off-by: Nelio Laranjeiro Signed-off-by: Yaacov Hazan Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_ethdev.c | 4 +- drivers/net/mlx5/mlx5_fdir.c | 2 +- drivers/net/mlx5/mlx5_rxq.c| 291

[dpdk-dev] [PATCH 13/24] mlx5: refactor TX data path

2016-06-08 Thread Nelio Laranjeiro
Bypass Verbs to improve TX performance. Signed-off-by: Nelio Laranjeiro Signed-off-by: Yaacov Hazan Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/Makefile | 5 - drivers/net/mlx5/mlx5_ethdev.c | 10 +- drivers/net/mlx5/mlx5_mr.c | 4 +- drivers/net/mlx5/mlx5_rxtx.c |

[dpdk-dev] [PATCH 14/24] mlx5: handle RX CQE compression

2016-06-08 Thread Nelio Laranjeiro
Mini (compressed) CQEs are returned by the NIC when PCI back pressure is detected, in which case the first CQE64 contains common packet information followed by a number of CQE8 providing the rest, followed by a matching number of empty CQE64 entries to be used by software for decompression.

[dpdk-dev] [PATCH 15/24] mlx5: replace countdown with threshold for TX completions

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Replacing the variable countdown (which depends on the number of descriptors) with a fixed relative threshold known at compile time improves performance by reducing the TX queue structure footprint and the amount of code to manage completions

[dpdk-dev] [PATCH 16/24] mlx5: add support for inline send

2016-06-08 Thread Nelio Laranjeiro
From: Yaacov Hazan Implement send inline feature which copies packet data directly into WQEs for improved latency. The maximum packet size and the minimum number of TX queues to qualify for inline send are user-configurable. This feature is effective when HW causes a

[dpdk-dev] [PATCH 17/24] mlx5: add support for multi-packet send

2016-06-08 Thread Nelio Laranjeiro
This feature enables the TX burst function to emit up to 5 packets using only two WQEs on devices that support it. Saves PCI bandwidth and improves performance. Signed-off-by: Nelio Laranjeiro Signed-off-by: Adrien Mazarguil Signed-off-by: Olga Shern --- doc/guides/nics/mlx5.rst | 10

[dpdk-dev] [PATCH 18/24] mlx5: add debugging information about TX queues capabilities

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_txq.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c index 15c8f73..d013230 100644 --- a/drivers/net/mlx5/mlx5_txq.c

[dpdk-dev] [PATCH 19/24] mlx5: check remaining space while processing TX burst

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil The space necessary to store segmented packets cannot be known in advance and must be verified for each of them. Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_rxtx.c | 136 ++- 1 file

[dpdk-dev] [PATCH 20/24] mlx5: resurrect TX gather support

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Compared to its previous incarnation, the software limit on the number of mbuf segments is no more (previously MLX5_PMD_SGE_WR_N, set to 4 by default) hence no need for linearization code and related buffers that permanently consumed a non

[dpdk-dev] [PATCH 21/24] mlx5: work around spurious compilation errors

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Since commit "mlx5: resurrect TX gather support", older GCC versions (such as 4.8.5) may complain about the following: mlx5_rxtx.c: In function `mlx5_tx_burst': mlx5_rxtx.c:705:25: error: `wqe' may be used uninitialized in this function

[dpdk-dev] [PATCH 22/24] mlx5: remove redundant RX queue initialization code

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil Toggling RX checksum offloads is already done at initialization time. This code does not belong in rxq_rehash(). Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_rxq.c | 10 -- 1 file changed, 10 deletions(-) diff --git

[dpdk-dev] [PATCH 23/24] mlx5: make RX queue reinitialization safer

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil The primary purpose of rxq_rehash() function is to stop and restart reception on a queue after re-posting buffers. This may fail if the array that temporarily stores existing buffers for reuse cannot be allocated. Update rxq_rehash() to work on

[dpdk-dev] [PATCH 24/24] mlx5: resurrect RX scatter support

2016-06-08 Thread Nelio Laranjeiro
From: Adrien Mazarguil This commit brings back RX scatter and related support by the MTU update function. The maximum number of segments per packet is not a fixed value anymore (previously MLX5_PMD_SGE_WR_N, set to 4 by default) as it caused performance issues when

[dpdk-dev] [PATCH v2 3/3] doc: add keepalive enhancement documentation

2016-06-08 Thread Thomas Monjalon
2016-05-18 10:30, Remy Horton: There is no explanation and it is totally normal, because this patch must be squashed with the code change. > Signed-off-by: Remy Horton > --- > doc/guides/rel_notes/release_16_07.rst | 5 + > 1 file changed, 5 insertions(+) > > diff --git

[dpdk-dev] [PATCH v2 1/3] eal: add new keepalive states & callback hooks

2016-06-08 Thread Thomas Monjalon
2016-05-18 10:30, Remy Horton: The explanations are missing. Probably you should split this patch. > Signed-off-by: Remy Horton [...] > +enum rte_keepalive_state { > + UNUSED = 0, > + ALIVE = 1, > + MISSING = 4, > + DEAD = 2, > + GONE = 3, > + DOZING = 5, > + SLEEP =

[dpdk-dev] [PATCH v2 0/7] examples/ip_pipeline: CLI rework and improvements

2016-06-08 Thread Azarewicz, PiotrX T
> > Piotr Azarewicz (7): > > examples/ip_pipeline: add helper functions for parsing string > > examples/ip_pipeline: modifies common pipeline CLI > > examples/ip_pipeline: modifies firewall pipeline CLI > > examples/ip_pipeline: modifies flow classifications pipeline CLI > >

[dpdk-dev] [PATCH v4 30/39] bnxt: add start/stop/link update operations

2016-06-08 Thread Bruce Richardson
On Mon, Jun 06, 2016 at 03:08:34PM -0700, Stephen Hurd wrote: > From: Ajit Khaparde > > This patch adds code to add the start, stop and link update dev_ops. > The BNXT driver will now minimally pass traffic with testpmd. > > v4: > - Fix issues pointed out by checkpatch. > - Shorten the string

[dpdk-dev] [PATCH v4 01/39] bnxt: new driver for Broadcom NetXtreme-C devices

2016-06-08 Thread Bruce Richardson
On Mon, Jun 06, 2016 at 03:08:05PM -0700, Stephen Hurd wrote: > From: Ajit Khaparde > > This patch adds the initial skeleton for bnxt driver along with the > nic guide to tie into the build system. > At this point, the driver simply fails init. > > v4: > Fix a warning that the document isn't

[dpdk-dev] [PATCH v3 0/7] examples/ip_pipeline: CLI rework and improvements

2016-06-08 Thread Piotr Azarewicz
Using the latest librte_cmdline improvements, the CLI implementation of the ip_pipeline application is streamlined and improved, which results in eliminating thousands of lines of code from the application, thus leading to code that is easier to maintain and extend. v3 changes: - fix the

[dpdk-dev] [PATCH v3 1/7] examples/ip_pipeline: add helper functions for parsing string

2016-06-08 Thread Piotr Azarewicz
Add a couple of additional functions that will allow to parse many types of input parameters, i.e.: bool, 16, 32, 64 bits, hex, etc. Signed-off-by: Piotr Azarewicz Acked-by: Cristian Dumitrescu --- examples/ip_pipeline/Makefile |1 + examples/ip_pipeline/config_parse.c | 257

[dpdk-dev] [PATCH v3 2/7] examples/ip_pipeline: modifies common pipeline CLI

2016-06-08 Thread Piotr Azarewicz
From: Tomasz Kulasek All link commands are merged into one command: cmd_link_parsed. Improve run command to allow run periodically. Adding static keyword to a lot of token declarations. Signed-off-by: Tomasz Kulasek Signed-off-by: Michal Kobylinski Acked-by:

[dpdk-dev] [PATCH v4 01/39] bnxt: new driver for Broadcom NetXtreme-C devices

2016-06-08 Thread Bruce Richardson
On Tue, Jun 07, 2016 at 08:25:44AM +0200, Thomas Monjalon wrote: > Hi Stephen, > Reminder from http://dpdk.org/dev#send: > > git send-email -39 -v4 --cover-letter --annotate > --in-reply-to > > Please do not forget --in-reply-to. Thanks Three other minor style updates as well that would

[dpdk-dev] [PATCH] mbuf: remove inconsistent assert statements

2016-06-08 Thread Ananyev, Konstantin
Hi Adrien, > > An assertion failure occurs in __rte_mbuf_raw_free() (called by a few PMDs) > when compiling DPDK with CONFIG_RTE_LOG_LEVEL=RTE_LOG_DEBUG and starting > applications with a log level high enough to trigger it. > > While rte_mbuf_raw_alloc() sets refcount to 1,

[dpdk-dev] [PATCH v3 3/7] examples/ip_pipeline: modifies firewall pipeline CLI

2016-06-08 Thread Piotr Azarewicz
From: Daniel Mrzyglod Each command are merged into one: cmd_firewall_parsed. ADD command format is changed: p firewall add priority ipv4 port and bulk command was modified: 1. firewall add bulk File line format: priority ipv4 port

[dpdk-dev] [PATCH v3 5/7] examples/ip_pipeline: modifies flow action pipeline CLI

2016-06-08 Thread Piotr Azarewicz
All commands merged into one: cmd_action_parsed. modified bulk command: action flow bulk File line format: flow meter 0 meter 1 meter 2 meter 3 policer 0policer 1 policer 2policer 3 port Signed-off-by: Marcin Kerlin Signed-off-by: Piotr Azarewicz Acked-by:

[dpdk-dev] [PATCH v3 6/7] examples/ip_pipeline: modifies routing pipeline CLI

2016-06-08 Thread Piotr Azarewicz
Several routing commands are merged into two commands: route and arp - these two commands are handled by cli library. Rest of the commands are handled internaly by the pipeline code. Signed-off-by: Piotr Azarewicz Acked-by: Cristian Dumitrescu --- examples/ip_pipeline/config/l2fwd.cfg

[dpdk-dev] [PATCH v2 0/3] testpmd: extend commands for better scatter-gather tests

2016-06-08 Thread Thomas Monjalon
> > Maciej Czekaj (3): > > app/testpmd: add "enable-scatter" parameter > > app/testpmd: extend port config with scatter parameter > > app/testpmd: support setting up txq_flags value in command line > > Series-acked-by: Pablo de Lara Applied, thanks

[dpdk-dev] [PATCH v3 7/7] examples/ip_pipeline: update edge router usecase

2016-06-08 Thread Piotr Azarewicz
Update edge router usecase config files to use bulk commands. Signed-off-by: Piotr Azarewicz Acked-by: Cristian Dumitrescu --- .../ip_pipeline/config/edge_router_downstream.cfg | 30 +++- .../ip_pipeline/config/edge_router_downstream.sh |7 ++--

[dpdk-dev] [PATCH] app/test: fix bond device name too long

2016-06-08 Thread Thomas Monjalon
2016-05-27 18:38, Thomas Monjalon: > 2016-05-27 17:20, Michal Jastrzebski: > > Bond device name was too long (grather than 32 signs) that > > cause mempool allocation to fail. > > Maybe that this kind of failure would be avoided if the test > was added to autotests (app/test/autotest_data.py). >

[dpdk-dev] [PATCH v4 01/39] bnxt: new driver for Broadcom NetXtreme-C devices

2016-06-08 Thread Bruce Richardson
On Wed, Jun 08, 2016 at 11:21:23AM +0100, Bruce Richardson wrote: > On Mon, Jun 06, 2016 at 03:08:05PM -0700, Stephen Hurd wrote: > > From: Ajit Khaparde > > > > This patch adds the initial skeleton for bnxt driver along with the > > nic guide to tie into the build system. > > At this point, the

[dpdk-dev] [PATCH v4 30/39] bnxt: add start/stop/link update operations

2016-06-08 Thread Bruce Richardson
On Wed, Jun 08, 2016 at 11:02:08AM +0100, Bruce Richardson wrote: > On Mon, Jun 06, 2016 at 03:08:34PM -0700, Stephen Hurd wrote: > > From: Ajit Khaparde > > > > This patch adds code to add the start, stop and link update dev_ops. > > The BNXT driver will now minimally pass traffic with testpmd.

[dpdk-dev] [PATCH v3 01/10] rte: change xstats to use integer ids

2016-06-08 Thread Remy Horton
'noon, On 08/06/2016 10:37, Thomas Monjalon wrote: > 2016-05-30 11:48, Remy Horton: >> struct rte_eth_xstats { >> +/* FIXME: Remove name[] once remaining drivers converted */ >> char name[RTE_ETH_XSTATS_NAME_SIZE]; > > What is the plan? This field must be deprecated with an attribute.

[dpdk-dev] [PATCH v5] eal: fix allocating all free hugepages

2016-06-08 Thread Sergio Gonzalez Monroy
On 31/05/2016 04:37, Jianfeng Tan wrote: > EAL memory init allocates all free hugepages of the whole system, > which seen from sysfs, even when applications do not ask so many. > When there is a limitation on how many hugepages an application can > use (such as cgroup.hugetlb), or hugetlbfs is

[dpdk-dev] [PATCH] doc: remove reference to MATCH

2016-06-08 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Chen Jing D(Mark) > Sent: Wednesday, June 8, 2016 9:44 AM > To: thomas.monjalon at 6wind.com > Cc: dev at dpdk.org; Chen, Jing D > Subject: [dpdk-dev] [PATCH] doc: remove reference to MATCH > > From: "Chen

[dpdk-dev] [PATCH] mem: fix overflowed return value

2016-06-08 Thread Mrzyglod, DanielX T
>-Original Message- >From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen Hemminger >Sent: Friday, April 22, 2016 6:25 PM >To: Kobylinski, MichalX >Cc: thomas.monjalon at 6wind.com; dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH] mem: fix overflowed return value > >On Fri,

[dpdk-dev] [PATCH] app/test: fix bond device name too long

2016-06-08 Thread Jastrzebski, MichalX K
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, June 08, 2016 12:40 PM > To: Jastrzebski, MichalX K ; Iremonger, > Bernard > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] app/test: fix bond device name too long > > 2016-05-27

[dpdk-dev] [PATCH v3 19/20] thunderx/nicvf: updated driver documentation and release notes

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Updated doc/guides/nics/overview.rst, doc/guides/nics/thunderx.rst > and release notes > > Changed "*" to "P" in overview.rst to capture the partially supported > feature as "*" creating alignment issues with Sphinx table > > Signed-off-by: Jerin Jacob

[dpdk-dev] [PATCH v3 12/20] thunderx/nicvf: add single and multi segment tx functions

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej Czekaj > Signed-off-by: Kamil Rytarowski > Signed-off-by: Zyta Szpak > Signed-off-by: Slawomir Rosek > Signed-off-by: Radoslaw Biernacki > --- > drivers/net/thunderx/Makefile | 2 + >

[dpdk-dev] [PATCH v8 1/3] mempool: support external mempool operations

2016-06-08 Thread Olivier Matz
Hi David, Please find some comments below. On 06/03/2016 04:58 PM, David Hunt wrote: > --- a/lib/librte_mempool/rte_mempool.h > +++ b/lib/librte_mempool/rte_mempool.h > +/** > + * Prototype for implementation specific data provisioning function. > + * > + * The function should provide the

[dpdk-dev] [PATCH v3 01/20] thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Adds hardware specific API for ThunderX nicvf inbuilt NIC device under > drivers/net/thunderx/nicvf/base directory. > > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej Czekaj > Signed-off-by: Kamil Rytarowski > Signed-off-by: Zyta Szpak >

[dpdk-dev] [PATCH v3 02/20] thunderx/nicvf: add pmd skeleton

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Introduce driver initialization and enable build infrastructure for > nicvf pmd driver. > > By default, It is enabled only for defconfig_arm64-thunderx-* > config as it is an inbuilt NIC device. > > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej

[dpdk-dev] [PATCH v3 01/10] rte: change xstats to use integer ids

2016-06-08 Thread Thomas Monjalon
2016-06-08 12:16, Remy Horton: > 'noon, > > On 08/06/2016 10:37, Thomas Monjalon wrote: > > 2016-05-30 11:48, Remy Horton: > >> struct rte_eth_xstats { > >> + /* FIXME: Remove name[] once remaining drivers converted */ > >>char name[RTE_ETH_XSTATS_NAME_SIZE]; > > > > What is the plan? This

[dpdk-dev] [PATCH v3 09/10] remove name field from struct rte_eth_xstats

2016-06-08 Thread Thomas Monjalon
2016-05-30 11:48, Remy Horton: > struct rte_eth_xstats { > - /* FIXME: Remove name[] once remaining drivers converted */ > - char name[RTE_ETH_XSTATS_NAME_SIZE]; > uint64_t id; > uint64_t value; > }; While changing the content of this struct, it can be the opportunity to fix

[dpdk-dev] [PATCH v3 08/20] thunderx/nicvf: add tx_queue_setup/release support

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej Czekaj > Signed-off-by: Kamil Rytarowski > Signed-off-by: Zyta Szpak > Signed-off-by: Slawomir Rosek > Signed-off-by: Radoslaw Biernacki > --- ... > + > + /* Roundup nb_desc to avilable qsize

[dpdk-dev] [PATCH v3 17/20] thunderx/nicvf: add device start, stop and close support

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej Czekaj > Signed-off-by: Kamil Rytarowski > Signed-off-by: Zyta Szpak > Signed-off-by: Slawomir Rosek > Signed-off-by: Radoslaw Biernacki > --- ... > + > + /* Userspace process exited witout

[dpdk-dev] [PATCH] mbuf: remove inconsistent assert statements

2016-06-08 Thread Adrien Mazarguil
Hi Konstantin, On Wed, Jun 08, 2016 at 10:34:17AM +, Ananyev, Konstantin wrote: > Hi Adrien, > > > > > An assertion failure occurs in __rte_mbuf_raw_free() (called by a few PMDs) > > when compiling DPDK with CONFIG_RTE_LOG_LEVEL=RTE_LOG_DEBUG and starting > > applications with a log level

[dpdk-dev] [PATCH v3 19/20] thunderx/nicvf: updated driver documentation and release notes

2016-06-08 Thread Jerin Jacob
On Wed, Jun 08, 2016 at 01:08:35PM +0100, Ferruh Yigit wrote: > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > Updated doc/guides/nics/overview.rst, doc/guides/nics/thunderx.rst > > and release notes > > > > Changed "*" to "P" in overview.rst to capture the partially supported > > feature as "*"

[dpdk-dev] [PATCH v3 00/20] DPDK PMD for ThunderX NIC device

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > This patch set provides the initial version of DPDK PMD for the > built-in NIC device in Cavium ThunderX SoC family. > > Implemented features and ThunderX nicvf PMD documentation added > in doc/guides/nics/overview.rst and doc/guides/nics/thunderx.rst >

[dpdk-dev] [PATCH] mk: generate internal library dependencies from DEPDIRS-y automatically

2016-06-08 Thread Olivier Matz
On 06/07/2016 04:40 PM, Wiles, Keith wrote: > On 6/7/16, 9:19 AM, "dev on behalf of Thomas Monjalon" dpdk.org on behalf of thomas.monjalon at 6wind.com> wrote: > >> 2016-06-07 15:07, Bruce Richardson: >>> On Tue, Jun 07, 2016 at 03:00:45PM +0200, Thomas Monjalon wrote: 2016-06-07 14:36,

[dpdk-dev] [PATCH] eal/linux: fix undefined allocation of 0 bytes (CERT MEM04-C; CWE-131)

2016-06-08 Thread Sergio Gonzalez Monroy
On 27/04/2016 18:06, Daniel Mrzyglod wrote: > Fix issue reported by clang scan-build > > there is a chance that nr_hugepages will be 0 if conditions for loop > for (i = 0; i < (int) internal_config.num_hugepage_sizes; i++) > will be unmeet. > > Fixes: b6a468ad41d5 ("memory: add --socket-mem

[dpdk-dev] [PATCH v3 00/20] DPDK PMD for ThunderX NIC device

2016-06-08 Thread Jerin Jacob
On Wed, Jun 08, 2016 at 01:30:28PM +0100, Ferruh Yigit wrote: > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > Jerin Jacob (20): > > thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC > > thunderx/nicvf: add pmd skeleton > > thunderx/nicvf: add link status and link update

[dpdk-dev] [PATCH v3 12/20] thunderx/nicvf: add single and multi segment tx functions

2016-06-08 Thread Ferruh Yigit
On 6/7/2016 5:40 PM, Jerin Jacob wrote: > Signed-off-by: Jerin Jacob > Signed-off-by: Maciej Czekaj > Signed-off-by: Kamil Rytarowski > Signed-off-by: Zyta Szpak > Signed-off-by: Slawomir Rosek > Signed-off-by: Radoslaw Biernacki > --- > drivers/net/thunderx/Makefile | 2 + >

[dpdk-dev] about rx checksum flags

2016-06-08 Thread Olivier Matz
Hi, On 06/08/2016 10:22 AM, Chandran, Sugesh wrote: >>> I guess the IP checksum also important as L4. In some cases, UDP >>> checksum is zero and no need to validate it. But Ip checksum is >>> present on all the packets and that must be validated all the time. >>> At higher packet rate, the ip

[dpdk-dev] [PATCH] mbuf: remove inconsistent assert statements

2016-06-08 Thread Ananyev, Konstantin
> > Hi Konstantin, > > On Wed, Jun 08, 2016 at 10:34:17AM +, Ananyev, Konstantin wrote: > > Hi Adrien, > > > > > > > > An assertion failure occurs in __rte_mbuf_raw_free() (called by a few > > > PMDs) > > > when compiling DPDK with CONFIG_RTE_LOG_LEVEL=RTE_LOG_DEBUG and starting > > >

[dpdk-dev] [PATCH v3 00/20] DPDK PMD for ThunderX NIC device

2016-06-08 Thread Ferruh Yigit
On 6/8/2016 1:43 PM, Jerin Jacob wrote: > On Wed, Jun 08, 2016 at 01:30:28PM +0100, Ferruh Yigit wrote: >> On 6/7/2016 5:40 PM, Jerin Jacob wrote: >>> Jerin Jacob (20): >>> thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC >>> thunderx/nicvf: add pmd skeleton >>>

[dpdk-dev] [PATCH v3 19/20] thunderx/nicvf: updated driver documentation and release notes

2016-06-08 Thread Bruce Richardson
On Wed, Jun 08, 2016 at 05:57:16PM +0530, Jerin Jacob wrote: > On Wed, Jun 08, 2016 at 01:08:35PM +0100, Ferruh Yigit wrote: > > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > > Updated doc/guides/nics/overview.rst, doc/guides/nics/thunderx.rst > > > and release notes > > > > > > Changed "*" to "P"

[dpdk-dev] [PATCH v3 00/20] DPDK PMD for ThunderX NIC device

2016-06-08 Thread Bruce Richardson
On Wed, Jun 08, 2016 at 06:13:21PM +0530, Jerin Jacob wrote: > On Wed, Jun 08, 2016 at 01:30:28PM +0100, Ferruh Yigit wrote: > > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > > Jerin Jacob (20): > > > thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC > > > thunderx/nicvf: add

[dpdk-dev] [PATCH] app/test: fix bond device name too long

2016-06-08 Thread Thomas Monjalon
2016-06-08 11:50, Jastrzebski, MichalX K: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-05-27 18:38, Thomas Monjalon: > > > 2016-05-27 17:20, Michal Jastrzebski: > > > > Bond device name was too long (grather than 32 signs) that > > > > cause mempool allocation to fail. >

[dpdk-dev] [PATCH v3 00/20] DPDK PMD for ThunderX NIC device

2016-06-08 Thread Jerin Jacob
On Wed, Jun 08, 2016 at 02:22:55PM +0100, Bruce Richardson wrote: > On Wed, Jun 08, 2016 at 06:13:21PM +0530, Jerin Jacob wrote: > > On Wed, Jun 08, 2016 at 01:30:28PM +0100, Ferruh Yigit wrote: > > > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > > > Jerin Jacob (20): > > > > thunderx/nicvf/base:

[dpdk-dev] [PATCH] mk: generate internal library dependencies from DEPDIRS-y automatically

2016-06-08 Thread Thomas Monjalon
2016-06-08 14:34, Olivier Matz: > On 06/07/2016 04:40 PM, Wiles, Keith wrote: > > On 6/7/16, 9:19 AM, "dev on behalf of Thomas Monjalon" > dpdk.org on behalf of thomas.monjalon at 6wind.com> wrote: > > > >> 2016-06-07 15:07, Bruce Richardson: > >>> On Tue, Jun 07, 2016 at 03:00:45PM +0200,

[dpdk-dev] [PATCH v5 0/9] add packet capture framework

2016-06-08 Thread Reshma Pattan
This patch set include below changes 1)Changes to librte_ether. 2)A new library librte_pdump added for packet capture framework. 3)A new app/pdump tool added for packet capturing. 4)Test pmd changes done to initialize packet capture framework. 5)Documentation update. 1)librte_pdump

  1   2   >