Re: [dpdk-dev] [PATCH] maintainers: handoff ownership of Tap PMD

2017-03-30 Thread Pascal Mazon
On Thu, 30 Mar 2017 11:05:31 -0500 Keith Wiles wrote: > Pascal has added many features to the Tap PMD and the code > is now mostly his code. We talked and he suggested I send > the patch to change ownership. > > Signed-off-by: Keith Wiles > --- > MAINTAINERS | 2 +- > 1 file changed, 1 inserti

Re: [dpdk-dev] [PATCH v3 1/3] mempool: fix segfault when shared mempool handler not linked

2017-03-30 Thread Shreyansh Jain
On Friday 31 March 2017 11:05 AM, Shreyansh Jain wrote: Fixes: 449c49b93a6b ("mempool: support handler operations") In case the stack or ring mempool handler are compiled as shared library and not linked in with test binary, segfault is reported. This is because return value of rte_mempool_set_o

[dpdk-dev] [PATCH v3 3/3] mempool: introduce stack mempool driver

2017-03-30 Thread Shreyansh Jain
Moved from lib/librte_mempool, stack mempool handler is an independent driver. Shared builds would now require to link in librte_mempool_stack for "stack" mempool handler. Signed-off-by: Shreyansh Jain --- v3: * fix shared build issue MAINTAINERS| 1 +

[dpdk-dev] [PATCH v3 2/3] mempool: introduce ring mempool driver

2017-03-30 Thread Shreyansh Jain
Moved from lib/librte_mempool, ring mempool is now an independent driver. Shared builds would now need to add librte_mempool_ring for: * ring_mp_mc * ring_sp_sc * ring_sp_mc * ring_mp_sc Signed-off-by: Shreyansh Jain --- v3: * Update MAINTAINER file * fix shared build issue MAINTAINERS

[dpdk-dev] [PATCH v3 1/3] mempool: fix segfault when shared mempool handler not linked

2017-03-30 Thread Shreyansh Jain
Fixes: 449c49b93a6b ("mempool: support handler operations") In case the stack or ring mempool handler are compiled as shared library and not linked in with test binary, segfault is reported. This is because return value of rte_mempool_set_ops_byname is not being checked in rte_mempool_ops_alloc.

Re: [dpdk-dev] [PATCH 1/3] mempool: fix segfault for unlinked mempool handler

2017-03-30 Thread Shreyansh Jain
On Friday 31 March 2017 10:59 AM, Shreyansh Jain wrote: Fixes: 449c49b93a6b ("mempool: support handler operations") In case the stack or ring mempool handler are compiled as shared library and not linked in with test binary, segfault is reported. This is because return value of rte_mempool_set_o

[dpdk-dev] [PATCH 3/3] mempool: introduce stack mempool driver

2017-03-30 Thread Shreyansh Jain
Moved from lib/librte_mempool, stack mempool handler is an independent driver. Shared builds would now require to link in librte_mempool_stack for "stack" mempool handler. Signed-off-by: Shreyansh Jain --- MAINTAINERS| 1 + config/common_base

[dpdk-dev] [PATCH 1/3] mempool: fix segfault for unlinked mempool handler

2017-03-30 Thread Shreyansh Jain
Fixes: 449c49b93a6b ("mempool: support handler operations") In case the stack or ring mempool handler are compiled as shared library and not linked in with test binary, segfault is reported. This is because return value of rte_mempool_set_ops_byname is not being checked in rte_mempool_ops_alloc.

[dpdk-dev] [PATCH 2/3] mempool: introduce ring mempool driver

2017-03-30 Thread Shreyansh Jain
Moved from lib/librte_mempool, ring mempool is now an independent driver. Shared builds would now need to add librte_mempool_ring for: * ring_mp_mc * ring_sp_sc * ring_sp_mc * ring_mp_sc Signed-off-by: Shreyansh Jain --- MAINTAINERS| 1 + config/common_ba

[dpdk-dev] [PATCH v2 5/7] net/enic: flow API for Legacy NICs

2017-03-30 Thread John Daley
5-tuple exact Flow support for 1200 series adapters. This allows: Attributes: ingress Items: ipv4, ipv6, udp, tcp (must exactly match src/dst IP addresses and ports and all must be specified). Actions: queue and void Selectors: 'is' Signed-off-by: John Daley Reviewed-by: Nelson Escobar --

[dpdk-dev] [PATCH v2 1/7] net/enic: bring NIC interface functions up to date

2017-03-30 Thread John Daley
Update the base functions for the Cisco VIC. These files are mostly common with other VIC drivers so are left alone is as much as possilbe. Includes in a new filter/action interface which is needed for Generic Flow API PMD support. Update FDIR code to use the new interface. Signed-off-by: John Dal

[dpdk-dev] [PATCH v2 3/7] net/enic: flow API for NICs with advanced filters enabled

2017-03-30 Thread John Daley
Flow support for 1300 series adapters with the 'Advanced Filter' mode enabled via the UCS management interface. This enables: Attributes: ingress Items: eth, ipv4, ipv6, udp, tcp, vxlan, inner eth (full hdr masking) Actions: queue, mark, flag and void Selectors: 'is', 'spec' and 'mask'. 'last' is n

[dpdk-dev] [PATCH v2 7/7] net/enic: flow API documentation

2017-03-30 Thread John Daley
Update enic NIC guide, release notes and add flow API to the supported features list. Signed-off-by: John Daley --- doc/guides/nics/enic.rst | 44 ++ doc/guides/nics/features/enic.ini | 1 + doc/guides/rel_notes/release_17_05.rst | 6 + 3

[dpdk-dev] [PATCH v2 6/7] net/enic: flow API debug

2017-03-30 Thread John Daley
Added a debug function to print enic filters and actions when rte_validate_flow is called. Compiled in CONFIG_RTE_LIBRTE_ENIC_DEBUG_FLOW is enabled and log level is INFO. Signed-off-by: John Daley Reviewed-by: Nelson Escobar --- drivers/net/enic/enic_flow.c | 138 +++

[dpdk-dev] [PATCH v2 4/7] net/enic: flow API for NICs with advanced filters disabled

2017-03-30 Thread John Daley
Flow support for 1300 series adapters with the 'Advanced Filter' mode disabled via the UCS management interface. This allows: Attributes: ingress Items: eth, ipv4, ipv6, udp, tcp, vxlan, inner eth (full hdr masking) Actions: queue and void Selectors: 'is', 'spec' and 'mask'. 'last' is not supported

[dpdk-dev] [PATCH v2 2/7] net/enic: flow API skeleton

2017-03-30 Thread John Daley
Stub callbacks for the generic flow API and a new FLOW debug define. Signed-off-by: John Daley Reviewed-by: Nelson Escobar --- config/common_base | 1 + drivers/net/enic/Makefile | 1 + drivers/net/enic/enic.h| 1 + drivers/net/enic/enic_ethdev.c | 21 +- dri

[dpdk-dev] [PATCH v2 0/7] *** flow API support for enic ***

2017-03-30 Thread John Daley
*** flow API support for enic *** Version 2 fixes some checkpatch fails and adds documentaion. My checkpatch didn't catch typos and I missed a couple valid errors. There are still some warnings: 1 for returning non-negative errnos from a function which I'm OK with this because the caller uses them

[dpdk-dev] [PATCH] net/i40e: fix a doxygen warning

2017-03-30 Thread Beilei Xing
Fix the doxygen warning by adding comments for enumeration. Fixes: fad01fd77ac3 ("net/i40e: add dynamic device personalization processing") Signed-off-by: Beilei Xing --- drivers/net/i40e/rte_pmd_i40e.h | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/net/i

Re: [dpdk-dev] [PATCH] net/fm10k: fix secondary process crash

2017-03-30 Thread Chen, Jing D
> -Original Message- > From: Wang, Xiao W > Sent: Tuesday, March 28, 2017 11:59 AM > To: Chen, Jing D > Cc: dev@dpdk.org; Wang, Xiao W ; sta...@dpdk.org > Subject: [PATCH] net/fm10k: fix secondary process crash > > If the primary process has initialized all the queues to vector pmd mode

[dpdk-dev] [PATCH v3 2/2] app/test: update test code

2017-03-30 Thread Qi Zhang
Update the test code to algin with callback function change. Signed-off-by: Qi Zhang --- app/test/test_interrupts.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/app/test/test_interrupts.c b/app/test/test_interrupts.c index 371101f..5377292 100644

[dpdk-dev] [PATCH v3 1/2] eal: clean up interrupt handle

2017-03-30 Thread Qi Zhang
The patch change the prototype of callback function (rte_intr_callback_fn) by removing the unnecessary parameter. v3: - update bnx2x driver which is missed in v1,v2. Signed-off-by: Qi Zhang --- drivers/net/bnx2x/bnx2x_ethdev.c | 5 +++-- drivers/net/bnxt/bnxt_irq.c

[dpdk-dev] [PATCH v3 0/2] clean up interrupt handle

2017-03-30 Thread Qi Zhang
It seems its not necessary to register an intr_handle for interrupt callback function. "void* cb_arg" shows enough when be used to pass the object that contain the information be required to handle the interrupt event( A typical way which is implemented by almost all driver is by passing a rte_et

Re: [dpdk-dev] [PATCH 0/9] mbuf: structure reorganization

2017-03-30 Thread Ananyev, Konstantin
> > > > > > > > > -Original Message- > > > From: Richardson, Bruce > > > Sent: Thursday, March 30, 2017 1:23 PM > > > To: Olivier Matz > > > Cc: dev@dpdk.org; Ananyev, Konstantin ; > > > m...@smartsharesystems.com; Chilikin, Andrey > > > ; jblu...@infradead.org; > > > nelio.laranje...@

Re: [dpdk-dev] [PATCH v4] net/ixgbe: ping VF when PF status changes

2017-03-30 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Dai, Wei > Sent: Friday, March 31, 2017 8:48 AM > To: al...@att.com; dev@dpdk.org > Cc: Ananyev, Konstantin; Zhang, Helin; Lu, Wenzhuo; Iremonger, Bernard > Subject: RE: [PATCH v4] net/ixgbe: ping VF when PF status changes > > Hi, Alexz > Thanks for your

Re: [dpdk-dev] [PATCH v5 3/5] net/i40e: parse QinQ pattern

2017-03-30 Thread Lu, Wenzhuo
Hi Bernard, > -Original Message- > From: Iremonger, Bernard > Sent: Friday, March 31, 2017 12:10 AM > To: dev@dpdk.org; Xing, Beilei; Wu, Jingjing > Cc: Zhang, Helin; Lu, Wenzhuo; Iremonger, Bernard > Subject: [PATCH v5 3/5] net/i40e: parse QinQ pattern > > add QinQ pattern. > add i40e_fl

Re: [dpdk-dev] [PATCH v2 2/3] eal: clean up interrupt handle

2017-03-30 Thread Zhang, Qi Z
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Friday, March 31, 2017 12:40 AM > To: Zhang, Qi Z > Cc: dev@dpdk.org > Subject: Re: [PATCH v2 2/3] eal: clean up interrupt handle > > 2017-03-07 04:19, Qi Zhang: > > The patch change the prototype of

Re: [dpdk-dev] [PATCH v4] net/ixgbe: ping VF when PF status changes

2017-03-30 Thread Dai, Wei
Hi, Alexz Thanks for your quick reponse. Hi, Wenzhuo Would you please review it again and add your acknowledgement if it is OK for you. > Subject: [PATCH v4] net/ixgbe: ping VF when PF status changes > > From: Alex Zelezniak > > v4: > * Removed duplicate line > > v3: > * Added 17.05 block to

Re: [dpdk-dev] [PATCH v4 0/3] net/i40e: configurable PTYPE mapping

2017-03-30 Thread Wu, Jingjing
> -Original Message- > From: Zhang, Qi Z > Sent: Thursday, March 30, 2017 7:30 PM > To: Wu, Jingjing ; Zhang, Helin > > Cc: dev@dpdk.org; Zhang, Qi Z > Subject: [PATCH v4 0/3] net/i40e: configurable PTYPE mapping > > The patch set create new APIs that help to change the mapping from ha

[dpdk-dev] [PATCH v4] mempool: update non-EAL thread note in the header

2017-03-30 Thread Gage Eads
Commit 30e6399892276 ("mempool: support non-EAL thread") added the capability for non-EAL threads to use the mempool library. This commit removes the note indicating that the mempool library cannot be used safely by non-EAL threads, and replaces it with a more up-to-date note. Signed-off-by: Gage

[dpdk-dev] [PATCH v2 5/5] add support for new xstats API retrieving by id

2017-03-30 Thread Michal Jastrzebski
Add support for new xstats API retrieving by id in testpmd application Signed-off-by: Jacek Piasecki Signed-off-by: Kuba Kozak --- app/test-pmd/config.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index 804

[dpdk-dev] [PATCH v2 4/5] add support for new xstats API retrieving by id

2017-03-30 Thread Michal Jastrzebski
Add support for new xstats API retrieving by id to proc_info application Signed-off-by: Jacek Piasecki Signed-off-by: Kuba Kozak --- app/proc_info/main.c | 56 +--- 1 file changed, 45 insertions(+), 11 deletions(-) diff --git a/app/proc_info/main

[dpdk-dev] [PATCH v2 3/5] add new xstats API id support for ixgbe

2017-03-30 Thread Michal Jastrzebski
add new xstats API id support for ixgbe Signed-off-by: Jacek Piasecki Signed-off-by: Kuba Kozak --- drivers/net/ixgbe/ixgbe_ethdev.c | 178 +++ 1 file changed, 178 insertions(+) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c

Re: [dpdk-dev] [PATCH] crypto/qat: fix segmentation fault in session create

2017-03-30 Thread De Lara Guarch, Pablo
CC'ing stable repo. > -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, March 29, 2017 11:32 AM > To: dev@dpdk.org > Cc: Trahe, Fiona; De Lara Guarch, Pablo; Griffin, John; Jain, Deepak K; > Kusztal, ArkadiuszX > Subject: [PATCH] crypto/qat: fix segmentation fault in sessio

Re: [dpdk-dev] [PATCH] crypto/qat: fix segmentation fault in session create

2017-03-30 Thread De Lara Guarch, Pablo
> -Original Message- > From: Trahe, Fiona > Sent: Thursday, March 30, 2017 2:37 PM > To: Kusztal, ArkadiuszX; dev@dpdk.org > Cc: De Lara Guarch, Pablo; Griffin, John; Jain, Deepak K > Subject: RE: [PATCH] crypto/qat: fix segmentation fault in session create > > > > > -Original Mess

[dpdk-dev] [PATCH v2 2/5] add new xstats API id support for e1000

2017-03-30 Thread Michal Jastrzebski
Add new xstats API id support for e1000 Signed-off-by: Jacek Piasecki Signed-off-by: Kuba Kozak --- drivers/net/e1000/igb_ethdev.c | 92 +- 1 file changed, 90 insertions(+), 2 deletions(-) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/i

[dpdk-dev] [PATCH v2 1/5] add new xstats API retrieving by id

2017-03-30 Thread Michal Jastrzebski
Extended xstats API in ethdev library to allow grouping of stats logically so they can be retrieved per logical grouping – managed by the application. Changed existing functions rte_eth_xstats_get_names and rte_eth_xstats_get to use a new list of arguments: array of ids and array of values. ABI ver

[dpdk-dev] [PATCH v2 0/5] Extended xstats API in ethdev library to allow grouping of stats

2017-03-30 Thread Michal Jastrzebski
Extended xstats API in ethdev library to allow grouping of stats logically so they can be retrieved per logical grouping – managed by the application. Changed existing functions rte_eth_xstats_get_names and rte_eth_xstats_get to use a new list of arguments: array of ids and array of values. ABI ver

Re: [dpdk-dev] [PATCH v2 1/3] crypto/qat: refactor capabilities infrastructure

2017-03-30 Thread De Lara Guarch, Pablo
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, March 29, 2017 2:04 PM > To: dev@dpdk.org > Cc: Trahe, Fiona; De Lara Guarch, Pablo; Griffin, John; Jain, Deepak K; > Kusztal, ArkadiuszX > Subject: [PATCH v2 1/3] crypto/qat: refactor capabilities infrastructure > > Ref

Re: [dpdk-dev] [PATCH v2 3/3] test: add ZUC test cases for QAT

2017-03-30 Thread De Lara Guarch, Pablo
Hi Arek, > -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, March 29, 2017 2:04 PM > To: dev@dpdk.org > Cc: Trahe, Fiona; De Lara Guarch, Pablo; Griffin, John; Jain, Deepak K; > Kusztal, ArkadiuszX > Subject: [PATCH v2 3/3] test: add ZUC test cases for QAT > > This patch

[dpdk-dev] [PATCH 6/6] net/enic: flow API debug

2017-03-30 Thread John Daley
Added a debug function to print enic filters and actions when rte_validate_flow is called. Compiled in CONFIG_RTE_LIBRTE_ENIC_DEBUG_FLOW is enabled and log level is INFO. Signed-off-by: John Daley Reviewed-by: Nelson Escobar --- drivers/net/enic/enic_flow.c | 138 +++

[dpdk-dev] [PATCH 5/6] net/enic: flow API support for Legacy NICs

2017-03-30 Thread John Daley
5-tuple exact Flow support for 1200 series adapters. This allows: Attributes: ingress Items: ipv4, ipv6, udp, tcp (must exactly match src/dst IP addresses and ports and all must be specified). Actions: queue and void Selectors: 'is' Signed-off-by: John Daley Reviewed-by: Nelson Escobar --

[dpdk-dev] [PATCH 4/6] net/enic: flow API for NICs with advanced filters disabled

2017-03-30 Thread John Daley
Flow support for 1300 series adapters with the 'Advanced Filter' mode disabled via the UCS management interface. This allows: Attributes: ingress Items: eth, ipv4, ipv6, udp, tcp, vxlan, inner eth (full hdr masking) Actions: queue and void Selectors: 'is', 'spec' and 'mask'. 'last' is not supported

[dpdk-dev] [PATCH 3/6] net/enic: flow API for NICs with advanced filters enabled

2017-03-30 Thread John Daley
Flow support for 1300 series adapters with the 'Advanced Filter' mode enabled via the UCS management interface. This enables: Attributes: ingress Items: eth, ipv4, ipv6, udp, tcp, vxlan, inner eth (full hdr masking) Actions: queue, mark, flag and void Selectors: 'is', 'spec' and 'mask'. 'last' is n

[dpdk-dev] [PATCH 2/6] net/enic: flow API skeleton

2017-03-30 Thread John Daley
Stub callbacks for the generic flow API and a new FLOW debug define. Signed-off-by: John Daley Reviewed-by: Nelson Escobar --- config/common_base | 1 + drivers/net/enic/Makefile | 1 + drivers/net/enic/enic.h| 1 + drivers/net/enic/enic_ethdev.c | 21 +- dri

[dpdk-dev] [PATCH 1/6] net/enic: bring NIC interface functions up to date

2017-03-30 Thread John Daley
Update the base functions for the Cisco VIC. These files are mostly common with other VIC drivers so are left alone is as much as possilbe. Includes in a new filter/action interface which is needed for Generic Flow API PMD support. Update FDIR code to use the new interface. Signed-off-by: John Dal

[dpdk-dev] [PATCH v4] net/ixgbe: ping VF when PF status changes

2017-03-30 Thread alexz
From: Alex Zelezniak v4: * Removed duplicate line v3: * Added 17.05 block to rte_pmd_ixgbe_version.map file v2: * Removed trailing spaces v1: * Implements function used by application managing PF to inform VF when link status changes Signed-off-by: Alex Zelezniak --- drivers/net/ixgbe/ixgbe

[dpdk-dev] [PATCH v14 6/6] app/test-pmd: add latency statistics calculation

2017-03-30 Thread Remy Horton
From: Reshma Pattan 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 | 22 +-

[dpdk-dev] [PATCH v14 3/6] lib: add bitrate statistics library

2017-03-30 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 v14 5/6] lib: added new library for latency stats

2017-03-30 Thread Remy Horton
From: Reshma Pattan 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 stats

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

2017-03-30 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, 39 insertions(+), 2 deletions(-) diff --git a/app/test-pmd/te

[dpdk-dev] [PATCH v14 2/6] app/proc_info: add metrics displaying

2017-03-30 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 v14 1/6] lib: add information metrics library

2017-03-30 Thread Remy Horton
This patch adds a new information metrics library. This Metrics library implements a mechanism by which producers can publish numeric information for later querying by consumers. Metrics themselves are statistics that are not generated by PMDs, and hence are not reported via ethdev extended statist

[dpdk-dev] [PATCH v14 0/6] Expanded statistics reporting

2017-03-30 Thread Remy Horton
This patchset consists of three libraries: A Metrics library for distributing device information, a library that calculates bit-rate statistics, and a library that calculates latency statistics. The latter two libraries make use of the first library. Metrics Library --- The Metrics lib

Re: [dpdk-dev] [PATCH v2 4/4] devtools: add autotest app to build script

2017-03-30 Thread Thomas Monjalon
For info, Ferruh did a similar patch: http://dpdk.org/patch/22443 Ferruh, OK to keep Bruce one? 2017-03-30 16:33, Bruce Richardson: > On Thu, Mar 30, 2017 at 04:36:11PM +0200, Thomas Monjalon wrote: > > 2017-03-29 17:38, Bruce Richardson: > > > Since the test app is no longer being build b

Re: [dpdk-dev] [PATCH v2 3/4] devtools: remove special case build for perf thread example

2017-03-30 Thread Thomas Monjalon
2017-03-30 16:30, Bruce Richardson: > On Thu, Mar 30, 2017 at 04:31:45PM +0200, Thomas Monjalon wrote: > > 2017-03-29 17:38, Bruce Richardson: > > > Since performance-thread example app is now built as part of make examples > > > we don't need a special case line for it in testbuild.sh script > > [

Re: [dpdk-dev] [PATCH] mk: use icc default inline limit

2017-03-30 Thread Thomas Monjalon
2017-03-27 15:35, Ferruh Yigit: > ICC build time > Before this patch (bnx2x PMD enabled [1]) > real8m16.622s > > After this patch (bnx2x enabled) > real0m35.140s > > [1] > bnx2x cause the build take a lot, otherwise build times are more sane > numbers. > > ICC has a default inline limit

Re: [dpdk-dev] [PATCH] igb_uio: use non-threaded ISR

2017-03-30 Thread Thomas Monjalon
2017-02-27 16:15, Ferruh Yigit: > On 1/20/2017 11:08 PM, David Su wrote: > > This eliminates the overhead of a task switch when an interrupt arrives. > > > > Signed-off-by: David Su > > Overall I agree with Stephen to switch vfio when possible but for the > cases igb_uio still required: > > Ack

Re: [dpdk-dev] [PATCH] igb_uio: stop device when closing /dev/uioX

2017-03-30 Thread Thomas Monjalon
2016-12-02 16:45, Jianfeng Tan: > Depends-on: http://dpdk.org/dev/patchwork/patch/17493/ The above patch is marked as rejected. Do we want to reject also this patch? > When a DPDK application grab a PCI device, device and driver work > together to Rx/Tx packets. If the DPDK app crashes or gets ki

Re: [dpdk-dev] [PATCH] igb_uio: map dummy dma forcing iommu domain attachment

2017-03-30 Thread Thomas Monjalon
2017-02-17 12:29, Ferruh Yigit: > On 1/18/2017 12:27 PM, Alejandro Lucero wrote: > > For using a DPDK app when iommu is enabled, it requires to > > add iommu=pt to the kernel command line. But using igb_uio driver > > makes DMAR errors because the device has not an IOMMU domain. > > > > Since kern

Re: [dpdk-dev] Segm fault to the update acl context

2017-03-30 Thread Victor Detoni
Hi, I have tried to use rcu and rte_lock libs to synchronize but I put the ctx_reload before rte_acl_classify function and it is working fine. My code is working like this and I'm not having latency during reload process. while() { ... if (acl_ctx_reload) do_acl_reload(); if (acl_search.n

Re: [dpdk-dev] [PATCH v4 1/3] lib/librte_ether: add support for port reset

2017-03-30 Thread Thomas Monjalon
Hi, Please help reviewers, use --in-reply-to to keep patches threaded. 2017-03-30 17:34, Wei Zhao: > Add support for port reset in rte layer.This reset > feature can not only used in vf port reset in later code > develop, but alsopf port.But in this patch set, we only > limit the discussion scope

Re: [dpdk-dev] [PATCH] rename Cesnet Combo cards to Netcope NFB, NPC and NSF

2017-03-30 Thread Thomas Monjalon
2017-03-30 08:49, Viktor Pus: > This is to reflect that the cards are available from Netcope, not Cesnet. > > Signed-off-by: Viktor Pus > --- > doc/nics.html | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Wrong mailing list. Please send it to w...@dpdk.org

Re: [dpdk-dev] [PATCH v2 1/1] pci: default to whitelist mode

2017-03-30 Thread Thomas Monjalon
2017-03-28 13:02, Van Haaren, Harry: > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Gaëtan Rivet > > On Tue, Mar 28, 2017 at 01:20:00PM +0100, Bruce Richardson wrote: > > >On Tue, Mar 28, 2017 at 02:01:29PM +0200, Gaetan Rivet wrote: > > >> Expects all devices to be explicitly defined bef

[dpdk-dev] [PATCH v7 22/22] maintainers: add eventdev section and claim SW PMD

2017-03-30 Thread Harry van Haaren
Add a section for the eventdev PMDs, and note the next-tree. Claim maintainership of the software eventdev PMD. Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 711fbfb..55ca3f0 10

[dpdk-dev] [PATCH v7 20/22] doc: add event device and software eventdev

2017-03-30 Thread Harry van Haaren
This commit adds a section to the docs listing the event device PMDs available. It then adds the software eventdev PMD to the listed event devices. Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- v7: - Naming Consistency of EventDev, Eventdev and eventdev (Anatoly) - Reword "root ca

[dpdk-dev] [PATCH v7 21/22] doc: add SW eventdev PMD to 17.05 release notes

2017-03-30 Thread Harry van Haaren
Signed-off-by: Harry van Haaren --- doc/guides/rel_notes/release_17_05.rst | 7 +++ 1 file changed, 7 insertions(+) diff --git a/doc/guides/rel_notes/release_17_05.rst b/doc/guides/rel_notes/release_17_05.rst index 918f483..a5b8351 100644 --- a/doc/guides/rel_notes/release_17_05.rst +++ b/d

[dpdk-dev] [PATCH v7 19/22] test/eventdev: add SW deadlock tests

2017-03-30 Thread Harry van Haaren
This commit adds the worker loopback test to verify that the deadlock avoidance scheme is functioning, and a holb (head-of-line-blocking) test to ensure the head of line blocking avoidance is correct. Signed-off-by: Bruce Richardson Signed-off-by: David Hunt Signed-off-by: Harry van Haaren Ack

[dpdk-dev] [PATCH v7 18/22] test/eventdev: add SW xstats tests

2017-03-30 Thread Harry van Haaren
This commit introduces xstats tests for statistics and reset functionality. Signed-off-by: Bruce Richardson Signed-off-by: David Hunt Signed-off-by: Harry van Haaren Acked-by: Anatoly Burakov --- test/test/test_eventdev_sw.c | 806 +++ 1 file changed,

[dpdk-dev] [PATCH v7 16/22] test/eventdev: add basic SW tests

2017-03-30 Thread Harry van Haaren
This commit adds basic enqueue and dequeue unit tests, some negative invalid tests, and configuration. Signed-off-by: Bruce Richardson Signed-off-by: David Hunt Signed-off-by: Harry van Haaren Acked-by: Anatoly Burakov --- test/test/test_eventdev_sw.c | 1060 +

[dpdk-dev] [PATCH v7 17/22] test/eventdev: add SW tests for load balancing

2017-03-30 Thread Harry van Haaren
This commit adds various tests for load-balancing and queue prioritization. Signed-off-by: Bruce Richardson Signed-off-by: David Hunt Signed-off-by: Harry van Haaren Acked-by: Anatoly Burakov --- test/test/test_eventdev_sw.c | 566 +++ 1 file changed,

[dpdk-dev] [PATCH v7 15/22] test/eventdev: add SW test infrastructure

2017-03-30 Thread Harry van Haaren
Add the test infrastructure, create and destroy the test instance. Signed-off-by: Bruce Richardson Signed-off-by: David Hunt Signed-off-by: Harry van Haaren Acked-by: Anatoly Burakov --- test/test/Makefile | 5 +- test/test/autotest_data.py | 26 test/test/test_eventdev_s

[dpdk-dev] [PATCH v7 14/22] event/sw: add xstats support

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Add support for xstats to report out on the state of the eventdev. Useful for debugging and for unit tests, as well as observability at runtime and performance tuning of apps to work well with the scheduler. Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren

[dpdk-dev] [PATCH v7 13/22] event/sw: add dump function for easier debugging

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Segfault issue resolved when only partially configured and rte_event_dev_dump() is called before start(), Reported-by: Vipin Varghese Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: David Hunt --- drivers/event/sw/sw_evdev.c | 148 ++

[dpdk-dev] [PATCH v7 12/22] event/sw: add start stop and close functions

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- v7: - Removed __func__ and __LINE__ from SW_LOG_ERR (Jerin) - Updated start() error codes (Jerin) v6: - Removed printf() using SW_LOG_ERR instead (Jerin) - Added rte_smp_wmb() t

[dpdk-dev] [PATCH v7 11/22] event/sw: add scheduling logic

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Add in the scheduling function which takes the events from the producer queues and buffers them before scheduling them to consumer queues. The scheduling logic includes support for atomic, reordered, and parallel scheduling of flows. Signed-off-by: Bruce Richardson Signed

[dpdk-dev] [PATCH v7 09/22] event/sw: add support for linking queues to ports

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- v6: - Remove (void *) casts that are not required (Jerin) - Set rte_errno as appropriate if port link not established (Jerin) --- drivers/event/sw/sw_evdev.c | 85 ++

[dpdk-dev] [PATCH v7 10/22] event/sw: add worker core functions

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson add the event enqueue, dequeue and release functions to the eventdev. These also include tracking of stats for observability in the load of the scheduler. Internally in the enqueue function, the various types of enqueue operations, to forward an existing event, to send a ne

[dpdk-dev] [PATCH v7 08/22] event/sw: add support for event ports

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Add in the data-structures for the ports used by workers to send packets to/from the scheduler. Also add in the functions to create/destroy those ports. Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- v7: - Removed __func__ a

[dpdk-dev] [PATCH v7 07/22] event/sw: add support for event queues

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Add in the data structures for the event queues, and the eventdev functions to create and destroy those queues. Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- drivers/event/sw/iq_ring.h | 176

[dpdk-dev] [PATCH v7 05/22] event/sw: add configure function

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- v6: - Add check for per dequeue timeout config, return -ENOTSUP if so (Jerin) --- drivers/event/sw/sw_evdev.c | 18 ++ drivers/event/sw/sw_evdev.h | 11 +

[dpdk-dev] [PATCH v7 06/22] event/sw: add fns to return default port/queue config

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Signed-off-by: Bruce Richardson Acked-by: Jerin Jacob --- drivers/event/sw/sw_evdev.c | 32 1 file changed, 32 insertions(+) diff --git a/drivers/event/sw/sw_evdev.c b/drivers/event/sw/sw_evdev.c index 7166ef5..c0ec24c 100644 --- a/driv

[dpdk-dev] [PATCH v7 04/22] event/sw: add device capabilities function

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson Add in the info_get function to return details on the queues, flow, prioritization capabilities, etc. that this device has. Signed-off-by: Bruce Richardson Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob --- drivers/event/sw/sw_evdev.c | 23 +

[dpdk-dev] [PATCH v7 03/22] event/sw: add new software-only eventdev driver

2017-03-30 Thread Harry van Haaren
From: Bruce Richardson This adds the minimal changes to allow a SW eventdev implementation to be compiled, linked and created at run time. The eventdev does nothing, but can be created via vdev on commandline, e.g. sudo ./x86_64-native-linuxapp-gcc/app/test --vdev=event_sw0 ... PMD: Creati

[dpdk-dev] [PATCH v7 01/22] eventdev: improve API docs for start function

2017-03-30 Thread Harry van Haaren
This commit documents two error return values for the rte_event_dev_start() function. -ESTALE indicates not all ports are configured -ENOLINK indicates that not all queues are linked to ports. If an application enqueues to such a queue it can lead to deadlock Suggested-by: Jerin Jacob

[dpdk-dev] [PATCH v7 00/22] next-eventdev: event/sw software eventdev

2017-03-30 Thread Harry van Haaren
This is the v7 patchset of the software eventdev PMD. Changes include the following, see patch for context and details; - Docs patch now consistently uses eventdev (Anatoly) - Reworded "root cause" to "reason" for readability (Anatoly) - Removed __func__ and __LINE__ from SW_LOG_ERR (Jerin) - Fixe

[dpdk-dev] [PATCH v7 02/22] test/eventdev: pass timeout ticks unsupported

2017-03-30 Thread Harry van Haaren
This commit reworks the return value handling of the timeout ticks test. This feature is not mandatory for a pmd, the eventdev layer returns -ENOTSUP if the PMD doesn't implement the function. The test is modified to check if the return value is -ENOTSUP, and return -ENOTSUP to the test framework,

Re: [dpdk-dev] [PATCH 3/4] ethdev: count devices consistently

2017-03-30 Thread Thomas Monjalon
2017-03-03 16:40, Gaetan Rivet: > Make the rte_eth_dev_count() return the correct number of devices even > after some are detached by the hotplug API. Please explain what is the correct number, or that the wrong number was a max id. > This change does not affect existing applications that do not

[dpdk-dev] [PATCH v4 6/6] cfgfile: add support for empty value string

2017-03-30 Thread Allain Legacy
This commit adds support to the cfgfile library for parsing a key=value line that has no value string specified (e.g., "key="). This can be used to override a configuration attribute that has a default value or default list of values to set it back to an undefined value to disable functionality.

[dpdk-dev] [PATCH v4 5/6] cfgfile: increase local buffer size for max name and value

2017-03-30 Thread Allain Legacy
From: Joseph Richard When parsing a ini file with a "key = value" line that has both "key" and "value" sized to the maximum allowed length causes a parsing failure. The internal "buffer" variable should be sized at least as large as the maximum for both fields. This commit updates the local arr

[dpdk-dev] [PATCH v4 3/6] cfgfile: add support for configurable comment character

2017-03-30 Thread Allain Legacy
The current cfgfile comment character is hardcoded to ';'. This commit a new API to allow the user to specify which comment character to use while parsing the file. This is to ease adoption by applications that have an existing configuration file which may use a different comment character. For

[dpdk-dev] [PATCH v4 4/6] cfgfile: use strnlen to constrain memchr search

2017-03-30 Thread Allain Legacy
The call to memchr() uses the absolute length of the string buffer instead of the actual length of the string returned by fgets(). This causes the search to go beyond the '\n' character and find ';' characters in random garbage on the stack. This then causes the 'len' variable to be updated and t

[dpdk-dev] [PATCH v4 2/6] cfgfile: add support for global properties section

2017-03-30 Thread Allain Legacy
The current implementation of the cfgfile library requires that all key=value pairs be within [SECTION] definitions. The ini file standard allows for key=value pairs in an unnamed section. https://en.wikipedia.org/wiki/INI_file#Global_properties This commit adds the capability of parsing key=

[dpdk-dev] [PATCH v4 1/6] test: basic unit tests for cfgfile

2017-03-30 Thread Allain Legacy
This commit adds the basic infrastructure for the cfgfile library unit tests. It includes success path tests for the most commonly used APIs. More unit tests will be added later. Signed-off-by: Allain Legacy --- MAINTAINERS | 1 + test/test/Makefile

[dpdk-dev] [PATCH v4 0/6] librte_cfgfile enhancements

2017-03-30 Thread Allain Legacy
This patchset includes some minor enhancements that we have developped for our DPDK application. We would like to contribute them upstream to help ease adoption of the DPDK by anyone looking for this type of functionality. The commit logs on each patch should be self-sufficient in explaining the

Re: [dpdk-dev] [PATCH 0/4] crypto/qat: add DOCSISBPI mode support

2017-03-30 Thread De Lara Guarch, Pablo
> -Original Message- > From: Trahe, Fiona > Sent: Thursday, March 02, 2017 1:03 PM > To: dev@dpdk.org; De Lara Guarch, Pablo > Cc: Jain, Deepak K; Trahe, Fiona > Subject: [PATCH 0/4] crypto/qat: add DOCSISBPI mode support > > This patchset adds DOCSISBPI mode support to the > QuickAssist

Re: [dpdk-dev] [PATCH v4] crypto/openssl: add DES DOCSIS BPI support

2017-03-30 Thread De Lara Guarch, Pablo
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Thursday, March 30, 2017 6:27 PM > To: dev@dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [PATCH v4] crypto/openssl: add DES DOCSIS BPI support > > Adds support in OpenSSL PMD for algorithm following the DOCSIS > specification,

Re: [dpdk-dev] [PATCH v3] crypto/aesni_mb: add AES DOCSIS BPI support

2017-03-30 Thread De Lara Guarch, Pablo
> -Original Message- > From: Doherty, Declan > Sent: Thursday, March 30, 2017 5:35 PM > To: De Lara Guarch, Pablo > Cc: dev@dpdk.org > Subject: Re: [PATCH v3] crypto/aesni_mb: add AES DOCSIS BPI support > > On 30/03/2017 5:12 PM, Pablo de Lara wrote: > > Underlying IPSec Multi buffer lib

Re: [dpdk-dev] next technical board meeting, 2017-04-06

2017-03-30 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jay Rolette > Sent: Thursday, March 30, 2017 5:03 PM > To: Wiles, Keith > Cc: Olivier Matz ; Jerin Jacob > ; dev@dpdk.org; techbo...@dpdk.org > Subject: Re: [dpdk-dev] next technical board meeting, 2017-04-06 >

Re: [dpdk-dev] [PATCH v6 10/21] event/sw: add worker core functions

2017-03-30 Thread Jerin Jacob
On Thu, Mar 30, 2017 at 12:25:52AM +0100, Harry van Haaren wrote: > From: Bruce Richardson > > add the event enqueue, dequeue and release functions to the eventdev. > These also include tracking of stats for observability in the load of > the scheduler. > Internally in the enqueue function, the v

Re: [dpdk-dev] [PATCH v6 07/21] event/sw: add support for event queues

2017-03-30 Thread Jerin Jacob
On Thu, Mar 30, 2017 at 12:25:49AM +0100, Harry van Haaren wrote: > From: Bruce Richardson > > Add in the data structures for the event queues, and the eventdev > functions to create and destroy those queues. > > Signed-off-by: Bruce Richardson > Signed-off-by: Harry van Haaren Acked-by: Jeri

  1   2   3   >