[dpdk-dev] [PATCH] examples/vm_power_manager: fix build when libvirt version < 1.0

2015-12-07 Thread Thomas Monjalon
2015-12-07 15:24, Yong Liu: > virNodeGetCPUMap introduced in from libvirt 1.0. In some linux distributions > like Ubuntu12/14 and Fedora18, libvirt version is elder than 1.0. So this > sample will not build pass. > > Replace "virNodeGetCPUMap" with another libvirt API "virNodeGetInfo". > > Signed

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Thomas Monjalon
2015-12-03 16:05, Panu Matilainen: > In addition to git tags, support validating abi between any legal > gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " > "validate-abi.sh master mybranch " etc in addition to > validating between tags. Makes it easier to run the validator > for in-develop

[dpdk-dev] [PATCH v3 06/13] mk: split install rule

2015-12-07 Thread Arevalo, Mario Alfredo C
Hi Thomas, I'm testing this set of patches that has been applied, and I haven't noticed before that you exclude some binaries in "install-runtime" (test, testpipeline, testacl etc...), I would like to ask you the reason about this :) Thanks. Mario. From

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Thomas Monjalon
2015-12-07 14:58, Aaron Conole: > Panu Matilainen writes: > > Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but > > for qat and mpipe drivers. The former did not exist when the > > previous patch was sent and latter I just missed. > > > > Fixes: 5f9115e58cc6 ("mk: fix shared library

[dpdk-dev] [PATCH 1/1] kni: Fix compilation problems on RHEL 7.2.

2015-12-07 Thread Thomas Monjalon
2015-12-07 07:59, Lee Roberts: > RHEL 7.2 contains additional backports from newer upstream kernels. > Add RHEL_RELEASE_CODE logic for RHEL_RELEASE_VERSION(7,2) to pick up > the changes to kernel functions. > > Signed-off-by: Lee Roberts Applied, thanks If there is a new error in coming days du

[dpdk-dev] [PATCH] mk: fix warning spew when EXTRA_CFLAGS specifies warning flags

2015-12-07 Thread Thomas Monjalon
2015-12-07 14:49, Simon K?gstr?m: > On 2015-12-07 13:56, Panu Matilainen wrote: > > Starting with commit 9aa2053c6e81493b23346ff4e387903560de5c81 > > EXTRA_CFLAGS is sometimes being passed to the compiler without > > WERROR_FLAGS which can cause spurious warnings by the dozen, > > for example with

[dpdk-dev] [PATCH v3 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-07 Thread David Marchand
On Mon, Dec 7, 2015 at 7:36 PM, Kamil Rytarowski < Kamil.Rytarowski at caviumnetworks.com> wrote: > Currently rte_eal_check_module() detects Linux kernel modules via reading > /proc/modules. Built-in ones aren't listed there and therefore they are not > being found by the script. > > Add support f

[dpdk-dev] [PATCH 0/2] fix performance/cache resource issues with 128-byte cache line targets

2015-12-07 Thread Jerin Jacob
On Mon, Dec 07, 2015 at 03:40:13AM -0800, Thomas Monjalon wrote: > 2015-12-07 12:56, Jerin Jacob: > > On Sun, Dec 06, 2015 at 05:30:50PM +0100, Thomas Monjalon wrote: > > > 2015-12-06 21:29, Jerin Jacob: > > > > This patchset fixes performance/cache resource issues with 128-byte > > > > cache line

[dpdk-dev] [PATCH] config: cleanup existing RTE_CACHE_LINE_SIZE selection scheme

2015-12-07 Thread Jerin Jacob
by default, all the targets will be configured with the 64-byte cache line size, targets which have different cache line size can be overridden through target specific config file. Selected ThunderX and power8 as CONFIG_RTE_CACHE_LINE_SIZE=128 targets based on existing configuration. Signed-off-b

[dpdk-dev] [PATCH v3 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-07 Thread Kamil Rytarowski
Currently rte_eal_check_module() detects Linux kernel modules via reading /proc/modules. Built-in ones aren't listed there and therefore they are not being found by the script. Add support for checking built-in modules with parsing the sysfs files This commit obsoletes the /proc/modules parsing a

[dpdk-dev] [PATCH v3 1/2] tools: Add support for handling built-in kernel modules

2015-12-07 Thread Kamil Rytarowski
Currently dpdk_nic_bind.py detects Linux kernel modules via reading /proc/modules. Built-in ones aren't listed there and therefore they are not being found by the script. Add support for checking built-in modules with parsing the sysfs files. This commit obsoletes the /proc/modules parsing approa

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 03:55 PM, Thomas Monjalon wrote: > 2015-12-07 13:41, Panu Matilainen: >> On 12/07/2015 01:28 PM, Thomas Monjalon wrote: >>> 2015-12-07 08:29, Panu Matilainen: On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > 2015-12-02 15:53, Panu Matilainen: The vhost ABI break was anno

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Thomas Monjalon
2015-12-07 18:48, Panu Matilainen: > On 12/07/2015 03:55 PM, Thomas Monjalon wrote: > > 2015-12-07 13:41, Panu Matilainen: > >> On 12/07/2015 01:28 PM, Thomas Monjalon wrote: > >>> 2015-12-07 08:29, Panu Matilainen: > On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > > 2015-12-02 15:53, Pan

[dpdk-dev] [PATCH v2 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-07 Thread David Marchand
Hello Kamil, On Mon, Dec 7, 2015 at 5:57 PM, Kamil Rytarowski < Kamil.Rytarowski at caviumnetworks.com> wrote: > Currently rte_eal_check_module() detects Linux kernel modules via reading > /proc/modules. Built-in ones aren't listed there and therefore they are not > being found by the script. > >

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Panu Matilainen
On 12/07/2015 04:32 PM, Thomas Monjalon wrote: > 2015-12-07 16:09, Panu Matilainen: >> On 12/03/2015 04:05 PM, Panu Matilainen wrote: >>> In addition to git tags, support validating abi between any legal >>> gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " >>> "validate-abi.sh master mybra

[dpdk-dev] [PATCH v9 4/4] examples: add pthread_shim example to performance thread

2015-12-07 Thread Ian Betts
This commit adds an example that illustrates how to implement a pthread shim with the lthread subsystem included in the performance thread example application. Signed-off-by: Ian Betts --- examples/performance-thread/Makefile | 2 + examples/performance-thread/pthread_shim/Makefi

[dpdk-dev] [PATCH v9 3/4] examples: add l3fwd-thread example in performance-thread

2015-12-07 Thread Ian Betts
This commit adds an L3 forwarding application to the performace-thread example. Signed-off-by: Ian Betts --- MAINTAINERS |7 + examples/Makefile |6 +- examples/performance-thread/Makefile | 43 + exampl

[dpdk-dev] [PATCH v9 2/4] examples: add lthread subsystem for performance-thread

2015-12-07 Thread Ian Betts
This commit adds the lightweight thread subsystem used by the performance-thread sample applications. Signed-off-by: Ian Betts --- examples/performance-thread/common/arch/x86/ctx.c | 93 +++ examples/performance-thread/common/arch/x86/ctx.h | 57 ++ examples/performance-thread/common/common.

[dpdk-dev] [PATCH v9 1/4] doc: add sample application guide for performance-thread

2015-12-07 Thread Ian Betts
This commit adds the sample application user guide for the performance thread sample application. Signed-off-by: Ian Betts --- .../sample_app_ug/img/performance_thread_1.svg | 799 + .../sample_app_ug/img/performance_thread_2.svg | 865 ++ doc/guides/sample_app_

[dpdk-dev] [PATCH v9 0/4] examples: add performance-thread

2015-12-07 Thread Ian Betts
This patchset comprises a layer 3 forwarding derivative intended to facilitate characterization of performance with different threading models, specifically:- 1. EAL threads running on different physical cores 2. EAL threads running on the same physical core 3. Lightweight threads running in an EA

[dpdk-dev] [PATCH v2 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-07 Thread Kamil Rytarowski
Currently rte_eal_check_module() detects Linux kernel modules via reading /proc/modules. Built-in ones aren't listed there and therefore they are not being found by the script. Add support for checking built-in modules with parsing the sysfs files Signed-off-by: Kamil Rytarowski --- lib/librte_

[dpdk-dev] [PATCH v2 1/2] tools: Add support for handling built-in kernel modules

2015-12-07 Thread Kamil Rytarowski
Currently dpdk_nic_bind.py detects Linux kernel modules via reading /proc/modules. Built-in ones aren't listed there and therefore they are not being found by the script. Add support for checking built-in modules with parsing the sysfs files. Signed-off-by: Kamil Rytarowski --- tools/dpdk_nic_b

[dpdk-dev] KNI sample application running procedure

2015-12-07 Thread Abdul, Jaffar
Hi Nishant, Thanks for your suggestions but even it did not work, In the meanwhile, I will see what can I do from my side. Any other suggestions from the community is also really helpful at this time root at x10sdv-f:/opt/dpdk/x86_64-native-linuxapp-gcc/app# ./kni -c 0x0f -n 2 -- -P -p 0x3 --c

[dpdk-dev] [PATCH v9 3/4] examples: add l3fwd-thread example in performance-thread

2015-12-07 Thread Thomas Monjalon
2015-12-07 17:58, Ian Betts: > examples/performance-thread/l3fwd-thread/main.c | 3641 + It does not compile with clang: common/lthread_diag.c:165:1: error: unused function '_qnode_pool_display'

[dpdk-dev] [PATCH] bnx2x: set Ethernet address type during transmit for VF's

2015-12-07 Thread Harish Patil
> >On Sun, 2015-12-06 at 23:34 +, Harish Patil wrote: >> > >> >The original was always setting unicast. While here, clean up some >> >other references that also point into the Ethernet header. >> > >> >Signed-off-by: Chas Williams <3chas3 at gmail.com> >> >--- >> > drivers/net/bnx2x/bnx2x.c

[dpdk-dev] KNI sample application running procedure

2015-12-07 Thread Nishant Verma
As par error message, it seems error lies in validation. But it seems right Can you try to run the app without kthread option. if you haven't tried that. just to pin point the problem. ./kni -c 0x0f -n 2 -- -P -p 0x3 --config="(0,2,3),(1,0,1)" while executing, load kni module with single mode opti

[dpdk-dev] [PATCH] vhost: don't stall if guest is slow

2015-12-07 Thread Stephen Hemminger
When guest is booting (or any othertime guest is busy) it is possible for the small receive ring (256) to get full. If this happens the vhost library should just return normally. It's current behavior of logging just creates massive log spew/overflow which could even act as a DoS attack against hos

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Panu Matilainen
On 12/03/2015 04:05 PM, Panu Matilainen wrote: > In addition to git tags, support validating abi between any legal > gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " > "validate-abi.sh master mybranch " etc in addition to > validating between tags. Makes it easier to run the validator > fo

[dpdk-dev] [PATCH] tools: Add support for handling built-in kernel modules

2015-12-07 Thread Kamil Rytarowski
Currently dpdk_nic_bind.py detects Linux kernel modules via reading /proc/modules. Built-in ones aren't listed there and therefore they are not being found by the script. Add support for checking built-in modules with parsing the sysfs files. Signed-off-by: Kamil Rytarowski --- tools/dpdk_nic_b

[dpdk-dev] KNI sample application running procedure

2015-12-07 Thread Abdul, Jaffar
Hi Guys, I am trying to run the KNI sample application according to the DPDK documentation which is given below. http://dpdk.org/doc/guides/sample_app_ug/kernel_nic_interface.html#running-the-application I am not able to run the app successfully, I don't have this problem with any other DPDK

[dpdk-dev] [PATCH] config: cleanup existing RTE_CACHE_LINE_SIZE selection scheme

2015-12-07 Thread Thomas Monjalon
2015-12-07 19:52, Jerin Jacob: > by default, all the targets will be configured with the 64-byte cache line > size, targets which have different cache line size can be overridden > through target specific config file. > > Selected ThunderX and power8 as CONFIG_RTE_CACHE_LINE_SIZE=128 targets > bas

[dpdk-dev] [PATCH 0/2] fix performance/cache resource issues with 128-byte cache line targets

2015-12-07 Thread Thomas Monjalon
2015-12-07 20:03, Jerin Jacob: > On Mon, Dec 07, 2015 at 03:40:13AM -0800, Thomas Monjalon wrote: > > 2015-12-07 12:56, Jerin Jacob: > > > On Sun, Dec 06, 2015 at 05:30:50PM +0100, Thomas Monjalon wrote: > > > > 2015-12-06 21:29, Jerin Jacob: > > > > > This patchset fixes performance/cache resource

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Thomas Monjalon
2015-12-07 16:09, Panu Matilainen: > On 12/03/2015 04:05 PM, Panu Matilainen wrote: > > In addition to git tags, support validating abi between any legal > > gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " > > "validate-abi.sh master mybranch " etc in addition to > > validating between ta

[dpdk-dev] [PATCH] examples/vm_power_manager: fix build when libvirt version < 1.0

2015-12-07 Thread Yong Liu
virNodeGetCPUMap introduced in from libvirt 1.0. In some linux distributions like Ubuntu12/14 and Fedora18, libvirt version is elder than 1.0. So this sample will not build pass. Replace "virNodeGetCPUMap" with another libvirt API "virNodeGetInfo". Signed-off-by: Marvin Liu diff --git a/example

[dpdk-dev] [PATCH 1/2] mbuf: fix performance/cache resource issue with 128-byte cache line targets

2015-12-07 Thread Ananyev, Konstantin
Hi Jerin, > -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Sunday, December 06, 2015 3:59 PM > To: dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; Richardson, Bruce; olivier.matz at > 6wind.com; Dumitrescu, Cristian; Ananyev, Konstantin; Jeri

[dpdk-dev] [PATCH v3] scripts: add git hook scripts for checkpatch and auto doc generation

2015-12-07 Thread Ferruh Yigit
These scripts are to automate some common tasks, scripts needs to be deployed to specific folder to become active. Scripts: post-commit: Triggers after commit complete, re-generates api and guides html documents. "RTE_DOC_OUT" environment variable configures document output folder. Same script can

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Aaron Conole
Panu Matilainen writes: > Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but > for qat and mpipe drivers. The former did not exist when the > previous patch was sent and latter I just missed. > > Fixes: 5f9115e58cc6 ("mk: fix shared library dependencies of drivers") > > Signed-off-by:

[dpdk-dev] [PATCH] mk: fix warning spew when EXTRA_CFLAGS specifies warning flags

2015-12-07 Thread Panu Matilainen
Starting with commit 9aa2053c6e81493b23346ff4e387903560de5c81 EXTRA_CFLAGS is sometimes being passed to the compiler without WERROR_FLAGS which can cause spurious warnings by the dozen, for example with when compiling with EXTRA_CFLAGS="-Wformat-security": cc1: warning: -Wformat-security ignored w

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Thomas Monjalon
2015-12-07 13:41, Panu Matilainen: > On 12/07/2015 01:28 PM, Thomas Monjalon wrote: > > 2015-12-07 08:29, Panu Matilainen: > >> On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > >>> 2015-12-02 15:53, Panu Matilainen: > >> The vhost ABI break was announced for DPDK 2.2 in commit > >> 3c848bd7b1c6f4f6

[dpdk-dev] [PATCH] mk: fix warning spew when EXTRA_CFLAGS specifies warning flags

2015-12-07 Thread Simon Kågström
On 2015-12-07 13:56, Panu Matilainen wrote: > Starting with commit 9aa2053c6e81493b23346ff4e387903560de5c81 > EXTRA_CFLAGS is sometimes being passed to the compiler without > WERROR_FLAGS which can cause spurious warnings by the dozen, > for example with when compiling with EXTRA_CFLAGS="-Wformat-s

[dpdk-dev] [PATCH] i40e: remove redundant compiler warning disablers

2015-12-07 Thread Panu Matilainen
These may have been required at some point but current i40e base driver compiles cleanly without them, at least with clang 3.7.0 and gcc 5.1.1. Signed-off-by: Panu Matilainen --- drivers/net/i40e/Makefile | 13 - 1 file changed, 13 deletions(-) diff --git a/drivers/net/i40e/Makefile

[dpdk-dev] [PATCH v2 1/3] lpm: make rte_lpm_lookupx4 API definition architecture agnostic

2015-12-07 Thread Jianbo Liu
On 4 December 2015 at 23:14, Jerin Jacob wrote: > -Used architecture agnostic xmm_t to represent 128 bit SIMD variable > > -Introduced vect_* API abstraction in app/test to test rte_lpm_lookupx4 > API in architecture agnostic way > > -Moved rte_lpm_lookupx4 SSE implementation to architecture spe

[dpdk-dev] [PATCH v2 1/3] lpm: make rte_lpm_lookupx4 API definition architecture agnostic

2015-12-07 Thread Ananyev, Konstantin
> From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Friday, December 04, 2015 3:15 PM > To: dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; Ananyev, Konstantin; viktorin at > rehivetech.com; jianbo.liu at linaro.org; Jerin Jacob > Subject: [dpdk-dev] [PATCH v2 1/3] lpm: ma

[dpdk-dev] [PATCH v10 4/4] doc: add user-space ethtool sample app guide

2015-12-07 Thread Remy Horton
Signed-off-by: Remy Horton --- doc/guides/rel_notes/release_2_2.rst | 1 + doc/guides/sample_app_ug/ethtool.rst | 160 +++ doc/guides/sample_app_ug/index.rst | 1 + 3 files changed, 162 insertions(+) create mode 100644 doc/guides/sample_app_ug/ethtool.rst d

[dpdk-dev] [PATCH v10 3/4] example: add user-space ethtool sample application

2015-12-07 Thread Remy Horton
Further enhancements to the userspace ethtool implementation that was submitted in 2.1 and packaged as a self-contained sample application. Implements an rte_ethtool shim layer based on rte_ethdev API, along with a command prompt driven demonstration application. Signed-off-by: Remy Horton --- M

[dpdk-dev] [PATCH v10 2/4] mk: Fix missing directory with combined extlib build

2015-12-07 Thread Remy Horton
Signed-off-by: Remy Horton --- mk/rte.extlib.mk | 6 ++ 1 file changed, 6 insertions(+) diff --git a/mk/rte.extlib.mk b/mk/rte.extlib.mk index 4d459e4..47023a3 100644 --- a/mk/rte.extlib.mk +++ b/mk/rte.extlib.mk @@ -42,11 +42,17 @@ export NOT_FIRST_CALL all: $(Q)mkdir -p $(RTE_OUT

[dpdk-dev] [PATCH v10 1/4] Remove ABI requirement for external library builds.

2015-12-07 Thread Remy Horton
Signed-off-by: Andrew G. Harvey --- mk/rte.extlib.mk | 2 ++ mk/rte.lib.mk| 6 ++ 2 files changed, 8 insertions(+) diff --git a/mk/rte.extlib.mk b/mk/rte.extlib.mk index ba066bc..4d459e4 100644 --- a/mk/rte.extlib.mk +++ b/mk/rte.extlib.mk @@ -31,6 +31,8 @@ MAKEFLAGS += --no-print-dire

[dpdk-dev] [PATCH v10 0/4] User-space ethtool sample application

2015-12-07 Thread Remy Horton
Further enhancements to the userspace ethtool implementation that was submitted in 2.1 and packaged as a self-contained sample application. Implements an rte_ethtool shim layer based on rte_ethdev API, along with a command prompt driven demonstration application. This patchset depends on: * http:/

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 01:28 PM, Thomas Monjalon wrote: > 2015-12-07 08:29, Panu Matilainen: >> On 12/07/2015 01:07 AM, Thomas Monjalon wrote: >>> 2015-12-02 15:53, Panu Matilainen: This (and other changes in patch 2 breaks the librte_vhost ABI again, so you'd need to at least add a deprecation no

[dpdk-dev] [PATCH v2] scripts: add git hook scripts for checkpatch and auto doc generation

2015-12-07 Thread Ferruh Yigit
On Thu, Dec 03, 2015 at 11:09:30AM -0800, Thomas Monjalon wrote: > Ferruh, > I have a lot of questions :) > > 2015-11-27 14:34, Ferruh Yigit: > > --- a/scripts/checkpatches.sh > > +++ b/scripts/checkpatches.sh > > @@ -43,6 +43,7 @@ length=${DPDK_CHECKPATCH_LINE_LENGTH:-80} > > > > # override de

[dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline

2015-12-07 Thread Thomas Monjalon
2015-12-07 09:44, Singh, Jasvinder: > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > > Flow actions pipeline is an extension of flow-classification pipeline. > > > Some of the operations of flow classification pipeline such as traffic > > > > > > Signed-off-by: Jasvinde

[dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline

2015-12-07 Thread Thomas Monjalon
2015-12-07 10:51, Panu Matilainen: > On 12/07/2015 03:17 AM, Thomas Monjalon wrote: > > 2015-11-18 17:09, Fan Zhang: > >> Flow actions pipeline is an extension of flow-classification pipeline. > >> Some of the operations of flow classification pipeline such as traffic > >> metering/marking(for e.g.

[dpdk-dev] [PATCH 0/2] fix performance/cache resource issues with 128-byte cache line targets

2015-12-07 Thread Jerin Jacob
On Sun, Dec 06, 2015 at 05:30:50PM +0100, Thomas Monjalon wrote: > 2015-12-06 21:29, Jerin Jacob: > > This patchset fixes performance/cache resource issues with 128-byte cache > > line targets > > found in mbuf and bitmap DPDK libraries > > > > Currently, we have two DPDK targets(ThunderX and ppc

[dpdk-dev] [PATCH v3 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-07 Thread Stephen Hemminger
On Mon, 7 Dec 2015 19:36:05 +0100 Kamil Rytarowski wrote: > + /* Check if there is sysfs mounted */ > + if (stat("/sys/module", &st) != 0) { > + RTE_LOG(DEBUG, EAL, "Open /sys/module failed: %s\n", > + strerror(errno)); > return -1; > }

[dpdk-dev] [PATCH v2] mk: pass EXTRA_CFLAGS to AUTO_CPUFLAGS to enable local modifications

2015-12-07 Thread Thomas Monjalon
2015-12-07 10:33, Panu Matilainen: > On 12/04/2015 08:53 PM, Thomas Monjalon wrote: > We have encountered a CPU where the AES-NI instruction set is disabled > due to export restrictions. Since the build machine and target machine > is different, using -native configs doesn't work, an

[dpdk-dev] [PATCH v2] mk: pass EXTRA_CFLAGS to AUTO_CPUFLAGS to enable local modifications

2015-12-07 Thread Simon Kågström
On 2015-12-07 09:33, Panu Matilainen wrote: > On 12/04/2015 08:53 PM, Thomas Monjalon wrote: > We have encountered a CPU where the AES-NI instruction set is disabled > due to export restrictions. Since the build machine and target machine > is different, using -native configs doesn't wo

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Panu Matilainen
Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but for qat and mpipe drivers. The former did not exist when the previous patch was sent and latter I just missed. Fixes: 5f9115e58cc6 ("mk: fix shared library dependencies of drivers") Signed-off-by: Panu Matilainen --- v2: - typo/co

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Thomas Monjalon
2015-12-07 07:47, Liu, Jijiang: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2015-12-07 03:30, Liu, Jijiang: > > > Hi Thomas, > > > > > > > -Original Message- > > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > Sent: Monday, December 07, 2015 11:

[dpdk-dev] Meaning of ETH_TXQ_FLAGS_NOREFCOUNT

2015-12-07 Thread Mike Stolarchuk
Hello, I would have thought that ETH_TXQ_FLAGS_NOREFCOUNT, when enabled, would mean to ignore ref counts, as commented in rte_ethdev.h: #define ETH_TXQ_FLAGS_NOREFCOUNT 0x0002 /**< refcnt can be ignored */ But it seems to be used differently: if (!(txq->txq_flags & (uint32_t)ETH_TXQ_FLAGS_N

[dpdk-dev] [PATCH] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Panu Matilainen
Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but for qat and mpipe drivers. The former did not exist when the previous patch was sent and latter I just missed. Fixes: 5f9115e58cc6 ("mk: fix shared library dependencies of drivers") Signed-off-by: Panu Matilainen --- drivers/crypto

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Thomas Monjalon
2015-12-07 08:29, Panu Matilainen: > On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > > 2015-12-02 15:53, Panu Matilainen: > >> This (and other changes in patch 2 breaks the librte_vhost ABI again, so > >> you'd need to at least add a deprecation note to 2.2 to be able to do it > >> in 2.3 at all a

[dpdk-dev] [PATCH v2 1/3] lpm: make rte_lpm_lookupx4 API definition architecture agnostic

2015-12-07 Thread Jerin Jacob
On Mon, Dec 07, 2015 at 02:15:28PM +0800, Jianbo Liu wrote: > On 4 December 2015 at 23:14, Jerin Jacob > wrote: > > -Used architecture agnostic xmm_t to represent 128 bit SIMD variable > > > > -Introduced vect_* API abstraction in app/test to test rte_lpm_lookupx4 > > API in architecture agnosti

[dpdk-dev] [PATCH v8 3/4] examples: add l3fwd-thread example in performance-thread

2015-12-07 Thread Thomas Monjalon
2015-12-07 04:46, Betts, Ian: > >>I had no time to check how it works. > >> I'm just surprised that you introduce a new config option. > >> +DIRS-$(CONFIG_RTE_PERFORMANCE_THREAD) += performance-thread > > I think this option should be removed. > > It is only to ensure that performance-thread examp

[dpdk-dev] [PATCH v9 2/3] example: add user-space ethtool sample application

2015-12-07 Thread Remy Horton
On 07/12/2015 02:29, Thomas Monjalon wrote: > 2015-11-20 15:34, Remy Horton: >> Signed-off-by: Remy Horton >> --- >> MAINTAINERS | 4 + >> examples/Makefile | 1 + >> examples/ethtool/Makefile | 48 ++ > > I'm really sorry, it doe

[dpdk-dev] [PATCH] vhost: note the ABI changes

2015-12-07 Thread Yuanhan Liu
Note the ABI changes and update the ABI version to 2. Cc: Thomas Monjalon Signed-off-by: Yuanhan Liu --- This patch is assumed to be applied on top of following patch: http://dpdk.org/dev/patchwork/patch/9262/ --- doc/guides/rel_notes/release_2_2.rst | 13 - lib/librte_vhost/M

[dpdk-dev] [i40e] Failed to init adminq?

2015-12-07 Thread Eimear Morrissey
Stephen Hemminger wrote on 12/04/2015 05:36:08 PM: > From: Stephen Hemminger > To: Eimear Morrissey/Ireland/IBM at IBMIE > Cc: dev at dpdk.org > Date: 12/04/2015 05:36 PM > Subject: Re: [dpdk-dev] [i40e] Failed to init adminq? > > On Fri, 4 Dec 2015 11:25:09 + > "Eimear Morrissey" wrote:

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Jijiang Liu
Announce ABI change for struct rte_eth_conf. Signed-off-by: Jijiang Liu --- doc/guides/rel_notes/deprecation.rst |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 1c7ab01..f50f0c7 100644

[dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline

2015-12-07 Thread Panu Matilainen
On 12/07/2015 03:17 AM, Thomas Monjalon wrote: > 2015-11-18 17:09, Fan Zhang: >> Flow actions pipeline is an extension of flow-classification pipeline. >> Some of the operations of flow classification pipeline such as traffic >> metering/marking(for e.g. Single Rate Three Color Marker (srTCM), Two

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Chilikin, Andrey
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jijiang Liu > Sent: Monday, December 7, 2015 3:02 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf > > Announce ABI change for struct rte_eth_conf. > > S

[dpdk-dev] [PATCH v2] mk: pass EXTRA_CFLAGS to AUTO_CPUFLAGS to enable local modifications

2015-12-07 Thread Panu Matilainen
On 12/04/2015 08:53 PM, Thomas Monjalon wrote: We have encountered a CPU where the AES-NI instruction set is disabled due to export restrictions. Since the build machine and target machine is different, using -native configs doesn't work, and on this CPU, the application refuses

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Yuanhan Liu
On Mon, Dec 07, 2015 at 03:03:24AM +0100, Thomas Monjalon wrote: > 2015-12-07 10:00, Yuanhan Liu: > > On Mon, Dec 07, 2015 at 12:07:28AM +0100, Thomas Monjalon wrote: > > > 2015-12-02 15:53, Panu Matilainen: > > > > This (and other changes in patch 2 breaks the librte_vhost ABI again, > > > > so

[dpdk-dev] [PATCH 0/6] Add virtio support in arm/arm64

2015-12-07 Thread Yuanhan Liu
On Fri, Dec 04, 2015 at 11:05:13PM +0530, Santosh Shukla wrote: > This patch set add basic infrastrucure to run virtio-net-pci pmd driver for > arm64/arm. Tested on ThunderX platfrom. Verified for existing dpdk(s) test > applications like: > - ovs-dpdk-vhost-user: across the VM's, for the use-cases

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Yuanhan Liu
On Mon, Dec 07, 2015 at 12:07:28AM +0100, Thomas Monjalon wrote: > 2015-12-02 15:53, Panu Matilainen: > > This (and other changes in patch 2 breaks the librte_vhost ABI again, so > > you'd need to at least add a deprecation note to 2.2 to be able to do it > > in 2.3 at all according to the ABI po

[dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline

2015-12-07 Thread Singh, Jasvinder
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, December 7, 2015 1:18 AM > To: Zhang, Roy Fan > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline > > 2015-11-18 17:09, Fan Zhang:

[dpdk-dev] [PATCH] mk: fix the combined library problems by replacing it with a linker script

2015-12-07 Thread Christian Ehrhardt
Hi, FYI I kind of "gave up" (not as bad as it sounds) and started looking into shipping it as individual libraries + linker script as well. To me it seemed what was more accepted in all the former discussions. It will surely cause more work for me in the short term, but I hope after the initial hi

[dpdk-dev] [PATCH 3/6] virtio: armv7/v8: Introdice api to emulate x86-style of PCI/ISA ioport access

2015-12-07 Thread Stephen Hemminger
On Fri, 4 Dec 2015 23:05:16 +0530 Santosh Shukla wrote: > +#if defined(RTE_ARCH_ARM64) > + uint64_tio_base; > +#else /* !ARM64 */ > uint32_tio_base; > +#endif Is there a typedef that could be used instead of having #ifdef clutter?

[dpdk-dev] [PATCH 6/6] virtio: arm/arm64: memory mapped IO support in pmd driver

2015-12-07 Thread Stephen Hemminger
On Fri, 4 Dec 2015 23:05:19 +0530 Santosh Shukla wrote: > > +#ifdef RTE_EXEC_ENV_LINUXAPP > +/* start address of first pci_iobar slot (user-space virtual-addres) */ > +void *ioport_map; > +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) > + > +#include > +#define DEV_NAME "/d

[dpdk-dev] [PATCH] examples/dpdk_qat: Fix RX queue start number to the one just received the packets

2015-12-07 Thread Declan Doherty
On 07/12/15 01:32, Thomas Monjalon wrote: > 2015-09-29 19:07, Zhe Tao: >> Every time we started to receive the packets, the start queue number >> should be the one that just received the packets, should not start from zero! >> > [...] >> -for (i = 0; i < qconf->n_rx_queue; i++) { >> +

[dpdk-dev] [PATCH] mk: fix the combined library problems by replacing it with a linker script

2015-12-07 Thread Martinx - ジェームズ
Hi Christian, You can count on me to help testing DPDK for Ubuntu, I have plans for it! I have some experience with Debian packaging too... I'm currently maintaining few Ubuntu PPAs, for fun...=) Also, I have hardware available, with 10G, 40G and 100G NIC cards and traffic generators. I

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > 2015-12-02 15:53, Panu Matilainen: >> This (and other changes in patch 2 breaks the librte_vhost ABI again, so >> you'd need to at least add a deprecation note to 2.2 to be able to do it >> in 2.3 at all according to the ABI policy. >> >> Perhaps a b

[dpdk-dev] [PATCH 1/1] kni: Fix compilation problems on RHEL 7.2.

2015-12-07 Thread Lee Roberts
RHEL 7.2 contains additional backports from newer upstream kernels. Add RHEL_RELEASE_CODE logic for RHEL_RELEASE_VERSION(7,2) to pick up the changes to kernel functions. Signed-off-by: Lee Roberts --- lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c | 4 ++-- lib/librte_eal/linuxapp/kni/ethto

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Liu, Jijiang
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, December 07, 2015 11:40 AM > To: Liu, Jijiang > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for struct > rte_eth_conf > > 2015-12-07 03:30, Liu, Jijiang:

[dpdk-dev] [PATCH] bnx2x: set Ethernet address type during transmit for VF's

2015-12-07 Thread Charles (Chas) Williams
On Sun, 2015-12-06 at 23:34 +, Harish Patil wrote: > > > >The original was always setting unicast. While here, clean up some > >other references that also point into the Ethernet header. > > > >Signed-off-by: Chas Williams <3chas3 at gmail.com> > >--- > > drivers/net/bnx2x/bnx2x.c | 23 +++

[dpdk-dev] [PATCH 0/2] Clear rx_mbuf_alloc_failed counter on rte_eth_stats_reset

2015-12-07 Thread Thomas Monjalon
2015-11-27 13:31, Igor Ryzhov: > The rx_mbuf_alloc_failed counter was only cleared by virtio driver. > Now it is cleared by common rte_eth_stats_clear function for all drivers at > once. > > Igor Ryzhov (2): > ethdev: clear rx_mbuf_alloc_failed counter on rte_eth_stats_reset > virtio: remove

[dpdk-dev] [PATCH v8 3/4] examples: add l3fwd-thread example in performance-thread

2015-12-07 Thread Betts, Ian
-Original Message- From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] Sent: Monday, December 7, 2015 2:36 AM To: Betts, Ian Cc: dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH v8 3/4] examples: add l3fwd-thread example in performance-thread > Hi, >>I had no time to check how it wo

[dpdk-dev] [PATCH] doc: Add missing new line before code block

2015-12-07 Thread Thomas Monjalon
> > The patch adds missing new line to "Managing ABI updates" section. > > > > Signed-off-by: Tetsuya Mukawa > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Thomas Monjalon
2015-12-07 03:30, Liu, Jijiang: > Hi Thomas, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Monday, December 07, 2015 11:17 AM > > To: Liu, Jijiang > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for s

[dpdk-dev] [PATCH] doc: fix examples in netmap compatibility docs

2015-12-07 Thread Thomas Monjalon
> > Fix the examples in the netmap compatibility sample application > > docs which referred to the packet_reordering application. > > > > Also fix some minor rst formatting issues. > > > > Reported-by: Qian Xu > > Signed-off-by: John McNamara > > Acked-by: Pablo de Lara Applied, thanks

[dpdk-dev] [PATCH v2] doc: fix repeated typo in sample app docs

2015-12-07 Thread Thomas Monjalon
> > Fix repeated typo in the "Compiling the Application" section of > > almost all of the sample app docs. > > > > This generally gets copied into new sample app guides. > > > > Signed-off-by: John McNamara > > Acked-by: Pablo de Lara Applied, thanks

[dpdk-dev] [PATCH] remove blank lines at end-of-file

2015-12-07 Thread Thomas Monjalon
2015-12-03 09:53, Bruce Richardson: > On Wed, Dec 02, 2015 at 12:06:12PM -0800, Stephen Hemminger wrote: > > This is one of those trivial things git and other tools complain > > about. > > > > Signed-off-by: Stephen Hemminger > Acked-by: Bruce Richardson Applied, thanks

[dpdk-dev] [PATCH] remove double semicolons

2015-12-07 Thread Thomas Monjalon
2015-12-03 09:53, Bruce Richardson: > On Wed, Dec 02, 2015 at 01:02:32PM -0800, Stephen Hemminger wrote: > > Trivial cleanup > > > > Signed-off-by: Stephen Hemminger > Acked-by: Bruce Richardson Applied, thanks

[dpdk-dev] [PATCH] librte_power: channel_commands: Remove duplicate definition

2015-12-07 Thread Thomas Monjalon
> > The CHANNEL_CMDS_MAX_VM_CHANNELS is duplicated in the channel_commands > > header file. This commit removes that duplication. > > > > fixes 210c383e247b5335cd95223fac1c0880ebbd5c96 ("power: packet format for > > ...") > > > > Signed-off-by: Aaron Conole > > Acked-by: Bruce Richardson App

[dpdk-dev] [PATCH] doc: announce ABI change for struct rte_eth_conf

2015-12-07 Thread Thomas Monjalon
2015-12-07 11:01, Jijiang Liu: > +* ABI changes are planned for struct rte_eth_conf in order to support > + tunneling packet configuration in unified tunneling API. The release 2.2 > does not contain these ABI > + changes, but release 2.3 will, and no backwards compatibility is planned. Please,

[dpdk-dev] [PATCH] app/testpmd: add 'show (rxq|txq)' command description into UG and cmdline help

2015-12-07 Thread Thomas Monjalon
> > Signed-off-by: Konstantin Ananyev > > Acked-by: Pablo de Lara Fixes: ab3257e13dc4 ("app/testpmd: add command to display queue info") Applied, thanks

[dpdk-dev] [PATCH] app/test: fix of returning wrong value

2015-12-07 Thread Thomas Monjalon
2015-12-03 11:22, Helin Zhang: > In the KNI unit test, if all test cases passed, it should > return with 0. This patch fixes the issue of returning > wrong value. > > Fixes: fc27caaafd4b ("kni: remove deprecated functions") > > Signed-off-by: Helin Zhang Applied, thanks

[dpdk-dev] [PATCH 1/1] table_test: renamed test_table_ACL to lowercase

2015-12-07 Thread Thomas Monjalon
> > Renamed function name to comply with coding standard. > > > > Signed-off-by: Maciej Gajdzica > > Acked-by: Pablo de Lara Applied, thanks

[dpdk-dev] [PATCH v4 0/2] ring pmd autotest

2015-12-07 Thread Thomas Monjalon
2015-11-28 11:01, Bernard Iremonger: > Fix failures in the ring_pmd_autotest program. > Correct errors in the ring PMD documentation. > > Changes in v4: > rebase to latest code. > update commit messages. > add Fixes: line to commit messages. > > Changes in v3: > rebase to latest code. > revise co

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Thomas Monjalon
2015-12-07 10:18, Yuanhan Liu: > On Mon, Dec 07, 2015 at 03:03:24AM +0100, Thomas Monjalon wrote: > > 2015-12-07 10:00, Yuanhan Liu: > > > On Mon, Dec 07, 2015 at 12:07:28AM +0100, Thomas Monjalon wrote: > > > > 2015-12-02 15:53, Panu Matilainen: > > > > > This (and other changes in patch 2 breaks

[dpdk-dev] [PATCH] app/test: fix memory_autotest integer overflow/wraparound

2015-12-07 Thread Thomas Monjalon
> > memory_autotest loops infinitely when at least one the memsegs > > is bigger than 4GB. > > > > The issue is the result of an integer overflow/wraparound of > > the offset variable. > > > > Fix it by using the correct type (size_t). > > > > Signed-off-by: Sergio Gonzalez Monroy > > > > Acke

[dpdk-dev] [PATCH] test: fix crash in pmd_perf_test

2015-12-07 Thread Thomas Monjalon
2015-11-23 11:45, David Hunt: > Fix crash in pmd_perf_test autotest (div by 0) when no packets received > Also fixes the fact that the test passes even if exec_burst fails > > To repeat the issue: > The system must be incorrectly set up so that all packets will be lost, > i.e. no loopback cab

  1   2   >