[dpdk-dev] [PATCH] lib/librte_sched: Fix compile with gcc 4.3.4

2015-11-26 Thread Thomas Monjalon
2015-11-26 18:49, Michael Qiu: > gcc 4.3.4 does not include "immintrin.h", and will post below error: > lib/librte_sched/rte_sched.c:56:23: error: > immintrin.h: No such file or directory > > To avoid this issue, a gcc version check is need and a flag to indicate > vector ablility. [...]

[dpdk-dev] Problems with SR-IOV and transparent vlan under DPDK

2015-11-26 Thread Tom Kiely
Hi, I wonder if anyone might have advice in relation to a major problem we're encountering. We have an SR-IOV setup on Debian 8 with 4 VFs. Each VF is configured with transparent vlan 1000. We have 4 VMs via KVM. 2xCentos (C1, C2) and 2xDebian8 (VR1, VR2) with DPDK based dataplane. C1

[dpdk-dev] Issues with running multicore

2015-11-26 Thread Bruce Richardson
On Wed, Nov 25, 2015 at 12:59:01PM -0500, Sugumaran, Varthamanan wrote: > Hi All, > Iam trying to create multi core application by mapping separate function for > each core. > I had followed Multi process example and I had used pthread. > The use case is: > > 1. In a single process, launch

[dpdk-dev] [PATCH] app/test: Fix compile issue with icc

2015-11-26 Thread Thomas Monjalon
2015-11-26 15:21, Michael Qiu: > app/test/test_cryptodev_perf.c(1837): error #192: unrecognized > character escape sequence > printf("\n%u\t%u\t\%u\t\t%u\t\t%u", dev_num, 0, > > "\%u" is the root cause of this issue, just fix it. > > Signed-off-by: Michael Qiu Fixes: 202d375c60bc ("app/test:

[dpdk-dev] [PATCH 2/2] Fix compile issue in i686 platform

2015-11-26 Thread Thomas Monjalon
2015-11-26 09:35, Michael Qiu: > In i686 platform, long is 32bit, so XXX_CYCLECOUNTER_MASK > need define as 'ULL' > > Signed-off-by: Michael Qiu This patch is correct but the description is not exact: I have no issue with my i686 compiler. For future reference, please could you be more precise

[dpdk-dev] [PATCH 1/2] examples/distributor: Fix compile issue

2015-11-26 Thread Thomas Monjalon
2015-11-26 09:35, Michael Qiu: > examples/distributor/main.c(338): error #167: > argument of type "struct rte_mbuf *" > is incompatible with parameter of type "const char *" > _mm_prefetch(bufs[0], 0); > > The first param of _mm_prefetch should be "const char *" and > need

[dpdk-dev] API feature check _HAS_

2015-11-26 Thread Thomas Monjalon
When introducing LRO, Vlad has defined the macro RTE_ETHDEV_HAS_LRO_SUPPORT: http://dpdk.org/browse/dpdk/commit/lib/librte_ether/rte_ethdev.h?id=8eecb329 It allows to use the feature without version check (before the release or after a backport). Do you think it is useful? Should we define other

[dpdk-dev] [PATCH] lib/librte_sched: Fix compile with gcc 4.3.4

2015-11-26 Thread Michael Qiu
gcc 4.3.4 does not include "immintrin.h", and will post below error: lib/librte_sched/rte_sched.c:56:23: error: immintrin.h: No such file or directory To avoid this issue, a gcc version check is need and a flag to indicate vector ablility. Signed-off-by: Michael Qiu ---

[dpdk-dev] [PATCH 1/2] examples/distributor: Fix compile issue

2015-11-26 Thread Thomas Monjalon
Hi Michael, Thank you very much for working on compilation issues. Please could you try to insert a "Fixes:" line in your logs?

[dpdk-dev] [PATCH] app/test: Fix compile issue with icc

2015-11-26 Thread Michael Qiu
app/test/test_cryptodev_perf.c(1837): error #192: unrecognized character escape sequence printf("\n%u\t%u\t\%u\t\t%u\t\t%u", dev_num, 0, "\%u" is the root cause of this issue, just fix it. Signed-off-by: Michael Qiu --- app/test/test_cryptodev_perf.c | 2 +- 1 file changed, 1 insertion(+), 1

[dpdk-dev] [PATCH] cryptodev: mark experimental state

2015-11-26 Thread Doherty, Declan
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, November 26, 2015 10:09 AM > To: Panu Matilainen; Doherty, Declan > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] cryptodev: mark experimental state > > 2015-11-26 10:00, Panu

[dpdk-dev] no hugepage with UIO poll-mode driver

2015-11-26 Thread Younghwan Go
Hello, Thank you all for helping us understand on issues with no hugepage option. As Konstantin mentioned at the end, I tried using VFIO module instead of IGB UIO module. I enabled all necessary parameters (IOMMU, virtualization, vfio-pci, VFIO permission) and ran my code with no hugepage

[dpdk-dev] remaining deprecation notices

2015-11-26 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, November 26, 2015 12:44 AM > To: stephen at networkplumber.org; Dumitrescu, Cristian > > Cc: dev at dpdk.org > Subject: remaining deprecation notices > > Stephen, Cristian, > There are

[dpdk-dev] [PATCH] cryptodev: mark experimental state

2015-11-26 Thread Thomas Monjalon
2015-11-26 10:00, Panu Matilainen: > On 11/26/2015 09:39 AM, Panu Matilainen wrote: > > On 11/25/2015 07:38 PM, Thomas Monjalon wrote: > >> --- a/config/common_linuxapp > >> +++ b/config/common_linuxapp > >> @@ -319,6 +319,7 @@ CONFIG_RTE_PMD_PACKET_PREFETCH=y > >> > >> # > >> # Compile

[dpdk-dev] [PATCH v9 1/9] nfp: basic initialization

2015-11-26 Thread Stephen Hemminger
On Thu, 26 Nov 2015 09:49:21 + Alejandro Lucero wrote: > +static inline void > +nfp_qcp_ptr_add(__u8 *q, enum nfp_qcp_ptr ptr, uint32_t val) > +{ > + uint32_t off; Minor nit. why mix use of Linux specific basic size typedefs (__u8) with Posix standard values (uint32_t). The DPDK style

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

2015-11-26 Thread Panu Matilainen
On 11/25/2015 06:08 PM, Stephen Hemminger wrote: > On Wed, 25 Nov 2015 10:38:48 +0200 > Panu Matilainen wrote: > >> On 11/25/2015 12:46 AM, Stephen Hemminger wrote: >>> On Tue, 24 Nov 2015 16:31:17 +0200 >>> Panu Matilainen wrote: >>> The physically linked-together combined library has been

[dpdk-dev] [PATCH] cryptodev: mark experimental state

2015-11-26 Thread Panu Matilainen
On 11/26/2015 09:39 AM, Panu Matilainen wrote: > On 11/25/2015 07:38 PM, Thomas Monjalon wrote: >> The crypto API is in an early state. >> It requires more discussions and experiments to declare it stable, >> as discussed in http://dpdk.org/ml/archives/dev/2015-November/028634.html >> >> A

[dpdk-dev] [PATCH v2] examples/bond: fix bsd compile error

2015-11-26 Thread Mrzyglod, DanielX T
>-Original Message- >From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit >Sent: Wednesday, November 25, 2015 6:41 PM >To: dev at dpdk.org >Subject: [dpdk-dev] [PATCH v2] examples/bond: fix bsd compile error > >Error: >== bond > CC main.o >/.../examples/bond/main.c:431:24:

[dpdk-dev] [PATCH v9 8/9] nfp: adding nic guide

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- doc/guides/nics/index.rst |1 + doc/guides/nics/nfp.rst | 265 + 2 files changed, 266 insertions(+) create mode 100644 doc/guides/nics/nfp.rst diff --git

[dpdk-dev] [PATCH v9 7/9] nfp: link status change interrupt support

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 123 + 1 file changed, 123 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index 5383f51..3763790 100644 ---

[dpdk-dev] [PATCH v9 6/9] nfp: adding extra functionality

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 191 + 1 file changed, 191 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index 567ea26..5383f51 100644 ---

[dpdk-dev] [PATCH v9 5/9] nfp: adding link functionality

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 96 + 1 file changed, 96 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index fc02916..567ea26 100644 ---

[dpdk-dev] [PATCH v9 4/9] nfp: adding stats

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 179 + 1 file changed, 179 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index 8451a49..fc02916 100644 ---

[dpdk-dev] [PATCH v9 3/9] nfp: adding rss

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 218 + 1 file changed, 218 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index 7e30774..8451a49 100644 ---

[dpdk-dev] [PATCH v9 2/9] nfp: adding rx/tx functionality

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- drivers/net/nfp/nfp_net.c | 993 + 1 file changed, 993 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index 18067c0..7e30774 100644 ---

[dpdk-dev] [PATCH v9 1/9] nfp: basic initialization

2015-11-26 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero Signed-off-by: Rolf Neugebauer --- MAINTAINERS |3 + config/common_linuxapp |6 + doc/guides/rel_notes/release_2_2.rst|4 + drivers/net/Makefile|1 + drivers/net/nfp/Makefile

[dpdk-dev] [PATCH v9 0/9] support for netronome nfp-6xxx card

2015-11-26 Thread Alejandro Lucero
This patchset adds a new PMD for Netronome nfp-6xxx card. Just PCI Virtual Functions supported. Using this PMD requires previous Netronome BSP installation. v9: - Adding flag RTE_PCI_DRV_INTR_LSC - Makefile changes for compilation as a shared library - Adding map file for linker version script

[dpdk-dev] [PATCH] cryptodev: mark experimental state

2015-11-26 Thread Panu Matilainen
On 11/25/2015 07:38 PM, Thomas Monjalon wrote: > The crypto API is in an early state. > It requires more discussions and experiments to declare it stable, > as discussed in http://dpdk.org/ml/archives/dev/2015-November/028634.html > > A documentation section will be required in the guides. > >

[dpdk-dev] no hugepage with UIO poll-mode driver

2015-11-26 Thread Kyle Larose
On Wed, Nov 25, 2015 at 11:47 PM, Younghwan Go wrote: > Hello, > > Thank you all for helping us understand on issues with no hugepage option. > > As Konstantin mentioned at the end, I tried using VFIO module instead of IGB > UIO module. I enabled all necessary parameters (IOMMU, virtualization, >

[dpdk-dev] [PATCH 2/2] Fix compile issue in i686 platform

2015-11-26 Thread Michael Qiu
In i686 platform, long is 32bit, so XXX_CYCLECOUNTER_MASK need define as 'ULL' Signed-off-by: Michael Qiu --- drivers/net/e1000/igb_ethdev.c | 2 +- drivers/net/i40e/i40e_ethdev.c | 2 +- drivers/net/ixgbe/ixgbe_ethdev.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git

[dpdk-dev] [PATCH 1/2] examples/distributor: Fix compile issue

2015-11-26 Thread Michael Qiu
examples/distributor/main.c(338): error #167: argument of type "struct rte_mbuf *" is incompatible with parameter of type "const char *" _mm_prefetch(bufs[0], 0); The first param of _mm_prefetch should be "const char *" and need convert "struct rte_mbuf *" to "const void

[dpdk-dev] [dpdk-announce] release candidate 2.2.0-rc2

2015-11-26 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: http://dpdk.org/browse/dpdk/tag/?id=v2.2.0-rc2 Changelog (main changes since 2.2.0-rc1) - enhancements: * ARM v7 * ARM v8 * option -d works with directories * keep

[dpdk-dev] ABI changes in release notes

2015-11-26 Thread Thomas Monjalon
Hi all, I've just noticed that some ABI changes were announced and done but the corresponding entries in the release notes have not been added. Please check.