[dpdk-dev] xstats performance

2016-07-07 Thread Rasesh Mody
> From: Remy Horton [mailto:remy.horton at intel.com] > Sent: Wednesday, July 06, 2016 12:44 AM > > > On 05/07/2016 19:10, Rasesh Mody wrote: > [..] > >> For all the current drivers xstats_names[idx].id==idx so it would > >> just involve removing the references to the id field and updating the >

[dpdk-dev] [PATCH 1/3] net/bnx2x: remove unneeded dependency on hash library

2016-07-07 Thread Rasesh Mody
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Wednesday, July 06, 2016 7:45 AM > > Fixes: 9fb557035d90 ("bnx2x: enable PMD build") > > Signed-off-by: Thomas Monjalon Acked-by: Rasesh Mody Thanks! Rasesh

[dpdk-dev] [PATCH 07/11] pmdinfogen: fix build warnings

2016-07-07 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > Sent: Thursday, July 7, 2016 6:55 PM > To: Thomas Monjalon > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 07/11] pmdinfogen: fix build warnings > > On Thu, Jul 07, 2016 at 05:36:26PM

[dpdk-dev] [PATCH] net/mlx5: fix packet type and offload flags in Rx

2016-07-07 Thread Maxime Leroy
In mlx5 rx function, the packet_type and ol_flags mbuf fields are not properly initialized when no rx offload feature is enabled (checksum, l2 tun checksum, vlan_strip, crc). Thus, these fields can have a value different of 0 depending on their value when the mbuf was freed. This can result in an

[dpdk-dev] [PATCH v3 00/11] Fix build errors related to exported headers

2016-07-07 Thread Wiles, Keith
> On Jul 7, 2016, at 10:49 AM, Adrien Mazarguil > wrote: > > DPDK uses GNU C language extensions in most of its code base. This is fine > for internal source files whose compilation flags are controlled by DPDK, > however user applications that use exported "public" headers may experience >

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Thomas Monjalon
2016-07-07 12:14, Neil Horman: > On Thu, Jul 07, 2016 at 05:45:09PM +0200, Thomas Monjalon wrote: > > Neil, > > > > I would gladly drop this patch from the series if you send a similar one > > with your name as maintainer. > > > > Thanks > > > Lets get through the rest of this stuff first, then

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-07 Thread Thomas Monjalon
2016-07-07 12:11, Neil Horman: > On Thu, Jul 07, 2016 at 05:36:28PM +0200, Thomas Monjalon wrote: > > Remove include of rte_pci.h in the generic header rte_dev.h > > and move the macro DRIVER_REGISTER_PCI_TABLE in rte_pci.h. [...] > > --- a/lib/librte_eal/common/include/rte_dev.h > > +++

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Thomas Monjalon
2016-07-07 11:56, Neil Horman: > On Thu, Jul 07, 2016 at 05:36:22PM +0200, Thomas Monjalon wrote: > > When compiling the drivers, some code is generated with pmdinfogen. > > A fresh parallel build can fail if a driver is compiled before pmdinfogen: > > build/buildtools/dpdk-pmdinfogen:

[dpdk-dev] [PATCH 01/11] drivers: fix build with new register macro

2016-07-07 Thread Adrien Mazarguil
On Thu, Jul 07, 2016 at 05:36:20PM +0200, Thomas Monjalon wrote: > Compilation fails because of some typos. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon > --- > drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 2 +- >

[dpdk-dev] [PATCH 1/2] net/enic: fix setting MAC address when a port is restarted

2016-07-07 Thread Nelson Escobar
enic_disable() removed the MAC address when a port was shut down but enic_enable() didn't add the MAC address back when the port was started again. Move where we set the MAC address for the adapter from enic_setup_finish() to a enic_enable() so that port restarting works properly. Fixes:

[dpdk-dev] [PATCH 09/18] mbuf: support Mpls in software packet type parser

2016-07-07 Thread Olivier Matz
Hi Cunming, On 07/07/2016 10:48 AM, Liang, Cunming wrote: > Hi Olivier, > >> -Original Message- >> From: Olivier MATZ [mailto:olivier.matz at 6wind.com] >> Sent: Wednesday, July 06, 2016 4:00 PM >> To: Liang, Cunming ; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH 09/18] mbuf:

[dpdk-dev] [PATCH v3 11/11] scripts: check compilation of exported header files

2016-07-07 Thread Adrien Mazarguil
This script checks that header files in a given directory do not miss dependencies when included on their own, do not conflict and accept being compiled with the strictest possible flags. Signed-off-by: Adrien Mazarguil --- MAINTAINERS | 1 + scripts/check-includes.sh | 286

[dpdk-dev] [PATCH v3 10/11] lib: hide static functions never defined

2016-07-07 Thread Adrien Mazarguil
Arch-specific functions not defined for all architectures (missing on x86 in this case) and not used anywhere should not expose a prototype. This commit prevents the following error: error: `rte_mov48' declared `static' but never defined Signed-off-by: Adrien Mazarguil ---

[dpdk-dev] [PATCH v3 09/11] lib: remove named variadic macros in exported headers

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. Since there is no way to force named variadic macros as extensions, use a a standard

[dpdk-dev] [PATCH v3 08/11] lib: work around forward reference to enum types

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. This commit prevents the following errors: error: ISO C forbids forward references to

[dpdk-dev] [PATCH v3 07/11] lib: add missing include dependencies

2016-07-07 Thread Adrien Mazarguil
Exported header files for use by applications should be self sufficient and allow out of order inclusion. Moreover, they must include all the system headers they need for types and macros. This commit prevents the following errors: error: `RTE_MAX_LCORE' undeclared here (not in a function)

[dpdk-dev] [PATCH v3 06/11] lib: work around unnamed structs/unions

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked to avoid warnings and compilation failures. Unnamed structs/unions are allowed since C11, however many compiler versions do not use this mode by

[dpdk-dev] [PATCH v3 05/11] lib: work around structs with no members

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. This commit prevents the following errors: error: struct has no members Signed-off-by:

[dpdk-dev] [PATCH v3 04/11] lib: work around nonstandard bit-fields

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. This commit prevents the following errors: error: type of bit-field `[...]' is a GCC

[dpdk-dev] [PATCH v3 03/11] lib: use C99 syntax for zero-size arrays

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. The extension keyword is used whenever the C99 syntax cannot do it. This commit prevents

[dpdk-dev] [PATCH v3 02/11] lib: work around large enum values

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. This commit prevents the following errors: error: ISO C restricts enumerator values to

[dpdk-dev] [PATCH v3 01/11] lib: work around braced-groups within expressions

2016-07-07 Thread Adrien Mazarguil
Exported header files used by applications should allow the strictest compiler flags. Language extensions used in many places must be explicitly marked or removed to avoid warnings and compilation failures. This commit prevents the following errors: error: ISO C forbids braced-groups within

[dpdk-dev] [PATCH v3 00/11] Fix build errors related to exported headers

2016-07-07 Thread Adrien Mazarguil
DPDK uses GNU C language extensions in most of its code base. This is fine for internal source files whose compilation flags are controlled by DPDK, however user applications that use exported "public" headers may experience compilation failures when enabling strict error/standard checks (-std and

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-07 Thread Olivier Matz
Hi Cunming, Thank you for your feedback. On 07/07/2016 10:19 AM, Liang, Cunming wrote: > Hi Olivier, > >> -Original Message- >> From: Olivier MATZ [mailto:olivier.matz at 6wind.com] >> Sent: Wednesday, July 06, 2016 3:43 PM >> To: Liang, Cunming ; dev at dpdk.org >> Subject: Re:

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Thomas Monjalon
Neil, I would gladly drop this patch from the series if you send a similar one with your name as maintainer. Thanks 2016-07-07 17:36, Thomas Monjalon: > The author of this feature is Neil Horman. > > Signed-off-by: Thomas Monjalon > --- > +Driver information > +F: buildtools/pmdinfogen/ >

[dpdk-dev] [PATCH 10/11] doc: fix syntax in pmdinfogen guide

2016-07-07 Thread Thomas Monjalon
2016-07-07 17:36, Thomas Monjalon: > Sphynx reports this error: > > doc/guides/prog_guide/dev_kit_build_system.rst:337: WARNING: > Pygments lexer name u'C' is not known > > Fixes: 737ddf3fb ("doc: add prog guide section documenting pmdinfo script") > > Signed-off-by: Thomas Monjalon

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-07 Thread Thomas Monjalon
2016-07-07 17:36, Thomas Monjalon: > Remove include of rte_pci.h in the generic header rte_dev.h > and move the macro DRIVER_REGISTER_PCI_TABLE in rte_pci.h. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon Suggested-by: David Marchand

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Thomas Monjalon
The author of this feature is Neil Horman. Signed-off-by: Thomas Monjalon --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index a59191e..f996c2e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -68,6 +68,10 @@ F: lib/librte_compat/ F:

[dpdk-dev] [PATCH 10/11] doc: fix syntax in pmdinfogen guide

2016-07-07 Thread Thomas Monjalon
Sphynx reports this error: doc/guides/prog_guide/dev_kit_build_system.rst:337: WARNING: Pygments lexer name u'C' is not known Fixes: 737ddf3fb ("doc: add prog guide section documenting pmdinfo script") Signed-off-by: Thomas Monjalon --- doc/guides/prog_guide/dev_kit_build_system.rst | 5 ++---

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-07 Thread Thomas Monjalon
Remove include of rte_pci.h in the generic header rte_dev.h and move the macro DRIVER_REGISTER_PCI_TABLE in rte_pci.h. Fixes: cb6696d22023 ("drivers: update registration macro usage") Signed-off-by: Thomas Monjalon --- lib/librte_eal/common/include/rte_dev.h | 6 +-

[dpdk-dev] [PATCH 08/11] pmdinfogen: fix usage message

2016-07-07 Thread Thomas Monjalon
The name of the tool is pmdinfogen. Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") Signed-off-by: Thomas Monjalon --- buildtools/pmdinfogen/pmdinfogen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/buildtools/pmdinfogen/pmdinfogen.c

[dpdk-dev] [PATCH 07/11] pmdinfogen: fix build warnings

2016-07-07 Thread Thomas Monjalon
When compiled with a standard clang, pmdinfogen can raise a warning: buildtools/pmdinfogen/pmdinfogen.c:365:1: warning: control reaches end of non-void function Actually there can be more warnings with stricter compilers. In order to catch them early and fix most of them, the DPDK

[dpdk-dev] [PATCH 06/11] mk: fix verbose pmdinfogen run

2016-07-07 Thread Thomas Monjalon
When building with "make V=1" it is expected to see the output of each compiler command in order to debug them. Unfortunately the pmdinfogen related commands were always quiet. It is fixed by defining the commands in some Makefile variables. They are printed if the verbose mode is enabled. The

[dpdk-dev] [PATCH 05/11] mk: fix driver build with installed SDK

2016-07-07 Thread Thomas Monjalon
The tool pmdinfogen was called from RTE_OUTPUT/app/ which does not exist if building a driver outside of the SDK build. When building DPDK, RTE_SDK_BIN is RTE_OUTPUT. When building an external driver, RTE_SDK_BIN must point to the installed DPDK directory containing includes, libs, etc. That's

[dpdk-dev] [PATCH 04/11] mk: remove traces of hostapp build directory

2016-07-07 Thread Thomas Monjalon
The recipe rte.hostapp.mk does not build in hostapp/ anymore. Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") Signed-off-by: Thomas Monjalon --- doc/guides/freebsd_gsg/build_dpdk.rst | 2 +- doc/guides/linux_gsg/build_dpdk.rst | 2 +- mk/rte.sdkbuild.mk

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Thomas Monjalon
When compiling the drivers, some code is generated with pmdinfogen. A fresh parallel build can fail if a driver is compiled before pmdinfogen: build/buildtools/dpdk-pmdinfogen: Permission denied There was a dependency declared in drivers/Makefile but it cannot work because this file is

[dpdk-dev] [PATCH 02/11] crypto: fix parameters registration

2016-07-07 Thread Thomas Monjalon
The name aesni_gcm was used to register parameters of several crypto drivers. Fixes: cb6696d22023 ("drivers: update registration macro usage") Signed-off-by: Thomas Monjalon --- drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 2 +- drivers/crypto/null/null_crypto_pmd.c | 2 +-

[dpdk-dev] [PATCH 00/11] additions to pmdinfogen

2016-07-07 Thread Thomas Monjalon
These are the patches that I think needed for RC2. There are other patches needed as discussed on the mailing-list which are not part of this series: - overwritten driver name (reported by Pablo) - pmdinfo.py on FreeBSD (reported by Bruce) Thomas Monjalon (11): drivers: fix build with new

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-07 Thread Luca Boccassi
On Thu, 2016-07-07 at 13:12 +, Lu, Wenzhuo wrote: > > -Original Message- > > From: Luca Boccassi [mailto:lboccass at Brocade.com] > > Sent: Thursday, July 7, 2016 6:21 PM > > To: Lu, Wenzhuo > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v6 0/4] support reset of VF link > >

[dpdk-dev] 16.11 Roadmap

2016-07-07 Thread O'Driscoll, Tim
We published our initial roadmap for 16.11 at the start of May. Since then, we've been doing more detailed planning and would like to provide an update on the features that we plan to submit for this release. We'll schedule a community call where some of our software developers will describe

[dpdk-dev] [PATCH v1 28/28] ether: support SoC device/driver

2016-07-07 Thread Shreyansh jain
Hi Jan, Apologies for delay in my response. On Tuesday 05 July 2016 10:46 AM, Jan Viktorin wrote: > Hello Shreyansh, > ? >> On Monday 04 July 2016 08:06 PM, Jan Viktorin wrote: >>> On Mon, 4 Jul 2016 19:57:18 +0530 >>> Shreyansh jain wrote: >>> >>> [...] >>> >>> @@ -1431,7 +1524,7 @@

[dpdk-dev] [PATCH 2/2] bnx2x: add support for xstats

2016-07-07 Thread Rasesh Mody
This patch adds support for extended statistics for BNX2X PMD. Signed-off-by: Rasesh Mody --- doc/guides/nics/overview.rst |2 +- drivers/net/bnx2x/bnx2x_ethdev.c | 86 ++ 2 files changed, 87 insertions(+), 1 deletion(-) diff --git

[dpdk-dev] [PATCH 1/2] qede: add support for xstats

2016-07-07 Thread Rasesh Mody
This patch adds support for extended statistics for QEDE PMD. Signed-off-by: Rasesh Mody --- doc/guides/nics/overview.rst |2 +- drivers/net/qede/qede_ethdev.c | 204 ++-- 2 files changed, 197 insertions(+), 9 deletions(-) diff --git

[dpdk-dev] [PATCH 2/2] vfio: fix coding style

2016-07-07 Thread Yong Wang
Signed-off-by: Yong Wang --- lib/librte_eal/linuxapp/eal/eal_pci_vfio.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c b/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c index 3729c35..6492216 100644 ---

[dpdk-dev] [PATCH 1/2] vfio: fix pci_vfio_map_resource

2016-07-07 Thread Yong Wang
The offset of the 2nd mmap when mapping the region after msix_bar needs to take region address into consideration. This is exposed when using vfio-pci to manage vmxnet3 pmd. Fixes: 90a1633b2347 ("eal/linux: allow to map BARs with MSI-X tables") Signed-off-by: Yong Wang Signed-off-by: Ronghua

[dpdk-dev] [PATCH v3] mempool: replace c memcpy code semantics with optimized rte_memcpy

2016-07-07 Thread Ferruh Yigit
On 7/6/2016 5:21 PM, Ferruh Yigit wrote: > On 7/5/2016 3:09 PM, Ferruh Yigit wrote: >> On 7/5/2016 2:13 PM, Jerin Jacob wrote: >>> On Tue, Jul 05, 2016 at 07:32:46PM +0800, Yuanhan Liu wrote: On Tue, Jul 05, 2016 at 09:43:03AM +0100, Ferruh Yigit wrote: > On 6/30/2016 6:28 PM, Thomas

[dpdk-dev] [PATCH] net/virtio: fix gcc6 i686 compiler error

2016-07-07 Thread Ferruh Yigit
This is for target i686-native-linuxapp-gcc and gcc6, Compilation error is: == Build drivers/net/virtio CC virtio_rxtx_simple.o In file included from .../build/include/rte_mempool.h:77:0, from .../drivers/net/virtio/virtio_rxtx_simple.c:46:

[dpdk-dev] [PATCH] ixgbe: support checksum flags in sse vector Rx function

2016-07-07 Thread Olivier Matz
Update desc_to_olflags_v() to set PKT_RX_IP_CKSUM_BAD and PKT_RX_L4_CKSUM_BAD in the ol_fags of the mbuf. The Rx vector function can now be used with hw_ip_checksum enabled. Tested with: cd dpdk.org/ make config T=x86_64-native-linuxapp-gcc make -j32 mkdir -p /mnt/huge mount -t

[dpdk-dev] Help: How to read packet statistics from device registers via dpdk PMD?

2016-07-07 Thread Bill Bonaparte
I am so happy to get your reply. My dpdk version is 2.1?and the OS is centOS 7? the following is the output from "dpdk_nic_bind.py --status": [root at APV35 ~]# dpdk_nic_bind.py --status Network devices using DPDK-compatible driver :04:00.0

[dpdk-dev] [PATCH 08/11] pmdinfogen: fix usage message

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 06:24:16PM +0200, Thomas Monjalon wrote: > 2016-07-07 12:05, Neil Horman: > > On Thu, Jul 07, 2016 at 05:36:27PM +0200, Thomas Monjalon wrote: > > > - "usage: pmdinfo \n"); > > > + "usage: pmdinfogen \n"); > > > > That should probably be

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 06:21:49PM +0200, Thomas Monjalon wrote: > 2016-07-07 11:56, Neil Horman: > > On Thu, Jul 07, 2016 at 05:36:22PM +0200, Thomas Monjalon wrote: > > > When compiling the drivers, some code is generated with pmdinfogen. > > > A fresh parallel build can fail if a driver is

[dpdk-dev] [PATCH] ena: fix doorbell submission when not needed

2016-07-07 Thread Jan Mędala
Avoid submitting doorbell when: * no packets have been submitted to TX * no free resources have been submitted while RX Sending doorbell without actual work to be performed by device violates ENA specification and can lead to unpredictable behavior. Fixes: 1173fca25af9 ("ena: add polling-mode

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-07 Thread Lu, Wenzhuo
> -Original Message- > From: Luca Boccassi [mailto:lboccass at Brocade.com] > Sent: Thursday, July 7, 2016 6:21 PM > To: Lu, Wenzhuo > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 0/4] support reset of VF link > > On Thu, 2016-07-07 at 01:09 +, Lu, Wenzhuo wrote: > > Hi

[dpdk-dev] [PATCH] maintainers: claim responsibility for KNI

2016-07-07 Thread Zhang, Helin
> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, July 7, 2016 7:00 PM > To: dev at dpdk.org > Cc: Zhang, Helin ; Thomas Monjalon > > Subject: [PATCH] maintainers: claim responsibility for KNI > > Signed-off-by: Ferruh Yigit Acked-by: Helin Zhang > --- > MAINTAINERS | 2

[dpdk-dev] [PATCH 2/2] examples/ipsec-secgw: add sample configuration files

2016-07-07 Thread Fan Zhang
This patch adds two sample configuration files to ipsec-secgw sample application. The sample configuration files shows how to set-up systems back-to-back that would forward traffic through an IPsec tunnel. Signed-off-by: Fan Zhang --- examples/ipsec-secgw/ep0.cfg | 119

[dpdk-dev] [PATCH 1/2] examples/ipsec_secgw: add configuration file support

2016-07-07 Thread Fan Zhang
This patch adds the configuration file support to ipsec_secgw sample application. Instead of hard-coded rules, the users can specify their own SP, SA, and routing rules in the configuration file. An command line option "-f" is added to pass the configuration file location to the application.

[dpdk-dev] [PATCH 0/2] examples/ipsec_secgw: add configuration file support

2016-07-07 Thread Fan Zhang
This patchset adds the configuration file supported to ipsec_secgw sample application. Two sample configuration files, ep0.cfg and ep1.cfg are also added to show how to configure two systems back-to-back that would forward traffic through an IPsec tunnel Fan Zhang (2): examples/ipsec_secgw:

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-07-07 Thread Zhe Tao
Problem: When using the TSO + VXLAN feature in i40e, the outer UDP length fields in the multiple UDP segments which are TSOed by the i40e will have the wrong value. Fix this problem by adding the tunnel type field in the i40e descriptor which was missed before. Fixes: 77b8301733c3 ("i40e: VXLAN

[dpdk-dev] [RFC] Generic flow director/filtering/classification API

2016-07-07 Thread Adrien Mazarguil
Hi Lu Wenzhuo, Thanks for your feedback, I'm replying below as well. On Thu, Jul 07, 2016 at 07:14:18AM +, Lu, Wenzhuo wrote: > Hi Adrien, > I have some questions, please see inline, thanks. > > > -Original Message- > > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] >

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-07-07 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin > Sent: Thursday, July 07, 2016 11:51 AM > To: Tao, Zhe; dev at dpdk.org > Cc: Tao, Zhe; Wu, Jingjing > Subject: Re: [dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue > > > Hi Tao, > >

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:45:09PM +0200, Thomas Monjalon wrote: > Neil, > > I would gladly drop this patch from the series if you send a similar one > with your name as maintainer. > > Thanks > Lets get through the rest of this stuff first, then I'll send it when we're complete. Sound good?

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:30PM +0200, Thomas Monjalon wrote: > The author of this feature is Neil Horman. > > Signed-off-by: Thomas Monjalon > --- > MAINTAINERS | 4 > 1 file changed, 4 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index a59191e..f996c2e 100644 > ---

[dpdk-dev] [PATCH 10/11] doc: fix syntax in pmdinfogen guide

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:29PM +0200, Thomas Monjalon wrote: > Sphynx reports this error: > > doc/guides/prog_guide/dev_kit_build_system.rst:337: WARNING: > Pygments lexer name u'C' is not known > > Fixes: 737ddf3fb ("doc: add prog guide section documenting pmdinfo script") > >

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:28PM +0200, Thomas Monjalon wrote: > Remove include of rte_pci.h in the generic header rte_dev.h > and move the macro DRIVER_REGISTER_PCI_TABLE in rte_pci.h. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon >

[dpdk-dev] [PATCH 08/11] pmdinfogen: fix usage message

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:27PM +0200, Thomas Monjalon wrote: > The name of the tool is pmdinfogen. > > Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") > > Signed-off-by: Thomas Monjalon > --- > buildtools/pmdinfogen/pmdinfogen.c | 2 +- > 1 file changed, 1

[dpdk-dev] [PATCH 06/11] mk: fix verbose pmdinfogen run

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:25PM +0200, Thomas Monjalon wrote: > When building with "make V=1" it is expected to see the output of each > compiler command in order to debug them. > Unfortunately the pmdinfogen related commands were always quiet. > > It is fixed by defining the commands in some

[dpdk-dev] [PATCH 04/11] mk: remove traces of hostapp build directory

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:23PM +0200, Thomas Monjalon wrote: > The recipe rte.hostapp.mk does not build in hostapp/ anymore. > > Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") > > Signed-off-by: Thomas Monjalon > --- > doc/guides/freebsd_gsg/build_dpdk.rst | 2 +-

[dpdk-dev] [PATCH] maintainers: claim responsibility for KNI

2016-07-07 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index a59191e..250e542 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -249,7 +249,7 @@ F: app/test/test_link_bonding* F: examples/bond/ Linux KNI -M: Helin

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:22PM +0200, Thomas Monjalon wrote: > When compiling the drivers, some code is generated with pmdinfogen. > A fresh parallel build can fail if a driver is compiled before pmdinfogen: > build/buildtools/dpdk-pmdinfogen: Permission denied > > There was a

[dpdk-dev] [PATCH 02/11] crypto: fix parameters registration

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:21PM +0200, Thomas Monjalon wrote: > The name aesni_gcm was used to register parameters of several > crypto drivers. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon > --- >

[dpdk-dev] [PATCH 01/11] drivers: fix build with new register macro

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:20PM +0200, Thomas Monjalon wrote: > Compilation fails because of some typos. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon > --- > drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 2 +- >

[dpdk-dev] [PATCH v3 2/2] doc: add live migration overview image

2016-07-07 Thread Bernard Iremonger
This patch adds an image of the Live Migration test configuration. Signed-off-by: Bernard Iremonger --- doc/guides/howto/img/lm_overview.svg | 666 ++ doc/guides/howto/lm_bond_virtio_sriov.rst | 4 + 2 files changed, 670 insertions(+) create mode 100644

[dpdk-dev] [PATCH v3 1/2] doc: live migration of VM with Virtio and VF

2016-07-07 Thread Bernard Iremonger
This patch describes the procedure to be be followed to perform Live Migration of a VM with Virtio and VF PMD's using the bonding PMD. It includes sample host and VM scripts used in the procedure, and a sample switch configuration. Signed-off-by: Bernard Iremonger ---

[dpdk-dev] [PATCH v3 0/2] doc: live migration procedure

2016-07-07 Thread Bernard Iremonger
This pachset set describes the procedure to Live migrate a VM with Virtio and VF PMD's using the bonding PMD. Changes in v3: rename directory from how_to to howto move to after FAQ in the index Changes in v2: change primary port before removing slave port remove unused variables from QEMU

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-07 Thread Neil Horman
Recently reported, the introduction of pmd information exports led to a breakage of cryptodev unit tests because the test infrastructure relies on the cryptodev names being available in macros. This patch fixes the pmd naming to use the macro names. Note that the macro names were already

[dpdk-dev] [PATCHv8 2/6] drivers: Update driver registration macro usage

2016-07-07 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > Sent: Friday, June 17, 2016 7:46 PM > To: dev at dpdk.org > Cc: Neil Horman; Richardson, Bruce; Thomas Monjalon; Stephen Hemminger; > Panu Matilainen > Subject: [dpdk-dev] [PATCHv8 2/6] drivers:

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-07-07 Thread Ananyev, Konstantin
Hi Tao, Sorry hit send button too early by accident :) > > > > Problem: > > When using the TSO + VXLAN feature in i40e, the outer UDP length fields in > > the multiple UDP segments which are TSOed by the i40e will have the > > wrong value. > > > > Fix this problem by adding the tunnel type

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-07 Thread Luca Boccassi
On Thu, 2016-07-07 at 01:09 +, Lu, Wenzhuo wrote: > Hi Luca, > > > > -Original Message- > > From: Luca Boccassi [mailto:lboccass at Brocade.com] > > Sent: Thursday, July 7, 2016 12:23 AM > > To: Lu, Wenzhuo > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v6 0/4] support

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-07-07 Thread Ananyev, Konstantin
Hi Tao, > > Problem: > When using the TSO + VXLAN feature in i40e, the outer UDP length fields in > the multiple UDP segments which are TSOed by the i40e will have the > wrong value. > > Fix this problem by adding the tunnel type field in the i40e descriptor > which was missed before. > >

[dpdk-dev] [PATCH v4] Pci: Add the class_id support

2016-07-07 Thread Thomas Monjalon
2016-07-06 12:08, Ferruh Yigit: > On 6/14/2016 3:52 PM, Thomas Monjalon wrote: > > 2016-05-24 20:50, Ziye Yang: > >> This patch is used to add the class_id (class_code, > >> subclass_code, programming_interface) support for > >> pci_device probe. With this patch, it will be > >> flexible for users

[dpdk-dev] [PATCH 09/18] mbuf: support Mpls in software packet type parser

2016-07-07 Thread Liang, Cunming
Hi Olivier, > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Wednesday, July 06, 2016 4:00 PM > To: Liang, Cunming ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 09/18] mbuf: support Mpls in software packet > type parser > > Hi Cunming, > > On

[dpdk-dev] [PATCH v2 1/2] doc: live migration of VM with Virtio and VF

2016-07-07 Thread Iremonger, Bernard
Hi Thomas, > Subject: Re: [dpdk-dev] [PATCH v2 1/2] doc: live migration of VM with Virtio > and VF > > 2016-07-06 17:01, Bernard Iremonger: > > @@ -45,3 +45,4 @@ DPDK documentation > > faq/index > > rel_notes/index > > contributing/index > > + how_to/index > > 2 personal

[dpdk-dev] [PATCH 6/6] l3fwd: add altivec support for em_hash_key

2016-07-07 Thread Gowrishankar
From: gowrishankar This patch adds ppc64le port for em_mask_key function. Signed-off-by: Gowrishankar --- examples/l3fwd/l3fwd_em.c | 8 1 file changed, 8 insertions(+) diff --git a/examples/l3fwd/l3fwd_em.c b/examples/l3fwd/l3fwd_em.c index

[dpdk-dev] [PATCH 5/6] ip_pipeline: fix lcore mapping for varying SMT threads as in ppc64

2016-07-07 Thread Gowrishankar
From: gowrishankar offline lcore would still refer to original core id and this has to be considered while creating cpu core mask. Signed-off-by: Gowrishankar --- examples/ip_pipeline/cpu_core_map.c | 12 +--- examples/ip_pipeline/init.c | 4

[dpdk-dev] [PATCH 4/6] table: cache align rte_bucket_4_8

2016-07-07 Thread Gowrishankar
From: gowrishankar Align rte_bucket_4_8 for cache line. Signed-off-by: Gowrishankar --- lib/librte_table/rte_table_hash_key8.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_table/rte_table_hash_key8.c

[dpdk-dev] [PATCH 2/6] lpm: add altivec intrinsics for dpdk lpm on ppc_64

2016-07-07 Thread Gowrishankar
From: gowrishankar This patch adds ppc64le port for LPM library in DPDK. Signed-off-by: Gowrishankar --- app/test/test_xmmt_ops.h | 16 +++ .../common/include/arch/ppc_64/rte_vect.h | 60 lib/librte_lpm/Makefile

[dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le

2016-07-07 Thread Gowrishankar
From: gowrishankar In this patch, DPDK libraries such as LPM, port, table, pipeline, ACL, and sched are enabled for ppc64le. Signed-off-by: Gowrishankar --- config/defconfig_ppc_64-power8-linuxapp-gcc | 12 ++-- 1 file changed, 6 insertions(+), 6

[dpdk-dev] [PATCH 0/6] enable lpm, acl and other missing libraries in ppc64le

2016-07-07 Thread Gowrishankar
This patchset enables LPM, ACL and other few missing libs in ppc64le and also address few patches in related examples (ip_pipeline and l3fwd). Test report: LPM autotest test results as below. RTE>>lpm_autotest No. routes = 1076806 Route distribution per prefix width: DEPTHQUANTITY

[dpdk-dev] [PATCH 1/6] config: enable lpm, port, table, pipeline, acl, sched libraries for ppc64le

2016-07-07 Thread Gowrishankar
From: gowrishankar In this patch, DPDK libraries such as LPM, port, table, pipeline, ACL, and sched are enabled for ppc64le. Signed-off-by: Gowrishankar --- config/defconfig_ppc_64-power8-linuxapp-gcc | 12 ++-- 1 file changed, 6 insertions(+), 6

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-07 Thread Liang, Cunming
Hi Olivier, > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Wednesday, July 06, 2016 3:43 PM > To: Liang, Cunming ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type > from data > > Hi Cunming, > > On

[dpdk-dev] [PATCHv8 2/6] drivers: Update driver registration macro usage

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 11:00:59AM +, De Lara Guarch, Pablo wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > > Sent: Friday, June 17, 2016 7:46 PM > > To: dev at dpdk.org > > Cc: Neil Horman; Richardson, Bruce; Thomas

[dpdk-dev] Help: How to read packet statistics from device registers via dpdk PMD?

2016-07-07 Thread Jay Rolette
On Thu, Jul 7, 2016 at 12:52 AM, Bill Bonaparte wrote: > I am so happy to get your reply. > My dpdk version is 2.1?and the OS is centOS 7? > the following is the output from "dpdk_nic_bind.py --status": > > [root at APV35 ~]# dpdk_nic_bind.py --status > > Network devices using DPDK-compatible

[dpdk-dev] [RFC] Generic flow director/filtering/classification API

2016-07-07 Thread Lu, Wenzhuo
Hi Adrien, I have some questions, please see inline, thanks. > -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] > Sent: Wednesday, July 6, 2016 2:17 AM > To: dev at dpdk.org > Cc: Thomas Monjalon; Zhang, Helin; Wu, Jingjing; Rasesh Mody; Ajit Khaparde; >

[dpdk-dev] [PATCH] ip_frag: fix doxygen formatting

2016-07-07 Thread Hiroyuki Mikita
This commit fixes some functions missing in API documentation. Signed-off-by: Hiroyuki Mikita --- lib/librte_ip_frag/rte_ip_frag.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/lib/librte_ip_frag/rte_ip_frag.h b/lib/librte_ip_frag/rte_ip_frag.h index

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-07 Thread Lu, Wenzhuo
Hi Luca, > -Original Message- > From: Luca Boccassi [mailto:lboccass at Brocade.com] > Sent: Thursday, July 7, 2016 12:23 AM > To: Lu, Wenzhuo > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 0/4] support reset of VF link > > On Wed, 2016-07-06 at 00:45 +, Lu, Wenzhuo

[dpdk-dev] [PATCHv8 0/6] Implement pmd hardware support exports

2016-07-07 Thread Thomas Monjalon
2016-06-30 08:45, Remy Horton: > On 17/06/2016 19:46, Neil Horman wrote: > > Hey all- > > So heres attempt number 2 at a method for exporting PMD hardware support > > information. As we discussed previously, the consensus seems to be that pmd > > information should be: > [..] > >