Re: [dpdk-dev] Performance issue in DPDK setup

2017-01-18 Thread Arnon Warshavsky
> Is there a application available in DPDK where we can specify the sending > rate i.e. ask the application to send 1 Gbps traffic etc, so that it would > send 1 Gb traffic uniformly in the second. > Please guide us on this. > Hi The DPDK based Pkt-gen is doing a very good job for such uniform pac

[dpdk-dev] Performance issue in DPDK setup

2017-01-18 Thread Priyanka
Hi All, We have VMs on a physical server with both VMs having 1 core 7 GB RAM with DPDK 16.04. The VMs are communicating using SR-IOV (per VM 1 VF is assigned). We are performance benchmarking the setup with our application which sends burst of packet on the sender VM and receiver VM receive

Re: [dpdk-dev] I/O device memory read/write implementation for sfc

2017-01-18 Thread Andrew Rybchenko
On 01/19/2017 08:53 AM, Jerin Jacob wrote: On Wed, Jan 18, 2017 at 07:31:39PM +, Ferruh Yigit wrote: Hi Andrew, Jerin, Existing "I/O device memory read/write" doesn't cover sfc driver because driver was in next-net tree. And both sfc and "I/O device memory read/write" patches will be integ

Re: [dpdk-dev] I/O device memory read/write implementation for sfc

2017-01-18 Thread Jerin Jacob
On Wed, Jan 18, 2017 at 07:31:39PM +, Ferruh Yigit wrote: > Hi Andrew, Jerin, > > Existing "I/O device memory read/write" doesn't cover sfc driver because > driver was in next-net tree. > > And both sfc and "I/O device memory read/write" patches will be > integrated into main tree for rc1 rel

Re: [dpdk-dev] [PATCH v5 00/29] introduce I/O device memory read/write operations

2017-01-18 Thread Jerin Jacob
On Thu, Jan 19, 2017 at 05:51:38AM +0100, Thomas Monjalon wrote: > 2017-01-19 09:44, Jerin Jacob: > > On Wed, Jan 18, 2017 at 05:25:22PM +0100, Thomas Monjalon wrote: > > > Does it deserve an entry in the release notes? > > > > Not sure. If you like to add this in release note, then I can post a p

[dpdk-dev] [PATCH] doc: update release notes for I/O device memory access API

2017-01-18 Thread Jerin Jacob
Signed-off-by: Jerin Jacob --- doc/guides/rel_notes/release_17_02.rst | 13 + 1 file changed, 13 insertions(+) diff --git a/doc/guides/rel_notes/release_17_02.rst b/doc/guides/rel_notes/release_17_02.rst index 0ecd720..eb0e3c0 100644 --- a/doc/guides/rel_notes/release_17_02.rst +++

[dpdk-dev] [PATCH] doc: announce ABI change for cloud filter

2017-01-18 Thread Yong Liu
Signed-off-by: Yong Liu diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 755dc65..0defd02 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -62,3 +62,7 @@ Deprecation Notices PMDs that implement the la

[dpdk-dev] [dpdk-announce] release candidate 17.02-rc1

2017-01-18 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: http://dpdk.org/browse/dpdk/tag/?id=v17.02-rc1 It is the first release candidate for DPDK 17.02. There were a lot of things not ready until yesterday. Fortunately things have been fixed in a rush for most of you. Some features are still m

Re: [dpdk-dev] [PATCH v5 00/29] introduce I/O device memory read/write operations

2017-01-18 Thread Thomas Monjalon
2017-01-19 09:44, Jerin Jacob: > On Wed, Jan 18, 2017 at 05:25:22PM +0100, Thomas Monjalon wrote: > > Does it deserve an entry in the release notes? > > Not sure. If you like to add this in release note, then I can post a patch > for the same. Why not? It is an important step for ARM in DPDK, is

Re: [dpdk-dev] [PATCH v11 00/13] rte_bus + rte_pci_bus

2017-01-18 Thread Thomas Monjalon
2017-01-19 05:45, Thomas Monjalon: > Unfortunately, it is not ready to be integrated in 17.02. > More specifically: > - the tests need some review/rework > - the PCI implementation requires more work > - the documentation must be updated > > However there are some new buses waiting for the b

[dpdk-dev] [PATCH v6 1/2] kni: add bulk function to free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Suggested-by: Stephen Hemminger Signed-off-by: Sergey Vyazmitinov --- v3: * Fixed issue with possible different mempools in buffer list. * Fixed issue with wrong rte_pktmbuf_alloc_bulk function return value processing in the kni_allocate_mbufs. --- lib/librte_mbuf/rte_mbuf.h | 49 +++

[dpdk-dev] [PATCH v6 2/2] kni: Use bulk functions to allocate and free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Optimized kni_allocate_mbufs and kni_free_mbufs by using mbuf bulk functions. This can improve performance more than two times. Signed-off-by: Sergey Vyazmitinov --- v5: * use rte_pktmbuf_free_bulk for removing packets which was not put in alloc queue. v6: * fix c99 compilation error --- lib/lib

[dpdk-dev] [PATCH v6 0/2] kni: use bulk functions to allocate and free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Optimized kni_allocate_mbufs and kni_free_mbufs by using mbuf bulk functions Sergey Vyazmitinov (2): kni: add bulk function to free mbufs kni: Use bulk functions to allocate and free mbufs lib/librte_kni/rte_kni.c | 47 +++-- lib/librte_kni/rte_kni_fi

[dpdk-dev] [PATCH v11 13/13] doc: remove deprecation notice for rte_bus

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Signed-off-by: Shreyansh Jain --- doc/guides/rel_notes/deprecation.rst | 5 - doc/guides/rel_notes/release_17_02.rst | 2 +- lib/librte_eal/bsdapp/eal/Makefile | 2 +- lib/librte_eal/linuxapp/eal/Makefile | 2 +- 4 files changed, 3 insertions(+), 8 deletions(-)

[dpdk-dev] [PATCH v11 11/13] pci: use bus driver for scan/probe

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Remove EAL initiated direct PCI scan/probe and enable PCI Bus linkage. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit Signed-off-by: Thomas Monjalon --- lib/librte_eal/bsdapp/eal/eal.c | 7 - lib/librte_eal/bsdapp/eal/eal_pci.c |

[dpdk-dev] [PATCH v11 12/13] pci: use bus driver for attach/detach

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Given a bus, attach and detach callbacks allow the implementation to handles calls from EAL for attaching or detaching a named device. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- lib/librte_eal/bsdapp/eal/eal_pci.c | 2 + lib/librte_eal/common/eal_com

[dpdk-dev] [PATCH v11 10/13] app/test: add PCI bus driver

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- app/test/test_pci.c | 164 1 file changed, 114 insertions(+), 50 deletions(-) diff --git a/app/test/test_pci.c b/app/test/test_pci.c index cda186d..09261cc 100

[dpdk-dev] [PATCH v11 07/13] pci: split match and probe

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Matching of PCI device address and driver ID table is being done at two discreet locations duplicating the code. (rte_eal_pci_probe_one_driver and rte_eal_pci_detach_dev). Splitting the matching function into a public fn rte_pci_match. Signed-off-by: Shreyansh Jain Reviewe

[dpdk-dev] [PATCH v11 09/13] pci: add bus driver

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Based on EAL Bus APIs, PCI bus callbacks and support functions are introduced in this patch. EAL continues to have direct PCI init/scan calls as well. These would be removed in subsequent patches to enable bus only PCI devices. Signed-off-by: Shreyansh Jain Reviewed-by: Fe

[dpdk-dev] [PATCH v11 05/13] app/test: check bus scan

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Signed-off-by: Shreyansh Jain --- app/test/test_bus.c | 174 1 file changed, 174 insertions(+) diff --git a/app/test/test_bus.c b/app/test/test_bus.c index e2a58fa..de4f1ab 100644 --- a/app/test/test_bus.c +++ b/app/test

[dpdk-dev] [PATCH v11 08/13] pci: remove loop over drivers in device detach

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain rte_eal_pci_detach calls pci_detach_all_drivers which loops over all PCI drivers for detaching the device. This is unnecessary as the device already has the PCI driver reference which can be used directly. Removing pci_detach_all_drivers and restructuring rte_eal_pci_detach

[dpdk-dev] [PATCH v11 06/13] app/test: check bus probe

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Signed-off-by: Shreyansh Jain --- app/test/test_bus.c | 149 1 file changed, 149 insertions(+) diff --git a/app/test/test_bus.c b/app/test/test_bus.c index de4f1ab..a3220f3 100644 --- a/app/test/test_bus.c +++ b/app/test

[dpdk-dev] [PATCH v11 04/13] app/test: check bus registration

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Verification of bus registration, deregistration methods. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- MAINTAINERS | 1 + app/test/Makefile | 2 +- app/test/autotest_data.py

[dpdk-dev] [PATCH v11 02/13] bus: add scanning

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Scan for bus discovers the devices available on the bus and adds them to a bus specific device list. Each bus mandatorily implements this method. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit Signed-off-by: Thomas Monjalon --- lib/librte_eal/bsdapp/eal/eal.c

[dpdk-dev] [PATCH v11 03/13] bus: add probing

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain Bus implementations can implement a probe handler to match the devices scanned against the drivers registered. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit Signed-off-by: Thomas Monjalon --- lib/librte_eal/bsdapp/eal/eal.c | 4 lib/librte

[dpdk-dev] [PATCH v11 00/13] rte_bus + rte_pci_bus

2017-01-18 Thread Thomas Monjalon
I've continued the work done by Shreyansh for bus abstraction and PCI implementation. This change has two benefits: - ease future introduction of new bus - clean existing buses (only PCI here) The same kind of work must be done on VDEV in order to move these implementations as bus drivers. cha

[dpdk-dev] [PATCH v11 01/13] bus: introduce bus abstraction

2017-01-18 Thread Thomas Monjalon
From: Shreyansh Jain This patch introduces the rte_bus abstraction for EAL. The model is: - One or more devices are connected to a Bus - Drivers are running instances which manage one or more devices - Bus is responsible for identifying devices (and interrupt propogation) - Driver is responsi

[dpdk-dev] [PATCH v5 2/2] kni: Use bulk functions to allocate and free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Optimized kni_allocate_mbufs and kni_free_mbufs by using mbuf bulk functions. This can improve performance more than two times. Signed-off-by: Sergey Vyazmitinov --- v5: * use rte_pktmbuf_free_bulk for removing packets which was not put in alloc queue. --- lib/librte_kni/rte_kni.c | 47

[dpdk-dev] [PATCH v5 1/2] kni: add bulk function to free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Suggested-by: Stephen Hemminger Signed-off-by: Sergey Vyazmitinov --- v3: * Fixed issue with possible different mempools in buffer list. * Fixed issue with wrong rte_pktmbuf_alloc_bulk function return value processing in the kni_allocate_mbufs. --- lib/librte_mbuf/rte_mbuf.h | 49 +++

[dpdk-dev] [PATCH v5 0/2] kni: use bulk functions to allocate and free mbufs

2017-01-18 Thread Sergey Vyazmitinov
Optimized kni_allocate_mbufs and kni_free_mbufs by using mbuf bulk functions Sergey Vyazmitinov (2): kni: add bulk function to free mbufs kni: Use bulk functions to allocate and free mbufs lib/librte_kni/rte_kni.c | 47 +++-- lib/librte_kni/rte_kni_fi

Re: [dpdk-dev] [PATCH v5 00/29] introduce I/O device memory read/write operations

2017-01-18 Thread Jerin Jacob
On Wed, Jan 18, 2017 at 05:25:22PM +0100, Thomas Monjalon wrote: > 2017-01-18 06:51, Jerin Jacob: > > Based on the discussion in the below-mentioned thread, > > http://dev.dpdk.narkive.com/DpIRqDuy/dpdk-dev-patch-v2-i40e-fix-eth-i40e-dev-init-sequence-on-thunderx > > > > This patchset introduces 8

Re: [dpdk-dev] [PATCH v2 0/4] eal/common: introduce rte_memset and related test

2017-01-18 Thread Yang, Zhiyong
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Wednesday, January 18, 2017 3:43 PM > To: Yang, Zhiyong > Cc: Richardson, Bruce ; Ananyev, Konstantin > ; yuanhan@linux.intel.com; De Lara > Guarch, Pablo ; dev@dpdk.org > Subject: Re: [dpdk-dev] [

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/virtio-user: fix missing driver name

2017-01-18 Thread Tan, Jianfeng
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, January 18, 2017 11:55 PM > To: Tan, Jianfeng; dev@dpdk.org > Cc: Liu, Yuanhan; sta...@dpdk.org > Subject: Re: [dpdk-stable] [PATCH] net/virtio-user: fix missing driver name > > On 1/18/2017 1:49 PM, Tan, Jianfeng wr

Re: [dpdk-dev] [PATCH v4 2/2] kni: Use bulk functions to allocate and free mbufs

2017-01-18 Thread Michał Mirosław
2017-01-18 11:28 GMT+01:00 Sergey Vyazmitinov : > Optimized kni_allocate_mbufs and kni_free_mbufs by using mbuf bulk > functions. This can improve performance more than two times. > > Signed-off-by: Sergey Vyazmitinov > --- > lib/librte_kni/rte_kni.c | 46 > +

[dpdk-dev] [PATCH 1/2] pkg: add aarch64 support

2017-01-18 Thread Anders Roxell
Signed-off-by: Anders Roxell --- pkg/dpdk.spec | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/pkg/dpdk.spec b/pkg/dpdk.spec index 189cb2f2..a27bdd3b 100644 --- a/pkg/dpdk.spec +++ b/pkg/dpdk.spec @@ -40,10 +40,16 @@ Summary: Data Plane Development Kit core Group: Syst

[dpdk-dev] [PATCH 2/2] pkg: remove xen-devel for aarch64

2017-01-18 Thread Anders Roxell
Disable xen-devel since it just got available for aarch64 in Fedora 25. Signed-off-by: Anders Roxell --- pkg/dpdk.spec | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/dpdk.spec b/pkg/dpdk.spec index a27bdd3b..ad30ecf8 100644 --- a/pkg/dpdk.spec +++ b/pkg/dpdk.spec @

[dpdk-dev] [PATCH] pkg: add dkms to build kernel module

2017-01-18 Thread Anders Roxell
Build igb-uio and rte-kni kernel module using dkms. Signed-off-by: Anders Roxell --- pkg/dpdk.spec | 91 ++- 1 file changed, 90 insertions(+), 1 deletion(-) diff --git a/pkg/dpdk.spec b/pkg/dpdk.spec index 43ff9548..189cb2f2 100644 --- a/p

Re: [dpdk-dev] [PATCH v6 0/8] Add crypto PMD optimized for ARMv8

2017-01-18 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of > zbigniew.bo...@caviumnetworks.com > Sent: Wednesday, January 18, 2017 8:02 PM > To: dev@dpdk.org > Cc: De Lara Guarch, Pablo; Doherty, Declan; > jerin.ja...@caviumnetworks.com; jianbo@linaro.org; > hemant.ag

Re: [dpdk-dev] net/qede/base: build is broken on FreeBSD 11.0 with clang 3.8.0

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 1:33 AM, Mody, Rasesh wrote: >> From: Andrew Rybchenko [mailto:arybche...@solarflare.com] >> Sent: Tuesday, January 17, 2017 7:00 AM >> >> Hi, >> >> build is broken for me on FreeBSD 11.0. Just default config and gmake: >> >> == Build drivers/net/qede >>CC base/ecore_dev.o >>CC

Re: [dpdk-dev] [PATCH v7 5/6] lib: added new library for latency stats

2017-01-18 Thread Olivier Matz
Hi guys, On Tue, 17 Jan 2017 21:55:16 +0530, Jerin Jacob > Oliver, > > Could you please suggest how to proceed further? > Sorry for the lack of response. I know people are waiting for me, but these days I have too many things to do at the same time, and it's difficult to find time. In few word

[dpdk-dev] [PATCH v6 8/8] app/test: add ARMv8 crypto tests and test vectors

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Introduce unit tests for ARMv8 crypto PMD. Add test vectors for short cases such as 160 bytes. These test cases are ARMv8 specific since the code provides different processing paths for different input data sizes. User can validate correctness of algorithms' implementation u

[dpdk-dev] [PATCH v6 7/8] MAINTAINERS: update MAINTAINERS entry for ARMv8 crypto

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8d0fe40..0a1c889 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -458,6 +458,12 @@ M: Declan Doherty F: drivers/

[dpdk-dev] [PATCH v6 6/8] crypto/armv8: enable ARMv8 PMD in the configuration

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO option to the common configuration file. Don't enable it by default for ARM64 as it requires external library to build. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- config/common_base | 6 ++ 1 file changed, 6 inser

[dpdk-dev] [PATCH v6 5/8] doc: update documentation about ARMv8 crypto PMD

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add documentation about the driver and update release notes. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- doc/guides/cryptodevs/armv8.rst| 98 ++ doc/guides/cryptodevs/index.rst| 1 + doc/guides/cryptodevs/ove

[dpdk-dev] [PATCH v6 4/8] cryptodev/armv8: introduce ARM-specific feature flags

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add two new feature flags: * RTE_CRYPTODEV_FF_CPU_NEON represents ARM NEON (TM) instructions * RTE_CRYPTODEV_FF_CPU_ARM_CE represents ARM crypto extensions Add them to both cryptodev library, documentation and relevant PMD driver for ARMv8. Signed-off-by: Zbigniew Bodek

[dpdk-dev] [PATCH v6 3/8] mk: add PMD to the build system

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Build ARMv8 crypto PMD if compiling for ARM64 and CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO option is enable in the configuration file. ARMV8_CRYPTO_LIB_PATH environment variable will point to the appropriate library directory. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jaco

[dpdk-dev] [PATCH v6 0/8] Add crypto PMD optimized for ARMv8

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Introduce crypto poll mode driver using ARMv8 cryptographic extensions. This PMD is optimized to provide performance boost for chained crypto operations processing, such as: * encryption + HMAC generation * decryption + HMAC validation. In particular, cipher only or hash only

[dpdk-dev] [PATCH v6 2/8] crypto/armv8: add PMD optimized for ARMv8 processors

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek This patch introduces crypto poll mode driver using ARMv8 cryptographic extensions. CPU compatibility with this driver is detected in run-time and virtual crypto device will not be created if CPU doesn't provide: AES, SHA1, SHA2 and NEON. This PMD is optimized to provide per

[dpdk-dev] [PATCH v6 1/8] cryptodev: add cryptodev type for the ARMv8 PMD

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add type and name for ARMv8 crypto PMD Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- lib/librte_cryptodev/rte_cryptodev.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/librte_cryptodev/rte_cryptodev.h b/lib/librte_cryptodev/rte_cryptodev.h index

Re: [dpdk-dev] [PATCH v8 0/6] Elastic Flow Distributor

2017-01-18 Thread Thomas Monjalon
2017-01-17 22:23, Pablo de Lara: > EFD is a distributor library that uses perfect hashing to determine a > target/value for a given incoming flow key. It has the following advantages: > first, because it uses perfect hashing it does not store the key itself and > hence lookup performance is not dep

Re: [dpdk-dev] [PATCH v5 4/7] doc: update documentation about ARMv8 crypto PMD

2017-01-18 Thread De Lara Guarch, Pablo
Hi Zbigniew, > -Original Message- > From: Zbigniew Bodek [mailto:zbigniew.bo...@caviumnetworks.com] > Sent: Wednesday, January 18, 2017 7:52 PM > To: De Lara Guarch, Pablo; dev@dpdk.org > Cc: Doherty, Declan; jerin.ja...@caviumnetworks.com; > jianbo@linaro.org; hemant.agra...@nxp.com >

Re: [dpdk-dev] [PATCH v5 4/7] doc: update documentation about ARMv8 crypto PMD

2017-01-18 Thread Zbigniew Bodek
Hello Pablo, Thanks for the remarks. Please check my answers in-line below. Kind regards Zbigniew On 18.01.2017 18:05, De Lara Guarch, Pablo wrote: Hi Bodek, -Original Message- From: zbigniew.bo...@caviumnetworks.com [mailto:zbigniew.bo...@caviumnetworks.com] Sent: Wednesday, January

[dpdk-dev] [PATCH 2/2] acl: allow zero verdict

2017-01-18 Thread Michał Mirosław
This enables ACL matches to return 0 where the distinction from no-match case is not needed. From: Michał Mirosław Signed-off-by: Michał Mirosław --- app/test/test_acl.c | 13 - doc/guides/prog_guide/packet_classif_access_ctrl.rst | 3 ++- lib/librt

Re: [dpdk-dev] [PATCH v8 1/6] efd: new Elastic Flow Distributor library

2017-01-18 Thread Thomas Monjalon
2017-01-18 19:27, De Lara Guarch, Pablo: > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > > Sent: Wednesday, January 18, 2017 6:56 PM > > To: De Lara Guarch, Pablo > > Cc: dev@dpdk.org; Marohn, Byron; Edupuganti, Saikrishna > > Subject: Re: [dpdk-dev]

[dpdk-dev] [PATCH 1/2] acl: remove invalid test

2017-01-18 Thread Michał Mirosław
rte_acl_add_rules() has no way of checking rule size. This was hidden because the test effectively checked that adding a rule with userdata == 0 failed. From: Michał Mirosław Signed-off-by: Michał Mirosław --- app/test/test_acl.c | 20 1 file changed, 20 deletions(-) diff

[dpdk-dev] [PATCH 0/2] Enable zero verdicts in ACLs

2017-01-18 Thread Michał Mirosław
This set enables one to have ACL matches return 0 where the distinction from no-match case is not needed. This is a resubmission of the patches as a series, rebased on net-next tree, no other changes vs v2. v2: fixes to prog_guide and ACL tests Michał Mirosław (2): acl: remove invalid test a

[dpdk-dev] I/O device memory read/write implementation for sfc

2017-01-18 Thread Ferruh Yigit
Hi Andrew, Jerin, Existing "I/O device memory read/write" doesn't cover sfc driver because driver was in next-net tree. And both sfc and "I/O device memory read/write" patches will be integrated into main tree for rc1 release. If "I/O device memory read/write" implementation can be done for sfc

Re: [dpdk-dev] [PATCH v8 1/6] efd: new Elastic Flow Distributor library

2017-01-18 Thread De Lara Guarch, Pablo
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Wednesday, January 18, 2017 6:56 PM > To: De Lara Guarch, Pablo > Cc: dev@dpdk.org; Marohn, Byron; Edupuganti, Saikrishna > Subject: Re: [dpdk-dev] [PATCH v8 1/6] efd: new Elastic Flow Distributor > li

Re: [dpdk-dev] [PATCH 0/4] Solarflare PMD link status and flow control fixes

2017-01-18 Thread Ferruh Yigit
On 1/12/2017 9:03 AM, Andrew Rybchenko wrote: > Fix few problems with link status information and flow control settings. > > Patches are grouped into a series since flow control fix discovers > link status sync issue just after port start. > > Andrew Rybchenko (4): > net/sfc: avoid usage of pos

Re: [dpdk-dev] [PATCH 08/13] PMD/af_packet: guard against buffer overruns in RX path

2017-01-18 Thread Michał Mirosław
On Wed, Jan 18, 2017 at 11:48:52AM +, Ferruh Yigit wrote: > On 12/13/2016 1:08 AM, Michał Mirosław wrote: > > Signed-off-by: Michał Mirosław > Form main address () which sets git author > field, and Signed-off mail address are different. > > As author, any objection to use signed-off mail: >

Re: [dpdk-dev] [PATCH] acl: remove invalid test

2017-01-18 Thread Michal Miroslaw
On Wed, Jan 18, 2017 at 09:51:45AM +, Ananyev, Konstantin wrote: > Hi Thomas, > > > > > 2016-12-23 09:36, Ananyev, Konstantin: > > > From: Michal Miroslaw [mailto:mirq-li...@rere.qmqm.pl] > > > > On Mon, Dec 19, 2016 at 06:48:52PM +, Ananyev, Konstantin wrote: > > > > > I suppose that cha

Re: [dpdk-dev] replay MAC address in rte_eth_dev_config_restore()

2017-01-18 Thread Steve Shin (jonshin)
Thanks Ferruh for your quick turnaround! Sure, I’ll send a patch for your review. Regards, Steve On 1/18/17, 10:39 AM, "Ferruh Yigit" wrote: On 1/18/2017 4:12 AM, Steve Shin (jonshin) wrote: > Hi, > > I have a question on MAC address replay logic in rte_eth_dev_config_restore

Re: [dpdk-dev] [PATCH v8 1/6] efd: new Elastic Flow Distributor library

2017-01-18 Thread Thomas Monjalon
2017-01-17 22:23, Pablo de Lara: > +# this lib depends upon: > +DEPDIRS-$(CONFIG_RTE_LIBRTE_EFD) += lib/librte_eal > +DEPDIRS-$(CONFIG_RTE_LIBRTE_EFD) += lib/librte_ring It depends also on rte_hash. Pablo, I think you know this library ;) I will fix it for you. > +#include > +#include

Re: [dpdk-dev] [PATCH] devtools: fix lookup commit fixing a fix of many commits

2017-01-18 Thread Thomas Monjalon
2017-01-18 17:35, Ferruh Yigit: > On 1/17/2017 2:53 PM, Thomas Monjalon wrote: > > There was a bug when looking at a commit fixing a commit which > > itself was fixing many commits: > > I am having hard time to understand this .. Is it only me J Me too, and the title is a joke itself :) I should

Re: [dpdk-dev] [PATCH v2] doc: add known uio_pci_generic issue for i40e

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 2:48 AM, Guo, Jia wrote: > > > On 1/18/2017 1:36 AM, Ferruh Yigit wrote: >> On 1/17/2017 5:00 PM, Wu, Jingjing wrote: >>> -Original Message- From: Yigit, Ferruh Sent: Tuesday, January 17, 2017 4:31 AM To: Guo, Jia ; Zhang, Helin ; Wu, Jingjing C

Re: [dpdk-dev] replay MAC address in rte_eth_dev_config_restore()

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 4:12 AM, Steve Shin (jonshin) wrote: > Hi, > > I have a question on MAC address replay logic in rte_eth_dev_config_restore(): > > lib/librte_ether/rte_ethdev.c: > code snippet of rte_eth_dev_config_restore() > ~~ > /* replay MAC address configuration */ > for (i = 0;

Re: [dpdk-dev] [PATCH v6 14/18] net/ixgbe: parse L2 tunnel filter

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 1:59 AM, Zhao1, Wei wrote: > Hi, Ferruh > >> -Original Message- >> From: Yigit, Ferruh >> Sent: Tuesday, January 17, 2017 6:03 PM >> To: Zhao1, Wei ; Adrien Mazarguil >> >> Cc: dev@dpdk.org; Lu, Wenzhuo >> Subject: Re: [dpdk-dev] [PATCH v6 14/18] net/ixgbe: parse L2 tunnel f

Re: [dpdk-dev] [PATCH] devtools: fix lookup commit fixing a fix of many commits

2017-01-18 Thread Ferruh Yigit
On 1/17/2017 2:53 PM, Thomas Monjalon wrote: > There was a bug when looking at a commit fixing a commit which > itself was fixing many commits: I am having hard time to understand this .. Is it only me J

[dpdk-dev] [PATCH] doc: fix incomplete crypto dev matrices

2017-01-18 Thread Pablo de Lara
When ZUC PMD was added, it was not added in the Crypto Device Supported Functionality Matrices. This commit adds a column in all the matrices, plus the ZUC EEA3/EIA3 algorithms. Fixes: cf7685d68f00 ("crypto/zuc: add driver for ZUC library") Signed-off-by: Pablo de Lara --- doc/guides/cryptodevs

Re: [dpdk-dev] [PATCH] devtools: check stable tag in fixes

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 4:41 AM, Yuanhan Liu wrote: > On Tue, Jan 17, 2017 at 07:42:33PM +0100, Thomas Monjalon wrote: >> 2017-01-17 18:15, Ferruh Yigit: >>> On 1/17/2017 2:54 PM, Thomas Monjalon wrote: The tag "Cc: sta...@dpdk.org" must be set when the commit must be backported to a stable branch.

Re: [dpdk-dev] [PATCH v5 4/7] doc: update documentation about ARMv8 crypto PMD

2017-01-18 Thread De Lara Guarch, Pablo
Hi Bodek, > -Original Message- > From: zbigniew.bo...@caviumnetworks.com > [mailto:zbigniew.bo...@caviumnetworks.com] > Sent: Wednesday, January 18, 2017 2:27 PM > To: dev@dpdk.org > Cc: De Lara Guarch, Pablo; Doherty, Declan; > jerin.ja...@caviumnetworks.com; jianbo@linaro.org; > hema

Re: [dpdk-dev] [PATCH v2] eal: Support running as unprivileged user

2017-01-18 Thread Sergio Gonzalez Monroy
On 03/01/2017 22:56, Ben Walker wrote: For Linux kernel 4.0 and newer, the ability to obtain physical page frame numbers for unprivileged users from /proc/self/pagemap was removed. Instead, when an IOMMU is present, simply choose our own DMA addresses instead. Signed-off-by: Ben Walker @@ -255,

Re: [dpdk-dev] [PATCH v5 00/29] introduce I/O device memory read/write operations

2017-01-18 Thread Thomas Monjalon
2017-01-18 06:51, Jerin Jacob: > Based on the discussion in the below-mentioned thread, > http://dev.dpdk.narkive.com/DpIRqDuy/dpdk-dev-patch-v2-i40e-fix-eth-i40e-dev-init-sequence-on-thunderx > > This patchset introduces 8-bit, 16-bit, 32bit, 64bit I/O device > memory read/write operations along

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/virtio-user: fix missing driver name

2017-01-18 Thread Ferruh Yigit
On 1/18/2017 1:49 PM, Tan, Jianfeng wrote: > Hi Ferruh, > >> -Original Message- >> From: Yigit, Ferruh >> Sent: Wednesday, January 18, 2017 7:53 PM >> To: Tan, Jianfeng; dev@dpdk.org >> Cc: Liu, Yuanhan; sta...@dpdk.org >> Subject: Re: [dpdk-stable] [PATCH] net/virtio-user: fix missing dri

Re: [dpdk-dev] [PATCH v2] devtools: relax tag checking in fixes

2017-01-18 Thread Thomas Monjalon
2017-01-18 18:04, Yuanhan Liu: > On Wed, Jan 18, 2017 at 10:37:52AM +0100, Thomas Monjalon wrote: > > The tag "Cc: sta...@dpdk.org" must be set when the commit must be > > backported to a stable branch. The reminder is reworded. > > > > It should be located just below the "Fixes:" tag (without bla

Re: [dpdk-dev] [PATCH] devtools: fix lookup commit fixing a fix of many commits

2017-01-18 Thread Thomas Monjalon
2017-01-17 15:53, Thomas Monjalon: > There was a bug when looking at a commit fixing a commit which > itself was fixing many commits: > > % devtools/git-log-fixes.sh 12ee45a36~..12ee45a36 > devtools/git-log-fixes.sh: 96: local: 5499c1fc9baa: bad variable name > > In this case, the list of commits

Re: [dpdk-dev] [PATCH v5 0/7] Add crypto PMD optimized for ARMv8

2017-01-18 Thread Jerin Jacob
On Wed, Jan 18, 2017 at 03:27:23PM +0100, zbigniew.bo...@caviumnetworks.com wrote: > From: Zbigniew Bodek > > Introduce crypto poll mode driver using ARMv8 > cryptographic extensions. This PMD is optimized > to provide performance boost for chained > crypto operations processing, such as: > * en

[dpdk-dev] [PATCH v9 7/7] app/test-pmd: add latency statistics calculation

2017-01-18 Thread Remy Horton
From: Harry van Haaren This patch adds latency stats commandline argument to testpmd, allowing to specify the lcore to use for latencystats updates. Signed-off-by: Reshma Pattan Signed-off-by: Harry van Haaren Signed-off-by: Remy Horton --- app/test-pmd/parameters.c | 20 +++-

[dpdk-dev] [PATCH v9 5/7] mbuf: add a timestamp to the mbuf for latencystats

2017-01-18 Thread Remy Horton
From: Harry van Haaren This commit adds a uint64_t to the mbuf struct, allowing collection of latency and jitter statistics by measuring packet I/O timestamps. This change is required by the latencystats library. Signed-off-by: Reshma Pattan Signed-off-by: Harry van Haaren --- lib/librte_mbuf

[dpdk-dev] [PATCH v9 6/7] lib: added new library for latency stats

2017-01-18 Thread Remy Horton
From: Harry van Haaren Add a library designed to calculate latency statistics and report them to the application when queried. The library measures minimum, average and maximum latencies, and jitter in nano seconds. The current implementation supports global latency stats, i.e. per application st

[dpdk-dev] [PATCH v9 4/7] app/test-pmd: add bitrate statistics calculation

2017-01-18 Thread Remy Horton
Calculate bitrate statistics using the bitrate stats library. The resulting statistics can be viewed via proc_info. Signed-off-by: Remy Horton --- app/test-pmd/testpmd.c | 41 - 1 file changed, 40 insertions(+), 1 deletion(-) diff --git a/app/test-pmd/tes

[dpdk-dev] [PATCH v9 3/7] lib: add bitrate statistics library

2017-01-18 Thread Remy Horton
This patch adds a library that calculates peak and average data-rate statistics. For ethernet devices. These statistics are reported using the metrics library. Signed-off-by: Remy Horton --- MAINTAINERS| 4 + config/common_base

[dpdk-dev] [PATCH v9 1/7] lib: add information metrics library

2017-01-18 Thread Remy Horton
This patch adds a new information metric library that allows other modules to register named metrics and update their values. It is intended to be independent of ethdev, rather than mixing ethdev and non-ethdev information in xstats. Signed-off-by: Remy Horton --- MAINTAINERS

[dpdk-dev] [PATCH v9 2/7] app/proc_info: add metrics displaying

2017-01-18 Thread Remy Horton
From: Reshma Pattan Modify the dpdk-procinfo process to display the newly added metrics. Added new command line option "--metrics" to display metrics. Signed-off-by: Reshma Pattan Signed-off-by: Remy Horton --- app/proc_info/main.c | 79 +++- 1

[dpdk-dev] [PATCH v9 0/7] Expanded statistics reporting

2017-01-18 Thread Remy Horton
This patchset extends statistics reporting to include peak and average data-rate metrics. It comes in two parts: a statistics reporting library, and a bitrate calculation library that uses it. This structure is intended to seperate statistic reporting from ethdev and allow more flexible metric regi

[dpdk-dev] [PATCH] cryptodev: enable BPI for Cablelabs DOCSIS security spec

2017-01-18 Thread Fiona Trahe
Extend the DPDK cryptodev API to enable processing of packets according to the Baseline Privacy Interface Plus (BPI+) Specification described in the security specification of the Cablelabs Data-over-Cable Service Interface Specification (DOCSIS). Brief summary of BPI+ symmetric cryptography requi

[dpdk-dev] [PATCH v5 7/7] app/test: add ARMv8 crypto tests and test vectors

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Introduce unit tests for ARMv8 crypto PMD. Add test vectors for short cases such as 160 bytes. These test cases are ARMv8 specific since the code provides different processing paths for different input data sizes. User can validate correctness of algorithms' implementation u

[dpdk-dev] [PATCH v5 6/7] MAINTAINERS: update MAINTAINERS entry for ARMv8 crypto

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9645c9b..00c7adc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -447,6 +447,12 @@ M: Declan Doherty F: drivers/

[dpdk-dev] [PATCH v5 5/7] crypto/armv8: enable ARMv8 PMD in the configuration

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO option to the common configuration file. Don't enable it by default for ARM64 as it requires external library to build. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- config/common_base | 6 ++ 1 file changed, 6 inser

[dpdk-dev] [PATCH v5 4/7] doc: update documentation about ARMv8 crypto PMD

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add documentation about the driver and update release notes. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- doc/guides/cryptodevs/armv8.rst| 96 ++ doc/guides/cryptodevs/index.rst| 1 + doc/guides/rel_notes/rele

[dpdk-dev] [PATCH v5 3/7] mk: add PMD to the build system

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Build ARMv8 crypto PMD if compiling for ARM64 and CONFIG_RTE_LIBRTE_PMD_ARMV8_CRYPTO option is enable in the configuration file. ARMV8_CRYPTO_LIB_PATH environment variable will point to the appropriate library directory. Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jaco

[dpdk-dev] [PATCH v5 2/7] crypto/armv8: add PMD optimized for ARMv8 processors

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek This patch introduces crypto poll mode driver using ARMv8 cryptographic extensions. CPU compatibility with this driver is detected in run-time and virtual crypto device will not be created if CPU doesn't provide: AES, SHA1, SHA2 and NEON. This PMD is optimized to provide per

[dpdk-dev] [PATCH v5 1/7] cryptodev: add cryptodev type for the ARMv8 PMD

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Add type and name for ARMv8 crypto PMD Signed-off-by: Zbigniew Bodek Reviewed-by: Jerin Jacob --- lib/librte_cryptodev/rte_cryptodev.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/librte_cryptodev/rte_cryptodev.h b/lib/librte_cryptodev/rte_cryptodev.h index

[dpdk-dev] [PATCH v5 0/7] Add crypto PMD optimized for ARMv8

2017-01-18 Thread zbigniew.bodek
From: Zbigniew Bodek Introduce crypto poll mode driver using ARMv8 cryptographic extensions. This PMD is optimized to provide performance boost for chained crypto operations processing, such as: * encryption + HMAC generation * decryption + HMAC validation. In particular, cipher only or hash only

[dpdk-dev] [PATCH v10 13/13] doc: remove deprecation notice for rte_bus

2017-01-18 Thread Shreyansh Jain
Signed-off-by: Shreyansh Jain --- doc/guides/rel_notes/deprecation.rst | 5 - 1 file changed, 5 deletions(-) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 291e03d..60d2bad 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel

[dpdk-dev] [PATCH v10 11/13] eal: enable PCI bus

2017-01-18 Thread Shreyansh Jain
Remove EAL initiated direct PCI scan/probe and enable PCI Bus linkage. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- lib/librte_eal/bsdapp/eal/eal.c | 7 - lib/librte_eal/bsdapp/eal/eal_pci.c | 4 +++ lib/librte_eal/bsdapp/eal/rte_eal_version.map

[dpdk-dev] [PATCH v10 12/13] eal: enable hotplugging of devices on bus

2017-01-18 Thread Shreyansh Jain
Given a bus, attach and detach callbacks allow the implementation to handles calls from EAL for attaching or detaching a named device. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- lib/librte_eal/bsdapp/eal/eal_pci.c | 2 + lib/librte_eal/common/eal_common_dev.c | 56 +++

[dpdk-dev] [PATCH v10 10/13] test: add Bus based scan and probe test cases for PCI

2017-01-18 Thread Shreyansh Jain
Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- app/test/test_pci.c | 164 1 file changed, 114 insertions(+), 50 deletions(-) diff --git a/app/test/test_pci.c b/app/test/test_pci.c index cda186d..09261cc 100644 --- a/app/test/test

[dpdk-dev] [PATCH v10 09/13] test: add test cases for scan and probe on BUS

2017-01-18 Thread Shreyansh Jain
Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- app/test/test_bus.c | 152 1 file changed, 152 insertions(+) diff --git a/app/test/test_bus.c b/app/test/test_bus.c index 0b6d011..ef7fa89 100644 --- a/app/test/test_bus.c +++ b/app/

[dpdk-dev] [PATCH v10 08/13] eal/pci: add support for PCI bus

2017-01-18 Thread Shreyansh Jain
Based on EAL Bus APIs, PCI bus callbacks and support functions are introduced in this patch. EAL continues to have direct PCI init/scan calls as well. These would be removed in subsequent patches to enable bus only PCI devices. Signed-off-by: Shreyansh Jain Reviewed-by: Ferruh Yigit --- lib/li

  1   2   >