[dpdk-dev] [PATCH] vmxnet3: fix Rx deadlock

2016-11-14 Thread Stefan Puiu
to crash. Signed-off-by: Stefan Puiu --- drivers/net/vmxnet3/vmxnet3_rxtx.c | 38 -- 1 file changed, 36 insertions(+), 2 deletions(-) diff --git a/drivers/net/vmxnet3/vmxnet3_rxtx.c b/drivers/net/vmxnet3/vmxnet3_rxtx.c index b109168..c9d2488 100644

[dpdk-dev] [PATCH] vmxnet3: fix Rx deadlock

2016-11-14 Thread Stefan Puiu
Our use case is that we have an app that needs to keep mbufs around for a while. We've seen cases when calling vmxnet3_post_rx_bufs() from vmxet3_recv_pkts(), it might not succeed to add any mbufs to any RX descriptors (where it returns -err). Since there are no mbufs that the virtual hardware can

[dpdk-dev] including rte.app.mk from a Makefile.am

2016-02-24 Thread Stefan Puiu
On Wed, Feb 24, 2016 at 3:26 AM, Thomas Monjalon wrote: > Yes it is prefixed when used instead of assignment. > In 2.2, it is better fixed: > > ifeq ($(LINK_USING_CC),1) > override EXTRA_LDFLAGS := $(call linkerprefix,$(EXTRA_LDFLAGS)) > O_TO_EXE = $(CC) $(CFLAGS) $(LDFLAGS_$(@)) \ >

[dpdk-dev] including rte.app.mk from a Makefile.am

2016-02-23 Thread Stefan Puiu
Hi, I'm working on a Linux project that uses the DPDK and (unfornately, IMO) automake; so we have a Makefile.am where we include rte.extapp.mk and rte.vars.mk from the DPDK, add LDLIBS to the linker However, I've tried building against DPDK 2.2 and I'm getting linker errors about options like

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Stefan Puiu
Hi, On Wed, Mar 18, 2015 at 2:59 PM, Thomas Monjalon wrote: > Hi Sergio, > > Thank you for explaining the situation. > > 2015-03-18 12:11, Gonzalez Monroy, Sergio: >> Given that the patch to remove combined libraries is not welcome, I'll >> try to explain the current situation so we can agree on

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Stefan Puiu
Hi, 2 cents from a DPDK library user - I make 2 changes to the default linux+gcc configuration: combine libraries and build shared libraries (since I want 2 instances of the app, it didn't make sense to me to link statically). I tried working with the individual libs, but adding all of them with

[dpdk-dev] [PATCH 2/2] Headers: more C++11 compilation fixes

2015-02-20 Thread Stefan Puiu
Fixed two more places where non-C++11 string concatenation is used. Spotted by John McNamara. Signed-off-by: Stefan Puiu --- lib/librte_mempool/rte_mempool.h |8 lib/librte_vhost/vhost-net-cdev.h |4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lib

[dpdk-dev] [PATCH 1/2] rte_pci.h: Fix C++11 compilation

2015-02-20 Thread Stefan Puiu
-defined-literal] Signed-off-by: Stefan Puiu --- lib/librte_eal/common/include/rte_pci.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h index 66ed793..12ae5a7 100644 --- a/lib/librte_eal

[dpdk-dev] [PATCH] rte_pci.h: Fix C++11 compilation

2015-02-20 Thread Stefan Puiu
se check http://dpdk.org/dev#send for versioning and set the old versions > to "superseded" in patchwork. > Thanks > > > 2015-02-20 11:35, Stefan Puiu: >> In C++11 concatenated string literals need to have a >> space in between. Found with clang++-3.4, IIRC g++-4.8 al

[dpdk-dev] [PATCH] rte_pci.h: Fix C++11 compilation

2015-02-20 Thread Stefan Puiu
-defined-literal] Signed-off-by: Stefan Puiu --- lib/librte_eal/common/include/rte_pci.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h index 66ed793..12ae5a7 100644 --- a/lib/librte_eal

[dpdk-dev] [PATCH] Fix compilation error when compiling with clang-3.4 in C++11 mode: in C++11 concatenated string literals need to have a space in between.

2015-02-20 Thread Stefan Puiu
Hi Pablo, On Fri, Feb 20, 2015 at 10:19 AM, De Lara Guarch, Pablo wrote: > Hi Stefan, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stefan Puiu >> Sent: Thursday, February 19, 2015 8:59 PM >> To: dev at dpdk.org >&g

[dpdk-dev] [PATCH] Fix compilation error when compiling with clang-3.4 in C++11 mode: in C++11 concatenated string literals need to have a space in between.

2015-02-19 Thread Stefan Puiu
From: Stefan Puiu <sp...@ixiacom.com> Sample error message: dpdk/include/rte_pci.h:96:26: error: invalid suffix on literal; C++11 requires a space between literal and identifier [-Wreserved-user-defined-literal] --- lib/librte_eal/common/include/rte_pci.h |4 ++-- 1 file chan

[dpdk-dev] [PATCH] Fix C++11 compilation error with rte_pci.h

2015-02-19 Thread Stefan Puiu
] Signed-off-by: Stefan Puiu --- lib/librte_eal/common/include/rte_pci.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h index 66ed793..12ae5a7 100644 --- a/lib/librte_eal/common/include

[dpdk-dev] [PATCH] Minor C++11 compilation fix for rte_pci.h

2015-02-11 Thread Stefan Puiu
-by: Stefan Puiu --- lib/librte_eal/common/include/rte_pci.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h index 66ed793..12ae5a7 100644 --- a/lib/librte_eal/common/include/rte_pci.h +++ b

[dpdk-dev] upper limit on the size of allocation through rte_malloc in dpdk-1.8.0?

2015-02-10 Thread Stefan Puiu
Hi and thanks for replying, On Fri, Feb 6, 2015 at 1:25 PM, Olivier MATZ wrote: > Hi, > > On 02/06/2015 12:00 PM, Bruce Richardson wrote: >> On Wed, Feb 04, 2015 at 05:24:58PM +0200, Stefan Puiu wrote: >>> Hi, >>> >>> I'm trying to alter an existing pr

[dpdk-dev] upper limit on the size of allocation through rte_malloc in dpdk-1.8.0?

2015-02-04 Thread Stefan Puiu
Hi, I'm trying to alter an existing program to use the Intel DPDK. I'm using 1.8.0, compiled by me as a shared library (CONFIG_RTE_BUILD_COMBINE_LIBS=y and CONFIG_RTE_BUILD_SHARED_LIB=y in .config) on Ubuntu 12.04. The program needs to allocate large blocks of memory (between 1 and 4 chunks of