Re: [dpdk-dev] [PATCH 03/13] rte_ether: set PKT_RX_VLAN_STRIPPED in rte_vlan_strip()

2017-05-04 Thread Olivier MATZ
Hi Thomas, On Sun, 30 Apr 2017 17:58:45 +0200, Thomas Monjalon wrote: > 09/02/2017 16:56, Olivier MATZ: > > Hi, > > > > On Mon, 30 Jan 2017 10:54:08 +0100, Thomas Monjalon > > wrote: > > > It is fixing the introduction of the new flag PKT_RX_VLAN_STRIPPED. > > > > > > Fixes: b37b528d957c ("m

Re: [dpdk-dev] [PATCH] [RFC] cryptodev: crypto operation restructuring

2017-05-04 Thread Sergio Gonzalez Monroy
On 04/05/2017 07:09, Akhil Goyal wrote: Hi Sergio, On 5/3/2017 7:48 PM, Sergio Gonzalez Monroy wrote: On 03/05/2017 12:01, Akhil Goyal wrote: Hi Pablo, On 4/28/2017 11:33 PM, Pablo de Lara wrote: This is a proposal to correct and improve the current crypto operation (rte_crypto_op) and symme

Re: [dpdk-dev] [PATCH] [RFC] cryptodev: crypto operation restructuring

2017-05-04 Thread Akhil Goyal
On 5/4/2017 1:01 PM, Sergio Gonzalez Monroy wrote: On 04/05/2017 07:09, Akhil Goyal wrote: Hi Sergio, On 5/3/2017 7:48 PM, Sergio Gonzalez Monroy wrote: On 03/05/2017 12:01, Akhil Goyal wrote: Hi Pablo, On 4/28/2017 11:33 PM, Pablo de Lara wrote: This is a proposal to correct and improve th

Re: [dpdk-dev] [PATCH] [RFC] cryptodev: crypto operation restructuring

2017-05-04 Thread Sergio Gonzalez Monroy
On 04/05/2017 08:38, Akhil Goyal wrote: On 5/4/2017 1:01 PM, Sergio Gonzalez Monroy wrote: On 04/05/2017 07:09, Akhil Goyal wrote: Hi Sergio, On 5/3/2017 7:48 PM, Sergio Gonzalez Monroy wrote: On 03/05/2017 12:01, Akhil Goyal wrote: Hi Pablo, On 4/28/2017 11:33 PM, Pablo de Lara wrote: Thi

Re: [dpdk-dev] [PATCH 5/5] examples/l3fwd: add neon support for l3fwd

2017-05-04 Thread Jianbo Liu
Hi Ashwin, On 3 May 2017 at 13:24, Jianbo Liu wrote: > Hi Ashwin, > > On 2 May 2017 at 19:47, Sekhar, Ashwin wrote: >> Hi Jianbo, >> >> I tested your neon changes on thunderx. I am seeing a performance >> regression of ~10% for LPM case and ~20% for EM case with your changes. >> Did you see impr

[dpdk-dev] Patches verified by Intel engineers for DPDK 17.05 RC4.

2017-05-04 Thread Liu, Yu Y
Hi Thomas, Below patches have been verified. It would be great if they can get included into DPDK17.05. http://www.dpdk.org/dev/patchwork/patch/23957/ http://dpdk.org/dev/patchwork/patch/24044/ http://dpdk.org/dev/patchwork/patch/24021/ http://dpdk.org/dev/patchwork/patch/24022/ http://dpdk.or

[dpdk-dev] [PATCH] app/testpmd: fix port_numa and ring_numa not initialize issue

2017-05-04 Thread Yulong Pei
Previous numa_support = 0 by default, it need to add --numa to testpmd command line to enable numa, so port_numa and ring_numa were initialized at function launch_args_parse(), now testpmd change numa_support = 1 as default, so port_numa and ring_numa also need to initialize by default, otherwise p

Re: [dpdk-dev] Patches verified by Intel engineers for DPDK 17.05 RC4.

2017-05-04 Thread Thomas Monjalon
04/05/2017 11:00, Liu, Yu Y: > Hi Thomas, > > Below patches have been verified. It would be great if they can get included > into DPDK17.05. Thanks for the list Yu. > http://www.dpdk.org/dev/patchwork/patch/23957/ > > http://dpdk.org/dev/patchwork/patch/24044/ > > http://dpdk.org/dev/patchwo

[dpdk-dev] [PATCH v5] app/testpmd: fix port_numa and ring_numa not initialize issue

2017-05-04 Thread Yulong Pei
Previous numa_support = 0 by default, it need to add --numa to testpmd command line to enable numa, so port_numa and ring_numa were initialized at function launch_args_parse(), now testpmd change numa_support = 1 as default, so port_numa and ring_numa also need to initialize by default, otherwise p

[dpdk-dev] [PATCH] doc: announce public crypto PMD names removal

2017-05-04 Thread Slawomir Mrozowicz
The following PMD names definitions will be moved to the individual PMDs to remove the coupling between crypto PMDs and the librte_cryptodev: CRYPTODEV_NAME_NULL_PMD CRYPTODEV_NAME_AESNI_MB_PMD CRYPTODEV_NAME_AESNI_GCM_PMD CRYPTODEV_NAME_OPENSSL_PMD CRYPTODEV_NAME_QAT_SYM_PMD CRYPTODEV_NAME_SNOW3G_

[dpdk-dev] [PATCH] net/ixgbe: fix VF Rx mode if allmulticast is disabled

2017-05-04 Thread Wei Dai
Some customers find that 82599 NIC DPDK VF PMD can't receive any broadcast packets when it is bound to igb_uio in the first time to run a DPDK application like testpmd. But when the application is quited and run again, the DPDK VF PMD can receive broadcast packets again. The associated PF is run by

[dpdk-dev] Sharing tables among pipelines

2017-05-04 Thread Nidhia Varghese
Hi, For my pipeline application, I want to share same table between two different pipeline. Is that possible? If yes, how can I do it? Thanks for your reply and help. -- Regards, Nidhia Varghese

[dpdk-dev] [PATCH 1/2] usertools: fix adding devices of same class

2017-05-04 Thread Jerin Jacob
From: Guduri Prathyusha If multiple devices of same class are added to a device type, only devices that match first device listed in device type list are processed. Fixing it in device_type_match() by returning false after iterating through all the devices listed in a device type list. Fixes: 8

[dpdk-dev] [PATCH 2/2] usertools: add cavium pkx device as network device

2017-05-04 Thread Jerin Jacob
Signed-off-by: Jerin Jacob --- usertools/dpdk-devbind.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/usertools/dpdk-devbind.py b/usertools/dpdk-devbind.py index 79e7e8938..f0225b6ce 100755 --- a/usertools/dpdk-devbind.py +++ b/usertools/dpdk-devbind.py @@ -49,8 +49,10 @

[dpdk-dev] DPDK support for wireless NICs

2017-05-04 Thread Arun Kumar
Hi Developers, I believe wireless NICs would be able to handle higher amount of data in future and bringing in support of DPDK for wireless NICs earlier would be beneficial to the wireless community. I would like to work on this. Could you please share some light on how to proceed further. Whethe

[dpdk-dev] [PATCH] net/af_packet: prefer snprintf against strncpy

2017-05-04 Thread Ferruh Yigit
strncpy may left destination buffer not NULL terminated, switched using snprintf to be sure destination buffer is NULL terminated. Coverity issue: 1407495 Coverity issue: 1407498 Fixes: cc68ac4847bc ("net/af_packet: support MTU change") Fixes: 218259590ea4 ("net/af_packet: support promiscuous") S

Re: [dpdk-dev] [PATCH] [RFC] cryptodev: crypto operation restructuring

2017-05-04 Thread Akhil Goyal
On 5/4/2017 1:49 PM, Sergio Gonzalez Monroy wrote: On 04/05/2017 08:38, Akhil Goyal wrote: On 5/4/2017 1:01 PM, Sergio Gonzalez Monroy wrote: On 04/05/2017 07:09, Akhil Goyal wrote: Hi Sergio, On 5/3/2017 7:48 PM, Sergio Gonzalez Monroy wrote: On 03/05/2017 12:01, Akhil Goyal wrote: Hi Pabl

[dpdk-dev] Proposed schedule dates for DPDK 17.08, 17.11 and 18.02

2017-05-04 Thread Mcnamara, John
Hi, The current 17.08 schedule dates are: 17.08 * Proposal deadline:March 3, 2017 * Integration deadline: March 30, 2017 * Release: May1, 2017 http://dpdk.org/dev/roadmap#dates The following are proposed dates for 17.11 and 18.02

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-04 Thread Mcnamara, John
> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, April 20, 2017 7:55 PM > To: dev@dpdk.org > Cc: Yigit, Ferruh ; Mcnamara, John > ; Tahhan, Maryam > Subject: [RFC 17.08] flow_classify: add librte_flow_classify library CCing techbo...@dpdk.org since we would like this RFC add

Re: [dpdk-dev] Minutes of tech-board meeting 2017-04-27

2017-05-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Hemant Agrawal > Sent: Wednesday, May 3, 2017 6:57 AM > To: dev@dpdk.org; techbo...@dpdk.org > Subject: [dpdk-dev] Minutes of tech-board meeting 2017-04-27 > > Hi all, > > Here is the meeting notes for the last

Re: [dpdk-dev] [RFC] service core concept header implementation

2017-05-04 Thread Jerin Jacob
-Original Message- > Date: Thu, 4 May 2017 12:05:54 +0530 > From: Jerin Jacob > To: Harry van Haaren > Cc: dev@dpdk.org, bruce.richard...@intel.com, hemant.agra...@nxp.com, > nipun.gu...@nxp.com, narender.vang...@intel.com, gage.e...@intel.com > Subject: Re: [RFC] service core concept he

[dpdk-dev] Occasional instability in RSS Hashes/Queues from X540 NIC

2017-05-04 Thread Matt Laswell
Hey Folks, I'm seeing some strange behavior with regard to the RSS hash values in my applications and was hoping somebody might have some pointers on where to look. In my application, I'm using RSS to divide work among multiple cores, each of which services a single RX queue. When dealing with a

[dpdk-dev] [PATCH] drivers/net: add generic ethdev macro to get PCI device

2017-05-04 Thread Ferruh Yigit
Instead of many PMD define their own macro, define a generic one in ethdev and use that in PMDs. Signed-off-by: Ferruh Yigit --- drivers/net/ark/ark_ethdev.c | 4 ++-- drivers/net/ark/ark_ethdev.h | 4 drivers/net/avp/avp_ethdev.c | 25 +++-- drivers/net

Re: [dpdk-dev] [PATCH] drivers/net: add generic ethdev macro to get PCI device

2017-05-04 Thread Legacy, Allain
> -Original Message- > From: Ferruh Yigit [mailto:ferruh.yi...@intel.com] > Sent: Thursday, May 04, 2017 9:08 AM > To: Thomas Monjalon; Shepard Siegel; Ed Czeck; John Miller; Legacy, Allain; > Peters, Matt; LU, WENZHUO; ZHANG, HELIN; WU, JINGJING; ANANYEV, > KONSTANTIN; Andrew Rybchenko > C

[dpdk-dev] [PATCH] usertools: add option to unbind all devices

2017-05-04 Thread Ferruh Yigit
-u accepts "dpdk" argument to unbind all devices bound to a DPDK driver. Usage: usertools/dpdk-devbind.py -u dpdk Example: $ usertools/dpdk-devbind.py -s Network devices using DPDK-compatible driver :08:00.1 '...' drv=igb_uio unused= :81:00.0

Re: [dpdk-dev] [PATCH] kni: fix unit test segfault

2017-05-04 Thread Ferruh Yigit
On 5/4/2017 6:23 AM, gowrishankar muthukrishnan wrote: > Hi Ferruh, > Even w/o this patch, I mostly find mbufs returned by rte_kni_rx_burst() > always 0 > (or sometimes 1) where as more than 32000 mbufs approx created in > ingress side > for this test. > >dpdk/test/test $ ./test -l 0,1,2 --s

Re: [dpdk-dev] [PATCH v3 1/4] mk: add crypto capability for generic armv8a and thunderx

2017-05-04 Thread Jan Viktorin
On Wed, 3 May 2017 23:56:59 -0700 Ashwin Sekhar T K wrote: > armv8-a has optional CRYPTO extension which adds the > AES, PMULL, SHA1 and SHA2 capabilities. -march=armv8-a+crypto > enables code generation for the ARMv8-A architecture together > with the optional CRYPTO extensions. > > added the

Re: [dpdk-dev] [PATCH v3 2/4] eal: move gcc version definition to common header

2017-05-04 Thread Jan Viktorin
On Wed, 3 May 2017 23:57:00 -0700 Ashwin Sekhar T K wrote: > moved the definition of GCC_VERSION from lib/librte_table/rte_lru.h s/moved/Moved/ > to lib/librte_eal/common/include/rte_common.h dot after the sentence > > Tested compilation on arm64 with gcc > > Tested compilation on x86 with

[dpdk-dev] [RFC PATCH] cryptodev: make crypto session device independent

2017-05-04 Thread Tomasz Kulasek
This RFC describes a proposal to change crypto device's session management to make it device independent and simplify architecture when session is intended to be used on more than one device. 1. Overview Limitations of existing implementation: a) Each session can store only private data f

Re: [dpdk-dev] [PATCH] eal: remove generic driver and device lists

2017-05-04 Thread Thomas Monjalon
02/05/2017 01:05, Thomas Monjalon: > These lists were unused and useless because they are maintained per bus: > struct rte_driver_list dev_driver_list > struct rte_device_list dev_device_list > > Signed-off-by: Thomas Monjalon > --- > lib/librte_eal/bsdapp/eal/rte_eal_version.map |

[dpdk-dev] [PATCH 00/10] Enable DPDK core build with gcc 7

2017-05-04 Thread Bruce Richardson
Since GCC 7 is now yesterday's news :-), I figured we should try and make DPDK compile successfully with that new version. The main difficulties are because of new warnings being triggered. * New warnings for fall-though in case statements * New warnings for possible truncation using snprintf * Ne

[dpdk-dev] [PATCH 02/10] drivers/net: disable new gcc 7 warnings for base code

2017-05-04 Thread Bruce Richardson
For base code in drivers shared with other projects, disable the new warnings from gcc 7 about unlabelled fall-through in switch statements. Signed-off-by: Bruce Richardson --- drivers/net/e1000/Makefile | 3 +++ drivers/net/fm10k/Makefile | 3 +++ drivers/net/ixgbe/Makefile | 3 +++ drivers/net

[dpdk-dev] [PATCH 01/10] mk: adjust gcc flags for new gcc 7 warnings

2017-05-04 Thread Bruce Richardson
There are two new warnings in GCC 7 that cause problems in the DPDK compile. 1. GCC now warns if you have a switch fall-through without a suitable comment indicating that it was intentional. The compiler supports a number of levels of warning which are triggered depending on the type of message us

[dpdk-dev] [PATCH 03/10] net/bnx2x: fix warnings about switch fall-through

2017-05-04 Thread Bruce Richardson
Add in a comment for each switch fall-through indicating that it is intentional. This will fix compiler warnings with GCC 7. Fixes: b5bf7719221d ("bnx2x: driver support routines") Signed-off-by: Bruce Richardson --- drivers/net/bnx2x/ecore_sp.c | 2 +- drivers/net/bnx2x/elink.c| 3 +++ 2 fi

[dpdk-dev] [PATCH 04/10] net/ixgbe: fix gcc 7 warning for switch fallthrough

2017-05-04 Thread Bruce Richardson
Add a comment documenting explicitly that we are falling through the case statements to the next one. Fixes: f9072f8b90bb ("ixgbe: migrate flow director filtering to new API") Signed-off-by: Bruce Richardson --- drivers/net/ixgbe/ixgbe_fdir.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[dpdk-dev] [PATCH 05/10] net/vmxnet3: fix compile error with gcc 7

2017-05-04 Thread Bruce Richardson
GCC 7 flags a value as uninitialized before used. While it's a false positive, there is little harm in providing an initial value for the variable. Fixes: bb1d14b87fc3 ("vmxnet3: fix link state handling") Signed-off-by: Bruce Richardson --- drivers/net/vmxnet3/vmxnet3_ethdev.c | 2 +- 1 file ch

[dpdk-dev] [PATCH 06/10] lib: fix gcc 7 warnings for switch fall-through

2017-05-04 Thread Bruce Richardson
With GCC 7 we need to explicitly document when we are falling through from one switch case to another. Fixes: af75078fece3 ("first public release") Fixes: 8bae1da2afe0 ("hash: fallback to software CRC32 implementation") Fixes: 9ec201f5d6e7 ("mbuf: provide bulk allocation") Signed-off-by: Bruce Ri

[dpdk-dev] [PATCH 07/10] net: fix missing break inside conditional compile block

2017-05-04 Thread Bruce Richardson
The #ifdef only had the break in the else leg rather than in the first leg, leading to the value set their being overridden on fall-through. Fixes: 986ff526fb84 ("net: add CRC computation API") Signed-off-by: Bruce Richardson --- lib/librte_net/rte_net_crc.c | 2 +- 1 file changed, 1 insertion(

[dpdk-dev] [PATCH 09/10] test/test: fix missing break in switch

2017-05-04 Thread Bruce Richardson
Issue flagged by GCC 7 as a switch fall-through. Fixes: dbb860e03eb1 ("cmdline: tests") CC: sta...@dpdk.org Signed-off-by: Bruce Richardson --- test/test/test_cmdline_num.c | 1 + 1 file changed, 1 insertion(+) diff --git a/test/test/test_cmdline_num.c b/test/test/test_cmdline_num.c index 0426

[dpdk-dev] [PATCH 08/10] app/testpmd: document explicit switch fall-through

2017-05-04 Thread Bruce Richardson
This fixes compiler warnings with GCC 7. Fixes: 28d62131a1b1 ("app/testpmd: extend flow director input set commands") Signed-off-by: Bruce Richardson --- app/test-pmd/cmdline.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 06c1ce2..0

[dpdk-dev] [PATCH 10/10] test/test: fix gcc 7 compiler error

2017-05-04 Thread Bruce Richardson
GCC flags an uninitialized value, so provide an initializer. Fixes: 3a17ff401f1e ("test/eventdev: add basic SW tests") Signed-off-by: Bruce Richardson --- test/test/test_eventdev_sw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test/test_eventdev_sw.c b/test/test/te

[dpdk-dev] [PATCH] doc: announce API changes in crypto library

2017-05-04 Thread Tomasz Kulasek
API changes are planned for 17.08 to made sessions agnostic to the underlaying devices, removing coupling with crypto PMDs, so a single session can be used on multiple devices. It requires to change "struct rte_cryptodev_sym_session" to store more than one private data for devices, as well as remo

[dpdk-dev] [PATCH] net/null: do not touch mbuf next or nb segs on Rx

2017-05-04 Thread Ferruh Yigit
mbuf next and nb_segs fields already have the default values when get from mempool, no need to update them in PMD. See: 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool") Signed-off-by: Ferruh Yigit --- Cc: Olivier Matz --- drivers/net/null/rte_eth_null.c | 2 -- 1 file changed, 2 deletions(

[dpdk-dev] [PATCH] eal: remove forward declaration of generic driver

2017-05-04 Thread Thomas Monjalon
We can just move rte_driver definition before rte_device (which depends on rte_driver). Signed-off-by: Thomas Monjalon --- lib/librte_eal/common/include/rte_dev.h | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/lib/librte_eal/common/include/rte_dev.h b/l

[dpdk-dev] [PATCH] net/ixgbe: do not touch mbuf initialized fields

2017-05-04 Thread Ferruh Yigit
See: 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool") Signed-off-by: Ferruh Yigit --- Cc: Olivier Matz --- drivers/net/ixgbe/ixgbe_rxtx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c index 1e07895..590ab63 100644 --- a

[dpdk-dev] [PATCH 0/2] prepare function names for bus move

2017-05-04 Thread Thomas Monjalon
It is planned to move the bus PCI and VDEV to /drivers/bus/ during 17.08. So the eal_ prefix will not make sense. Let's rename now before releasing 17.05 where EAL API has already some breaking changes. Thomas Monjalon (2): pci: remove eal prefix vdev: remove eal prefix app/test-pmd/testpmd.

[dpdk-dev] [PATCH 1/2] pci: remove eal prefix

2017-05-04 Thread Thomas Monjalon
The PCI code will move to the bus drivers directory. Rename functions from rte_eal_pci_ to rte_pci_ to prepare the move of the driver out of EAL. Signed-off-by: Thomas Monjalon --- app/test-pmd/testpmd.c | 2 +- .../sample_app_ug/l2_forward_real_virtual.rst | 4

[dpdk-dev] [PATCH 2/2] vdev: remove eal prefix

2017-05-04 Thread Thomas Monjalon
The VDEV code will move to the bus drivers directory. Rename functions from rte_eal_vdev_ to rte_vdev_ to prepare the move of the driver out of EAL. The prefix rte_eal_vdrv_ is also renamed to rte_vdev_. It was used for registration of vdev drivers. Signed-off-by: Thomas Monjalon --- doc/guides

Re: [dpdk-dev] [PATCH] [RFC] cryptodev: crypto operation restructuring

2017-05-04 Thread De Lara Guarch, Pablo
> -Original Message- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Thursday, May 04, 2017 12:23 PM > To: Gonzalez Monroy, Sergio; De Lara Guarch, Pablo; dev@dpdk.org > Cc: Doherty, Declan; hemant.agra...@nxp.com; > zbigniew.bo...@caviumnetworks.com; jerin.ja...@caviumnetworks.c

[dpdk-dev] [PATCH v2] app/testpmd: disable latency stats by default

2017-05-04 Thread Pablo de Lara
Disable latency stats gathering by default, so there is not performance degradation if user is not interested in them. Signed-off-by: Pablo de Lara --- This patch depends on http://dpdk.org/dev/patchwork/patch/24064/ app/test-pmd/testpmd.c | 8 ++-- 1 file changed, 6 insertions(+), 2 delet

Re: [dpdk-dev] Occasional instability in RSS Hashes/Queues from X540 NIC

2017-05-04 Thread Wiles, Keith
> On May 4, 2017, at 8:04 AM, Matt Laswell wrote: > > Hey Folks, > > I'm seeing some strange behavior with regard to the RSS hash values in my > applications and was hoping somebody might have some pointers on where to > look. In my application, I'm using RSS to divide work among multiple > co

Re: [dpdk-dev] [PATCH 01/10] mk: adjust gcc flags for new gcc 7 warnings

2017-05-04 Thread Stephen Hemminger
On Thu, 4 May 2017 16:38:13 +0100 Bruce Richardson wrote: > There are two new warnings in GCC 7 that cause problems in the DPDK > compile. > > 1. GCC now warns if you have a switch fall-through without a suitable > comment indicating that it was intentional. The compiler supports a number > of

Re: [dpdk-dev] [PATCH 05/10] net/vmxnet3: fix compile error with gcc 7

2017-05-04 Thread Stephen Hemminger
On Thu, 4 May 2017 16:38:17 +0100 Bruce Richardson wrote: > diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c > b/drivers/net/vmxnet3/vmxnet3_ethdev.c > index 0e8eb75..1cd72c1 100644 > --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c > +++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c > @@ -935,7 +935,7 @@

Re: [dpdk-dev] [PATCH 0/2] prepare function names for bus move

2017-05-04 Thread Stephen Hemminger
On Thu, 4 May 2017 18:18:49 +0200 Thomas Monjalon wrote: > It is planned to move the bus PCI and VDEV to /drivers/bus/ > during 17.08. So the eal_ prefix will not make sense. > Let's rename now before releasing 17.05 where EAL API has > already some breaking changes. > > Thomas Monjalon (2): >

Re: [dpdk-dev] [PATCH] doc: notice for changes in kni structures

2017-05-04 Thread Ferruh Yigit
On 5/3/2017 4:50 PM, Stephen Hemminger wrote: > On Wed, 3 May 2017 17:01:31 +0530 > Hemant Agrawal wrote: > >> Signed-off-by: Hemant Agrawal >> --- >> doc/guides/rel_notes/deprecation.rst | 7 +++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/doc/guides/rel_notes/deprecation.rst >>

Re: [dpdk-dev] [PATCH] doc: notice for changes in kni structures

2017-05-04 Thread Ferruh Yigit
On 5/3/2017 12:31 PM, Hemant Agrawal wrote: > Signed-off-by: Hemant Agrawal > --- > doc/guides/rel_notes/deprecation.rst | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/doc/guides/rel_notes/deprecation.rst > b/doc/guides/rel_notes/deprecation.rst > index a3e7c72..0c1ef2c 100644

Re: [dpdk-dev] Proposed schedule dates for DPDK 17.08, 17.11 and 18.02

2017-05-04 Thread Mcnamara, John
Hi, The previous email had the 17.05 dates instead of the 17.08 dates due to an ENOC0FFEE error. Fixed 17.08 dates below. The current 17.08 schedule dates are: 17.08 * Proposal deadline:May 28, 2017 * Integration deadline: June 29, 2017 * Release:

[dpdk-dev] [PATCH 2/2] app/testpmd: fix MAC endian issue in flow command

2017-05-04 Thread Adrien Mazarguil
MAC addresses are implicitly handled in network order since they are actually byte strings, however this is not properly enforced with MAC masks provided as prefix lengths, which end up inverted on little endian systems. Fixes: 6df81b325fa4 ("app/testpmd: add items eth/vlan to flow command") Sign

[dpdk-dev] [PATCH 1/2] app/testpmd: fix stack overwriting by flow command

2017-05-04 Thread Adrien Mazarguil
The parameter type parser function stores a stack-local address in the global parser context when parsing a "prefix" keyword. This usually translates to "Bad arguments" errors even for correct flow rules as stack gets overwritten by subsequent function calls. Fixes: d3f61b7bad20 ("app/testpmd: ad

[dpdk-dev] Proposal - ARM Support for Arkville PMD in DPDK 17.08

2017-05-04 Thread Shepard Siegel
This email is notification ahead of the May 28 proposal deadline for 17.08 that Atomic Rules proposes to implement, test and support our Arkville net/ark PMD for ARM architecture in the DPDK 17.08 release. Our intent is for ARM architecture support with Arkville to stand equal with the current deep

Re: [dpdk-dev] Occasional instability in RSS Hashes/Queues from X540 NIC

2017-05-04 Thread Matt Laswell
Hey Keith, Here is a hexdump of a subset of one of my packet captures. In this capture, all of the packets are part of the same TCP connection, which happens to be NFSv3 traffic. All of them except packet number 6 get the correct RSS hash and go to the right queue. Packet number 6 (an NFS rename

Re: [dpdk-dev] [PATCH v2 01/13] EAL: count nr_overcommit_hugepages as available

2017-05-04 Thread Michał Mirosław
On Sun, Apr 30, 2017 at 05:53:55PM +0200, Thomas Monjalon wrote: > 13/12/2016 02:28, Michał Mirosław: > > Signed-off-by: Michał Mirosław > > --- > > lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 43 > > ++--- > > 1 file changed, 32 insertions(+), 11 deletions(-) > There i

[dpdk-dev] [PATCH] doc: factorize overview table CSS

2017-05-04 Thread Thomas Monjalon
There were several tables in NIC and crypto guides with the same copy-pasted CSS addition. It is moved into one unique place: conf.py. Signed-off-by: Thomas Monjalon --- doc/guides/conf.py | 75 +++-- doc/guides/cryptodevs/overview.rst | 212 -

Re: [dpdk-dev] [PATCH] maintainers: claim responsability for xen

2017-05-04 Thread Thomas Monjalon
Ping The Xen dom0 support in DPDK seems dead. Reminder: Last time we talked about, it was because of a severe bug which is not fixed yet: http://dpdk.org/ml/archives/dev/2016-July/044207.html http://dpdk.org/ml/archives/dev/2016-July/044376.html The request (9 months ago) was to g

Re: [dpdk-dev] [PATCH v3 1/4] mk: add crypto capability for generic armv8a and thunderx

2017-05-04 Thread Thomas Monjalon
04/05/2017 17:20, Jan Viktorin: > On Wed, 3 May 2017 23:56:59 -0700 > Ashwin Sekhar T K wrote: > > > armv8-a has optional CRYPTO extension which adds the > > AES, PMULL, SHA1 and SHA2 capabilities. -march=armv8-a+crypto > > enables code generation for the ARMv8-A architecture together > > with t

[dpdk-dev] [PATCH v2] net: fix stripped VLAN flag for offload emulation

2017-05-04 Thread Thomas Monjalon
From: Michał Mirosław Apply the new flag PKT_RX_VLAN_STRIPPED to the software emulation case (currently only for virtio and af_packet). Fixes: b37b528d957c ("mbuf: add new Rx flags for stripped VLAN") Cc: sta...@dpdk.org Signed-off-by: Michał Mirosław Signed-off-by: Thomas Monjalon --- v2: ad

Re: [dpdk-dev] [PATCH v6 1/3] ethdev: fix adding invalid MAC addr

2017-05-04 Thread Dai, Wei
> Subject: Re: [PATCH v6 1/3] ethdev: fix adding invalid MAC addr > > On Tue, May 02, 2017 at 08:44:23PM +0800, Wei Dai wrote: > > Some customers find adding MAC addr to VF sometimes can fail, but it > > is still stored in dev->data->mac_addrs[ ]. So this can lead to some > > errors that assumes t

Re: [dpdk-dev] [PATCH] drivers/net: add generic ethdev macro to get PCI device

2017-05-04 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Yigit, Ferruh > Sent: Thursday, May 4, 2017 9:08 PM > To: Thomas Monjalon; Shepard Siegel; Ed Czeck; John Miller; Legacy, Allain > (Wind River); Peters, Matt (Wind River); Lu, Wenzhuo; Zhang, Helin; Wu, > Jingjing; Ananyev, Konstantin; Andrew Rybchenko > Cc

Re: [dpdk-dev] [PATCH 04/10] net/ixgbe: fix gcc 7 warning for switch fallthrough

2017-05-04 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > Sent: Thursday, May 4, 2017 11:38 PM > To: dev@dpdk.org > Cc: Richardson, Bruce > Subject: [dpdk-dev] [PATCH 04/10] net/ixgbe: fix gcc 7 warning for switch > fallthrough > > Add a comment d

[dpdk-dev] [PATCH v7 1/3] ethdev: fix adding invalid MAC addr

2017-05-04 Thread Wei Dai
Some customers find adding MAC addr to VF sometimes can fail, but it is still stored in dev->data->mac_addrs[ ]. So this can lead to some errors that assumes the non-zero entry in dev->data->mac_addrs[ ] is valid. Following acknowledgements are from specific NIC PMD maintainer for their managing pa

[dpdk-dev] [PATCH v7 2/3] doc: change type of return value of adding MAC addr

2017-05-04 Thread Wei Dai
Add following lines in section of API change in release note. If a MAC address fails to be added without this change, it is still stored and may be regarded as a valid one. This may lead to errors in application. The type of return value of eth_mac_addr_add_t in rte_ethdev.h is changed. Any specif

[dpdk-dev] [PATCH v7 0/3] MAC address fail to be added shouldn't be stored

2017-05-04 Thread Wei Dai
Current ethdev always stores the MAC address even it fails to be added. Other function may regard the failed MAC address valid and lead to some errors. So There is a need to check if the addr is added successfully or not and discard it if it fails. In 3rd patch, add a command "add_more_mac_addr po

[dpdk-dev] [PATCH v7 3/3] app/testpmd: add a command to add many MAC addrs

2017-05-04 Thread Wei Dai
This patch is added to introduce a testpmd command which is used to add more than one MAC addresses one time. This command can simplify the test for the change where the type of return value of adding MAC address. Normally a MAC address may fails to be added only after many MAC addresses have been

Re: [dpdk-dev] [PATCH] net/ixgbe: do not touch mbuf initialized fields

2017-05-04 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Yigit, Ferruh > Sent: Thursday, May 4, 2017 11:59 PM > To: Lu, Wenzhuo; Ananyev, Konstantin > Cc: dev@dpdk.org; Olivier Matz > Subject: [PATCH] net/ixgbe: do not touch mbuf initialized fields > > See: 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool") >

Re: [dpdk-dev] [PATCH v2 0/3] add a macro to enable support of backtrace

2017-05-04 Thread Dai, Wei
Hi, Thomas I noticed that my following patch set has been superseded. It address the compile errors when DPDK is built with musl. I know some customer products like Yocto Linux and that from Roph are using DPDK with musl and they have to manually change something to build it. Roph also referenced

Re: [dpdk-dev] [PATCH v7 1/3] ethdev: fix adding invalid MAC addr

2017-05-04 Thread Yang, Qiming
Hi, Dai wei > static void > diff --git a/drivers/net/e1000/igb_ethdev.c > b/drivers/net/e1000/igb_ethdev.c index b6b81cb..e8c6282 100644 > --- a/drivers/net/e1000/igb_ethdev.c > +++ b/drivers/net/e1000/igb_ethdev.c > @@ -171,9 +171,9 @@ static int eth_igb_led_off(struct rte_eth_dev *dev); > > s

Re: [dpdk-dev] [PATCH] net/ixgbe: fix VF Rx mode if allmulticast is disabled

2017-05-04 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Dai, Wei > Sent: Thursday, May 4, 2017 5:55 PM > To: Zhang, Helin; Ananyev, Konstantin; Lu, Wenzhuo > Cc: dev@dpdk.org; Dai, Wei; sta...@dpdk.org > Subject: [PATCH] net/ixgbe: fix VF Rx mode if allmulticast is disabled > > Some customers find that 82599 NI

[dpdk-dev] [PATCH v2] test: add delay time in test alarm

2017-05-04 Thread Qiming Yang
Because accuracy of timing to the microsecond is not guaranteed in rte_eal_alarm_set, this function will not be called before the requested time, but may be called a period of time afterwards which can not be calculated. In order to ensure test alarm running success, this patch added the delay time

[dpdk-dev] How to using add/lookup hash table in multithread environment

2017-05-04 Thread vuonglv
Hi DPDK team, I am using DPDK to build a program in multithread environment. I use 1 thread (on 1 core) to add and delete hash table key ( by rte_hash_add_key() and rte_hash_del_key() function); And use 8 thread (on 8 others core) to lookup (by rte_hash_lookup() function ). As I know, rte_hash_l

[dpdk-dev] [PATCH] doc: add tested Intel platforms with Intel NICs

2017-05-04 Thread Yulong Pei
Add tested Intel platforms with Intel NICs to the release note. Signed-off-by: Yulong Pei --- doc/guides/rel_notes/release_17_05.rst | 68 ++ 1 file changed, 68 insertions(+) diff --git a/doc/guides/rel_notes/release_17_05.rst b/doc/guides/rel_notes/release_17_0

Re: [dpdk-dev] [PATCH 5/5] examples/l3fwd: add neon support for l3fwd

2017-05-04 Thread Sekhar, Ashwin
On Thu, 2017-05-04 at 16:42 +0800, Jianbo Liu wrote: > Hi Ashwin, > > On 3 May 2017 at 13:24, Jianbo Liu wrote: > > > > Hi Ashwin, > > > > On 2 May 2017 at 19:47, Sekhar, Ashwin > > wrote: > > > > > > Hi Jianbo, > > > > > > I tested your neon changes on thunderx. I am seeing a performance >

[dpdk-dev] [PATCH] doc: add known issue about l3fwd-power

2017-05-04 Thread Qiming Yang
Because of UIO only support one interrupt, when insmod ``igb_uio`` and running l3fwd-power APP, link status getting doesn't work properly. Signed-off-by: Qiming Yang --- doc/guides/rel_notes/known_issues.rst | 21 + 1 file changed, 21 insertions(+) diff --git a/doc/guides/re

Re: [dpdk-dev] [PATCH 5/5] examples/l3fwd: add neon support for l3fwd

2017-05-04 Thread Jianbo Liu
On 5 May 2017 at 12:24, Sekhar, Ashwin wrote: > On Thu, 2017-05-04 at 16:42 +0800, Jianbo Liu wrote: >> Hi Ashwin, >> >> On 3 May 2017 at 13:24, Jianbo Liu wrote: >> > >> > Hi Ashwin, >> > >> > On 2 May 2017 at 19:47, Sekhar, Ashwin >> > wrote: >> > > >> > > Hi Jianbo, >> > > >> > > I tested you

Re: [dpdk-dev] [PATCH v2] test: add delay time in test alarm

2017-05-04 Thread Chen, Jing D
Hi, > diff --git a/test/test/test_alarm.c b/test/test/test_alarm.c index > ecb2f6d..cbae1a0 100644 > --- a/test/test/test_alarm.c > +++ b/test/test/test_alarm.c > @@ -96,6 +96,7 @@ static int > test_multi_alarms(void) > { > int rm_count = 0; > + int count = 0; > cb_count.cnt =