[dpdk-dev] Can't build DPDK-16.04 on CentOS 6.8

2016-06-28 Thread Martinx - ジェームズ
On 16 June 2016 at 11:31, Ferruh Yigit wrote: > On 6/15/2016 7:59 PM, Martinx - ? wrote: > > On 13 June 2016 at 12:15, Martinx - ? > > mailto:thiagocmartinsc at gmail.com>> wrote: > > > > > > > > On 12 June 2016 at 11:30, Rosen, Rami > > wrote: >

[dpdk-dev] [RFC] mk: filter duplicate configuration entries

2016-06-28 Thread Christian Ehrhardt
On Tue, Jun 28, 2016 at 6:11 PM, Ferruh Yigit wrote: > On 6/13/2016 4:10 PM, Christian Ehrhardt wrote: > > Due to the hierarchy and the demand to keep the base config shoing all > > options some options end up multiple times in the .config file. > > > > A suggested solution was to filter for

[dpdk-dev] Fwd: dpdk ixgbe PMD lro limits

2016-06-28 Thread Asim Jamshed
On Tue, Jun 28, 2016 at 5:42 PM, Lu, Wenzhuo wrote: > Hi Asim, > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Asim Jamshed >> Sent: Tuesday, June 28, 2016 2:41 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] Fwd: dpdk ixgbe PMD lro limits >> >> Hi,

[dpdk-dev] [PATCH v4 3/3] mempool: allow for user-owned mempool caches

2016-06-28 Thread Lazaros Koromilas
Hi Olivier, thanks for fixing those, just one comment below On Mon, Jun 27, 2016 at 4:50 PM, Olivier Matz wrote: > From: Lazaros Koromilas > > The mempool cache is only available to EAL threads as a per-lcore > resource. Change this so that the user can create and provide their own > cache on

[dpdk-dev] [PATCH v4 1/2] ethdev: remove get_reg_length callback

2016-06-28 Thread Zyta Szpak
OK On 27.06.2016 17:19, Thomas Monjalon wrote: > 2016-06-23 15:26, zr at semihalf.com: >> From: Zyta Szpak >> >> Version 4 of fixing the assumption of that device registers >> are always 32 bits long. rte_eth_dev_get_reg_length and >> rte_eth_dev_get_reg_info callbacks did not provide register

[dpdk-dev] [PATCH v4 1/2] ethdev: remove get_reg_length callback

2016-06-28 Thread Zyta Szpak
OK On 27.06.2016 12:46, Remy Horton wrote: > Morning, > > > On 23/06/2016 14:26, zr at semihalf.com wrote: >> From: Zyta Szpak >> >> Version 4 of fixing the assumption of that device registers >> are always 32 bits long. rte_eth_dev_get_reg_length and >> rte_eth_dev_get_reg_info callbacks did

[dpdk-dev] [RFC] mk: filter duplicate configuration entries

2016-06-28 Thread Ferruh Yigit
On 6/28/2016 5:38 PM, Christian Ehrhardt wrote: > On Tue, Jun 28, 2016 at 6:11 PM, Ferruh Yigit > wrote: > > On 6/13/2016 4:10 PM, Christian Ehrhardt wrote: > > Due to the hierarchy and the demand to keep the base config shoing all > > options some

[dpdk-dev] [PATCH v2] ip_pipeline: add Python script file for creating visual diagram of IP pipeline config file

2016-06-28 Thread Jasvinder Singh
This commit adds Python script for generating diagram of the application configuration file. This script requires graphviz package to be installed on the machine. The input config file is translated to an output file in DOT syntax, which is then used to create the image file using graphviz. To

[dpdk-dev] [RFC] mk: filter duplicate configuration entries

2016-06-28 Thread Ferruh Yigit
On 6/13/2016 4:10 PM, Christian Ehrhardt wrote: > Due to the hierarchy and the demand to keep the base config shoing all > options some options end up multiple times in the .config file. > > A suggested solution was to filter for duplicates at the end of the > actual config step which is

[dpdk-dev] [PATCH] net/virtio-user: fix O_CLOEXEC undeclared error

2016-06-28 Thread Yuanhan Liu
On Sun, Jun 26, 2016 at 01:49:46PM +, Jianfeng Tan wrote: > On some older systems, such as SUSE 11, the compiling error shows > as: >.../dpdk/drivers/net/virtio/virtio_user/virtio_user_dev.c:67:22: > error: ?O_CLOEXEC? undeclared (first use in this function) > > The fix is to

[dpdk-dev] [PATCH v2] ip_pipeline: add Python script file for creating visual diagram of IP pipeline config file

2016-06-28 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jasvinder Singh > Sent: Tuesday, June 28, 2016 5:22 PM > To: dev at dpdk.org > Cc: Dumitrescu, Cristian > Subject: [dpdk-dev] [PATCH v2] ip_pipeline: add Python script file for > creating visual diagram of IP

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

2016-06-28 Thread Thomas Monjalon
2016-05-11 17:06, Rasesh Mody: > Add extended stats support to provide additional statistical info. > > Signed-off-by: Rasesh Mody > Signed-off-by: Harish Patil This bnx2x patch and a qede one cannot be merged in master tree because of an API change: http://dpdk.org/commit/bd6aa172

[dpdk-dev] [PATCH v2 6/9] qede: add support for xstats

2016-06-28 Thread Thomas Monjalon
2016-06-15 22:47, Rasesh Mody: > This patch adds support for extended statistics for QEDE PMD. > > Signed-off-by: Rasesh Mody This qede patch and a bnx2x one cannot be merged in master tree because of an API change: http://dpdk.org/commit/bd6aa172 http://dpdk.org/commit/e2aae1c1

[dpdk-dev] Fwd: dpdk ixgbe PMD lro limits

2016-06-28 Thread Asim Jamshed
Hi, Apologies in advance if this question has been asked in the past. I have been performing mTCP-related tests on dpdk-16.04 with ixgbe PMD. I am using 82599ES 10-Gigabit adapters for my experiments. I have a few queries regarding LRO. 1) What is the theoretical maximum size of the Ethernet

[dpdk-dev] [PATCH 0/2] support mailbox interruption on ixgbe/igb VF

2016-06-28 Thread Luca Boccassi
On Tue, 2016-06-28 at 10:16 +, Luca Boccassi wrote: > On Tue, 2016-05-24 at 14:06 +0800, Wenzhuo Lu wrote: > > This patch set addes the support of the mailbox interruption on VF. > > So, VF can receice the messges for physical link down/up. > > > > PS: This patch set is splitted from a

[dpdk-dev] [PATCH 3/4] virtio: move SSE based Rx implementation to separate file

2016-06-28 Thread Jianbo Liu
On 27 June 2016 at 19:54, Jerin Jacob wrote: > split out SSE instruction based virtio simple rx > implementation to a separate file > > Signed-off-by: Jerin Jacob > --- > drivers/net/virtio/virtio_rxtx_simple.c | 166 +--- > drivers/net/virtio/virtio_rxtx_simple_sse.h | 225

[dpdk-dev] [PATCH v2 0/2] fix coverity issues

2016-06-28 Thread Bruce Richardson
On Sun, Jun 26, 2016 at 11:46:16PM +0800, Helin Zhang wrote: > It fixes several problematic dereference issues and missing > break issue reported by Coverity. > > Helin Zhang (2): > i40e: fix problematic dereference > i40e: fix missing break in switch > > drivers/net/i40e/i40e_pf.c | 11

[dpdk-dev] [PATCH] scripts: check for commit msgs starting with 'It'

2016-06-28 Thread Bruce Richardson
Signed-off-by: Bruce Richardson --- scripts/check-git-log.sh | 6 ++ 1 file changed, 6 insertions(+) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index 153f378..87b122f 100755 --- a/scripts/check-git-log.sh +++ b/scripts/check-git-log.sh @@ -114,6 +114,12 @@ bad=$(echo

[dpdk-dev] [PATCH] bnx2x: Call bnx2x_init_rte() later

2016-06-28 Thread Chas Williams
We need sc->igu_sb_cnt determined before calculating the number of queues we can support. Fixes: a787538ee754 ("bnx2x: fix MSIX vector and VF resource counts") Signed-off-by: Chas Williams <3chas3 at gmail.com> --- drivers/net/bnx2x/bnx2x.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[dpdk-dev] [PATCH] enic: fix issues when using Rx scatter with multiple RQs

2016-06-28 Thread Bruce Richardson
On Fri, Jun 24, 2016 at 03:39:57PM -0700, Nelson Escobar wrote: > The Rx scatter patch failed to make a few changes and resulted > in problems when using multiple RQs since the wrong RQ or CQ > was being used. > I think you might need to expand on what the "few changes" are. Also, please expand

[dpdk-dev] [PATCH v2] enic: fix free function to actually free memory

2016-06-28 Thread Bruce Richardson
On Thu, Jun 23, 2016 at 04:14:58PM -0700, Nelson Escobar wrote: > enic_alloc_consistent() allocated memory, but enic_free_consistent() > was an empty function, so allocated memory was never freed. > > This commit adds a list and lock to the enic structure to keep track > of the memzones allocated

[dpdk-dev] [PATCH 3/3] vhost: fix potential NULL pointer dereference

2016-06-28 Thread Yuanhan Liu
Fix the potential NULL pointer dereference issue raised by Coverity. 578 reconn = malloc(sizeof(*reconn)); >>> CID 127481: Null pointer dereferences (NULL_RETURNS) >>> Dereferencing a null pointer "reconn". 579 reconn->un = un; Coverity issue:

[dpdk-dev] [PATCH 2/3] vhost: fix not null terminated string

2016-06-28 Thread Yuanhan Liu
Fix an issue raised by Coverity. >>> CID 127475: Memory - illegal accesses (BUFFER_SIZE_WARNING) >>> Calling strncpy with a maximum size argument of 108 bytes on >>> destination array "un->sun_path" of size 108 bytes might leave >>> the destination string

[dpdk-dev] [PATCH 1/3] vhost: fix memory leak

2016-06-28 Thread Yuanhan Liu
Fix potential memory leak raised by Coverity. >>> Variable "vsocket" going out of scope leaks the storage it >>> points to. Coverity issue: 127483 Fixes: e623e0c6d8a5 ("vhost: add reconnect ability") Reported-by: John McNamara Signed-off-by: Yuanhan Liu ---

[dpdk-dev] [PATCH 0/3] vhost coverity issue fixes

2016-06-28 Thread Yuanhan Liu
This is a small series fixes 3 coverity issues. John, I'm wondering maybe maybe we could add the next-net and next-virtio tree into the coverity test as well? So that we could catch those errors as earlier as possible, say, at least before they got merged into mainline. --- Yuanhan Liu (3):

[dpdk-dev] [PATCH v2] enic: fix issues when using Rx scatter with multiple RQs

2016-06-28 Thread Nelson Escobar
The Rx scatter patch failed to make a few changes and resulted in problems when using multiple receive queues in dpdk (ie RSS) since the wrong adapter resources were being used. - get and use the correct completion queue index associated with a receive queue. - set the correct receive queue

[dpdk-dev] [Crypto-API query]

2016-06-28 Thread Anupam Kapoor
> [2016-06-28T11:39:08+0530]: "amartya.das" (amartya.das): ,[ amartya.das ] | EAL: no driver found for cryptodev_aesni_mb_pmd | EAL: failed to initialize cryptodev_aesni_mb_pmd device ` i have a _hunch_ that you have might have missed building the multi-buffer-crypto-pmd... for more

[dpdk-dev] [Crypto-API query]

2016-06-28 Thread Anupam Kapoor
> [2016-06-28T10:57:02+0530]: "amartya.das" (amartya.das): ,[ amartya.das ] | Is it possible to use Crypto API with virtual device without VFIO and by using UIO only. ` i have just taken a cursory look at the l2fwd-crypto, and it seems it is possible to have sw-only crypto devices...

[dpdk-dev] [PATCH v2] enic: fix name of hash table used for enic classifiers

2016-06-28 Thread Bruce Richardson
On Thu, Jun 23, 2016 at 04:10:02PM -0700, Nelson Escobar wrote: > The enic_clsf_init() function is called once per enic instance, but it > used a static name to create the hash table. Consequently when using > more than one enic instance, there was a name collision which caused > errors: > >

[dpdk-dev] [PATCH v3 4/5] testpmd: handle all rxqs in rss setup

2016-06-28 Thread Wang, Zhihong
Thanks Nelio and Pablo! > -Original Message- > From: N?lio Laranjeiro [mailto:nelio.laranjeiro at 6wind.com] > Sent: Tuesday, June 28, 2016 4:34 PM > To: De Lara Guarch, Pablo > Cc: Wang, Zhihong ; dev at dpdk.org; Ananyev, > Konstantin ; Richardson, Bruce > ; thomas.monjalon at

[dpdk-dev] [PATCH v3] mlx: fix icc compilation error

2016-06-28 Thread Bruce Richardson
On Thu, Jun 16, 2016 at 09:58:23AM +0200, Adrien Mazarguil wrote: > On Tue, Jun 14, 2016 at 05:17:24PM +0100, Ferruh Yigit wrote: > > Compilation errors: > > mlx4: > > CC mlx4.o > > .../dpdk/drivers/net/mlx4/mlx4.c(5409): error #188: enumerated type > > mixed with another type > >

[dpdk-dev] [PATCH] app/testpmd: limit number of forwarding cores

2016-06-28 Thread Nélio Laranjeiro
On Mon, Jun 27, 2016 at 11:35:19PM +0100, Pablo de Lara wrote: > Number of forwarding cores must be equal or less than > number of forwarding streams, otherwise two cores > would try to use a same queue on a port, which is not allowed. > > Fixes: f2bb7ae1d204 ("app/testpmd: handle all Rx queues

[dpdk-dev] [PATCH] net/virtio-user: fix O_CLOEXEC undeclared error

2016-06-28 Thread Ferruh Yigit
On 6/26/2016 2:49 PM, Jianfeng Tan wrote: > On some older systems, such as SUSE 11, the compiling error shows > as: >.../dpdk/drivers/net/virtio/virtio_user/virtio_user_dev.c:67:22: > error: ?O_CLOEXEC? undeclared (first use in this function) > > The fix is to declare _GNU_SOURCE

[dpdk-dev] [PATCH v3 4/5] testpmd: handle all rxqs in rss setup

2016-06-28 Thread Nélio Laranjeiro
Hi Pablo, On Mon, Jun 27, 2016 at 10:36:38PM +, De Lara Guarch, Pablo wrote: > Hi Nelio, > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of N?lio Laranjeiro > > Sent: Monday, June 27, 2016 3:24 PM > > To: Wang, Zhihong > > Cc: dev at dpdk.org;

[dpdk-dev] [PATCH 0/2] support mailbox interruption on ixgbe/igb VF

2016-06-28 Thread Luca Boccassi
On Tue, 2016-05-24 at 14:06 +0800, Wenzhuo Lu wrote: > This patch set addes the support of the mailbox interruption on VF. > So, VF can receice the messges for physical link down/up. > > PS: This patch set is splitted from a previous patch set, *automatic > link recovery on ixgbe/igb VF*. > >

[dpdk-dev] [PATCH] app/testpmd: limit number of forwarding cores

2016-06-28 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Monday, June 27, 2016 11:35 PM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [dpdk-dev] [PATCH] app/testpmd: limit number of forwarding cores > > Number of forwarding

[dpdk-dev] [PATCH 2/2] net/thunderx: fix unused function nicvf_mbox_msg_str error

2016-06-28 Thread Ferruh Yigit
On 6/27/2016 5:16 PM, Jerin Jacob wrote: > Fix the following error with clang > error: unused function 'nicvf_mbox_msg_str' > > The nicvf_mbox_msg_str() maybe unused, based on DEBUG > compilation option selected. Mark __attribute__((unused)) on > this function, to inform compiler that this

[dpdk-dev] [PATCH 1/2] net/thunderx: select -fno-prefetch-loop-arrays only for gcc

2016-06-28 Thread Ferruh Yigit
On 6/27/2016 5:16 PM, Jerin Jacob wrote: > 'fno-prefetch-loop-arrays' optimization flag is not supported with clang > > Fixes: dcc49041 (net/thunderx: add single and multi-segment Tx functions") > > Signed-off-by: Jerin Jacob > Reported-by: Ferruh Yigit check-git-log gives following errors:

[dpdk-dev] [PATCH] cmdline: fix unchecked return value

2016-06-28 Thread Mrzyglod, DanielX T
>From: Olivier Matz [mailto:olivier.matz at 6wind.com] >Sent: Monday, May 02, 2016 3:37 PM >To: Mrzyglod, DanielX T ; dev at dpdk.org >Subject: Re: [PATCH] cmdline: fix unchecked return value > >Hi Daniel, > >On 04/14/2016 03:01 PM, Daniel Mrzyglod wrote: >> This patch is for checking if error

[dpdk-dev] [PATCH] examples/l2fwd-crypto: fix incorrect stats array length

2016-06-28 Thread Declan Doherty
On 27/06/16 14:37, Pablo de Lara wrote: > crypto_statistics array was not big enough for storing > all the possible crypto device statistics, as its size was > RTE_MAX_ETHPORTS, but should be RTE_CRYPTO_MAX_DEVS, leading > this to a potential out-of-bounds issue. > > Coverity issue: 120145 > >

[dpdk-dev] Fwd: dpdk ixgbe PMD lro limits

2016-06-28 Thread Lu, Wenzhuo
Hi Asim, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Asim Jamshed > Sent: Tuesday, June 28, 2016 2:41 PM > To: dev at dpdk.org > Subject: [dpdk-dev] Fwd: dpdk ixgbe PMD lro limits > > Hi, > > Apologies in advance if this question has been asked in the

[dpdk-dev] Ignoring number of bytes read in eal

2016-06-28 Thread Kobylinski, MichalX
Hi David, I'm working on issue reported by Coverity. CID 13212 - Ignoring number of bytes read: The number of bytes copied into the buffer can be smaller than the requested number and the buffer can potentially be accessed out of range. In rte_mem_virt2phy: Value returned from a function and

[dpdk-dev] [Crypto-API]

2016-06-28 Thread Jain, Deepak K
HI Amartya, Please see answers below. > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of > amartya.das at wipro.com > Sent: Tuesday, June 28, 2016 6:26 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [Crypto-API] > > Hi, > > Is it possible to use Crypto API

[dpdk-dev] [Crypto-API query]

2016-06-28 Thread amartya....@wipro.com
It did compiled by make YASM=/usr/bin/yasm for in /home/test/ipsec_043/code: Making object file obj/gcm_avx_gen4.o /usr/bin/yasm -f x64 -f elf64 -X gnu -g dwarf2 -D LINUX -D __linux__ -I include -I . avx2/gcm_avx_gen4.asm -o obj/gcm_avx_gen4.o

[dpdk-dev] [Crypto-API query]

2016-06-28 Thread amartya....@wipro.com
Hi, Thanks for the prompt response. I am trying to run one sample application with crypto library with DPDK 16.04 from below path in VM: dpdk-16.04/examples/l2fwd-crypto I have followed below link: http://dpdk.org/doc/guides-16.04/cryptodevs/aesni_mb.html Command use: ./build/l2fwd-crypto -c

[dpdk-dev] [Crypto-API query]

2016-06-28 Thread amartya....@wipro.com
Hi, Is it possible to use Crypto API with virtual device without VFIO and by using UIO only. Thanks, Amartya The information contained in this electronic message and any attachments to this message are intended for the exclusive use of the addressee(s) and may contain proprietary,

[dpdk-dev] [Crypto-API]

2016-06-28 Thread amartya....@wipro.com
Hi, Is it possible to use Crypto API with virtual device without VFIO and by using UIO only. Thanks, Amartya The information contained in this electronic message and any attachments to this message are intended for the exclusive use of the addressee(s) and may contain proprietary,

[dpdk-dev] [PATCH] app/testpmd: limit number of forwarding cores

2016-06-28 Thread Pablo de Lara
Number of forwarding cores must be equal or less than number of forwarding streams, otherwise two cores would try to use a same queue on a port, which is not allowed. Fixes: f2bb7ae1d204 ("app/testpmd: handle all Rx queues in RSS setup") Signed-off-by: Pablo de Lara --- app/test-pmd/config.c |

[dpdk-dev] [PATCH v2] bnx2x: fix icc compilation error

2016-06-28 Thread Rasesh Mody
> From: Ferruh Yigit [mailto:ferruh.yigit at intel.com] > Sent: Monday, June 27, 2016 9:03 AM > > Fix icc warning: > error #188: enumerated type mixed with another type > > Fixed the code for simple cases caused the warning. > Still need to disable compiler warning because of more complex case.