[dpdk-dev] FYI: Using ccache a linux compiler caching tool with DPDK

2016-06-27 18:10, Wiles, Keith: > It appears the ccache and clang have a few problems without the global having > the environment variable set: > export CCACHE_CPP2=yes > > Without this environment variable clang starts to complain about a number of > issues. If you see a compile problem please

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

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; Ananyev, Konstantin; Richardson, Bruce; De Lara Guarch, > Pablo; thomas.monjalon at 6wind.com >

[dpdk-dev] [PATCH v2] examples/l3fwd: update usage and documentation

Update l3fwd example usage and documentation with missing options. Signed-off-by: Beilei Xing --- v2 changes: Update l3fwd main.c usage Update format in documentation. doc/guides/sample_app_ug/l3_forward.rst | 42 ++--- examples/l3fwd/main.c | 40

[dpdk-dev] [PATCH 2/4] virtio: introduce RTE_LIBRTE_VIRTIO_INC_VECTOR

On Mon, Jun 27, 2016 at 04:19:57PM +0200, Thomas Monjalon wrote: > 2016-06-27 17:24, Jerin Jacob: > > --- a/config/common_base > > +++ b/config/common_base > > @@ -267,6 +267,7 @@ CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_RX=n > > CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_TX=n > >

[dpdk-dev] [RFC] librte_vhost: Add unix domain socket fd registration

On Fri, Jun 24, 2016 at 08:23:52AM -0400, Aaron Conole wrote: > Is there still merit to this patch, given above? If so, I'd finish my > integration and testing work and submit it formally. Sorry, I don't see the strong need of this patch (at least so far), judging that vhost-user as the client

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

2016-06-27 16:51, Wiles, Keith: > On 6/27/16, 11:40 AM, "Richardson, Bruce" > wrote: > >On Mon, Jun 27, 2016 at 05:29:59PM +0100, Wiles, Keith wrote: > >> On 6/27/16, 7:58 AM, on behalf of keith.wiles at intel.com> wrote: > >> >On 6/27/16, 3:46 AM, "Richardson, Bruce" > >> >wrote: > >> >>On

[dpdk-dev] weak functions in some drivers

On 6/21/2016 4:01 PM, Damjan Marion (damarion) wrote: > > Hello, > ... > > What is not clear to me is motivation to use weak here instead of simply > using CONFIG_RTE_I40E_INC_VECTOR > macro to exclude stubs in i40e_rxtx.c. It will make library smaller and avoid > issues like this one >

[dpdk-dev] dpdk config on VM

hi, check with the below link .With little changes you can make it work for your use case . https://github.com/01org/dpdk-ovs/blob/development/docs/04_Sample_Configurations/02_Userspace-vHost.md On Mon, Jun 27, 2016 at 2:50 PM, Raja Jayapal wrote: > Hi All, > > I am tyrying to install dpdk on

[dpdk-dev] FYI: Using ccache a linux compiler caching tool with DPDK

FYI, Just to help document the issue here. Using ccache on Linux can improve your compile time from a few minutes to a few seconds depending the use case. On my machine Ubuntu 16.04 up to date as of 06/26/2016 using: sudo apt-get install ccache System Information: Model Name : Intel(R)

[dpdk-dev] [PATCH 2/3] cmdline: remove duplicated symbols from .map

On 06/27/2016 03:02 PM, Ferruh Yigit wrote: > Fixes: 9d41beed24b0 ("lib: provide initial versioning") > Signed-off-by: Ferruh Yigit > --- > lib/librte_cmdline/rte_cmdline_version.map | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/lib/librte_cmdline/rte_cmdline_version.map >

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

Hi Lazaros, On 06/27/2016 05:50 PM, Olivier Matz wrote: > Updated version of the user-owned cache patchset. It applies on top of > the latest external mempool manager patches from David Hunt [1]. > > [1] http://dpdk.org/ml/archives/dev/2016-June/041479.html > > v4 changes: > > * Fix

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

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 mempool get and put operations. This works with non-EAL threads too. This commit introduces the new

[dpdk-dev] [PATCH v4 2/3] mempool: use bit flags to set multi consumers or producers

From: Lazaros Koromilas Pass the same flags as in rte_mempool_create(). Changes API calls: rte_mempool_generic_put(mp, obj_table, n, flags) rte_mempool_generic_get(mp, obj_table, n, flags) Signed-off-by: Lazaros Koromilas Acked-by: Olivier Matz ---

[dpdk-dev] [PATCH v4 1/3] mempool: deprecate specific get/put functions

From: Lazaros Koromilas This commit introduces the API calls: rte_mempool_generic_put(mp, obj_table, n, is_mp) rte_mempool_generic_get(mp, obj_table, n, is_mc) Deprecates the API calls: rte_mempool_mp_put_bulk(mp, obj_table, n)

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

Updated version of the user-owned cache patchset. It applies on top of the latest external mempool manager patches from David Hunt [1]. [1] http://dpdk.org/ml/archives/dev/2016-June/041479.html v4 changes: * Fix compilation with shared libraries * Add a GOTO_ERR() macro to factorize code in

[dpdk-dev] [PATCH] ixgbe: fix compilation when offload flags disabled

The ixgbe driver does not compile if CONFIG_RTE_IXGBE_RX_OLFLAGS_ENABLE=n because the macro has not the proper number of parameters. Fixes: b37b528d957c ("mbuf: add new Rx flags for stripped VLAN") Reported-by: Amin Tootoonchian Signed-off-by: Olivier Matz ---

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

On Mon, Jun 27, 2016 at 05:29:59PM +0100, Wiles, Keith wrote: > On 6/27/16, 7:58 AM, "dev on behalf of Wiles, Keith" on behalf of keith.wiles at intel.com> wrote: > > > > >On 6/27/16, 3:46 AM, "Richardson, Bruce" > >wrote: > > > >>On Sun, Jun 26, 2016 at 10:54:12AM -0500, Keith Wiles wrote: >

[dpdk-dev] [PATCH] ixgbe: fix unused value

On Mon, Jun 27, 2016 at 05:29:35PM +0100, Bruce Richardson wrote: > On Thu, Jun 02, 2016 at 06:01:22PM +0200, Daniel Mrzyglod wrote: > > An assigned value that is never used may represent unnecessary computation, > > an incorrect algorithm, or possibly the need for cleanup or refactoring. > > > >

[dpdk-dev] [PATCH 4/4] virtio: add neon support

Added neon based Rx vector implementation for virtio. Selected neon based virtio implementation for ARM64 as default and updated the MAINTAINERS file. Signed-off-by: Jerin Jacob --- MAINTAINERS | 1 + config/defconfig_arm64-armv8a-linuxapp-gcc | 1 -

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

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 2 files changed, 226

[dpdk-dev] [PATCH 2/4] virtio: introduce RTE_LIBRTE_VIRTIO_INC_VECTOR

like other PMD drivers, introduce RTE_LIBRTE_VIRTIO_INC_VECTOR for vector based handler selection in virtio Enabled by default in common config and disabled for non X86 platforms Signed-off-by: Jerin Jacob --- config/common_base | 1 +

[dpdk-dev] [PATCH 1/4] virtio: Fix compile time dependency of use_simple_rxtx usage

Removed unnecessary compile time dependency on "use_simple_rxtx". Signed-off-by: Jerin Jacob --- drivers/net/virtio/Makefile | 3 --- drivers/net/virtio/virtio_pci.h | 1 + drivers/net/virtio/virtio_rxtx.c| 28 +---

[dpdk-dev] [PATCH 0/4] Virtio NEON support for ARM

This patchset includes, 1) General cleanup on compile time dependency of use_simple_rxtx with RTE_MACHINE_CPUFLAG_SSSE3 2) Added NEON support for optimized Rx handling This patchset is based on dpdk-next-virtio/master at a1d8bd4911b28e32c35f16ab2ff3e22180d1f1d7 Jerin Jacob (4): virtio: Fix

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

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 size > to the app in any way. It is needed to allocate

[dpdk-dev] [PATCH v2 0/7] fix dependencies

> Panu Matilainen (3): > pipeline: fix truncated dependency list > pdump: fix missing dependency on libpthread > mk: check shared library dependencies > > Thomas Monjalon (4): > mk: remove traces of combined library > mk: fix external library link > mk: fix internal dependencies >

[dpdk-dev] [PATCH] mempool: fix symbol export

> > Every new symbols in release 16.07 are exported with the version > > string DPDK_16.07. > > Also remove the empty local: section which is not needed because > > inherited from the DPDK_2.0 block. > > > > Signed-off-by: Thomas Monjalon > > Good catch, thanks Thomas > Acked-by: Olivier Matz

[dpdk-dev] [PATCH 2/4] virtio: introduce RTE_LIBRTE_VIRTIO_INC_VECTOR

2016-06-27 20:18, Jerin Jacob: > On Mon, Jun 27, 2016 at 04:19:57PM +0200, Thomas Monjalon wrote: > > 2016-06-27 17:24, Jerin Jacob: > > > --- a/config/common_base > > > +++ b/config/common_base > > > @@ -267,6 +267,7 @@ CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_RX=n > > >

[dpdk-dev] [PATCH] scripts: add verbose option in build test help

2016-06-24 12:18, Thomas Monjalon: > The verbose option was available but not advertised. > > Fixes: 6e38dfe21389 ("scripts: add verbose test build option") > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH] scripts: relax line length check for fixed commit

2016-06-25 13:59, Wiles, Keith: > On 6/24/16, 4:30 AM, "dev on behalf of Bruce Richardson" dpdk.org on behalf of bruce.richardson at intel.com> wrote: > > >On Fri, Jun 24, 2016 at 12:44:18AM +0200, Thomas Monjalon wrote: > >> It is better to keep the line "Fixes:" longer than 75 characters > >>

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

On 6/27/16, 11:40 AM, "Richardson, Bruce" wrote: >On Mon, Jun 27, 2016 at 05:29:59PM +0100, Wiles, Keith wrote: >> On 6/27/16, 7:58 AM, "dev on behalf of Wiles, Keith" > dpdk.org on behalf of keith.wiles at intel.com> wrote: >> >> > >> >On 6/27/16, 3:46 AM, "Richardson, Bruce" >> >wrote: >> >

[dpdk-dev] [PATCH v5 0/5] fix issues in packet capture framework

> Reshma Pattan (5): > pdump: fix default socket path > pdump: check getenv return value > pdump: fix string overflow > app/pdump: fix string overflow > app/pdump: fix type casting of ring size Applied, thanks

[dpdk-dev] [PATCH] app/test: avoid freeing mbufs twice in qat test

> > Test_multi_session was freeing mbufs used in the multiple sessions created > > and setting obuf to NULL after it, but ibuf was not being set to NULL, and > > therefore, it was being freed again (ibuf and obuf are pointing at the same > > address), in the ut_teardown() function. > > > > Fixes:

[dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain friendly

2016-06-27 13:06, Wiles, Keith: > > On 6/27/16, 4:05 AM, "Thomas Monjalon" wrote: > > >2016-06-27 10:27, Olivier Matz: > >> On 06/27/2016 10:21 AM, Ananyev, Konstantin wrote: > >> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles > >> >> Move the next pointer to the first

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

On 6/27/16, 7:58 AM, "dev on behalf of Wiles, Keith" wrote: > >On 6/27/16, 3:46 AM, "Richardson, Bruce" wrote: > >>On Sun, Jun 26, 2016 at 10:54:12AM -0500, Keith Wiles wrote: >>> Latest clang compiler 3.8.0 on latest update of Ubuntu >>> creates a few more warnings on -Warray-bounds and extra

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

On Tue, Jun 14, 2016 at 07:08:05PM -0400, Zhihong Wang wrote: > This patch removes constraints in rxq handling when multiqueue is enabled > to handle all the rxqs. > > Current testpmd forces a dedicated core for each rxq, some rxqs may be > ignored when core number is less than rxq number, and

[dpdk-dev] [PATCH 2/4] virtio: introduce RTE_LIBRTE_VIRTIO_INC_VECTOR

2016-06-27 17:24, Jerin Jacob: > --- a/config/common_base > +++ b/config/common_base > @@ -267,6 +267,7 @@ CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_RX=n > CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_TX=n > CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_DRIVER=n > CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_DUMP=n >

[dpdk-dev] [PATCH] e1000/base: Add missing braces to the 'if' statements

On Fri, Jun 24, 2016 at 08:31:05AM +, Lu, Wenzhuo wrote: > Hi Thomas, Markos, > > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, June 24, 2016 3:13 PM > > To: Lu, Wenzhuo > > Cc: dev at dpdk.org; Markos Chandras > > Subject:

[dpdk-dev] [PATCH] e1000/base: Add missing braces to the 'if' statements

On Thu, Jun 23, 2016 at 10:25:52AM +0100, Markos Chandras wrote: > Add the missing braces to the 'if' statements to fix the misleading > identation. This also fixes the following build errors when building > with gcc >= 6: > > drivers/net/e1000/base/e1000_phy.c:4156:2: > error: this 'if' clause

[dpdk-dev] [PATCH v4 00/29] ixgbe/base: update base driver

On Thu, Jun 23, 2016 at 03:22:08PM +0800, Beilei Xing wrote: > Update base driver for ixgbe, mainly work on new features and bug fixes. > > v4 changes: > Update some commit logs. > Merge related patched. > v3 changes: > Fix some commit log issues. > Applied to dpdk-next-net/rel_16_07 /Bruce

[dpdk-dev] [PATCH v14 2/2] i40e: add floating VEB support in i40e

This patch add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both of

[dpdk-dev] [PATCH v14 1/2] i40e: support floating VEB config

Add the new floating related argument option in the devarg. Using this parameter, all the applications can decide whether to use legacy VEB/VEPA or floating VEB. To enable this feature, the user should pass a devargs parameter to the EAL like "-w 84:00.0,enable_floating_veb=1", and the application

[dpdk-dev] [PATCH v14 0/2] i40e: add floating VEB support for i40e

This patch-set add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both

[dpdk-dev] [RFC] librte_vhost: Add unix domain socket fd registration

Yuanhan Liu writes: > On Fri, Jun 24, 2016 at 08:23:52AM -0400, Aaron Conole wrote: >> Is there still merit to this patch, given above? If so, I'd finish my >> integration and testing work and submit it formally. > > Sorry, I don't see the strong need of this patch (at least so far), > judging

[dpdk-dev] dpdk config on VM

Hi All, I am tyrying to install dpdk on ubuntu and test the applications. I have successfully installed dpdk on Host as per the instruction guide , but little bit confused on how to bind it with VM. VM1(TrafficGen)--Host(DPDK)--VM2(Receiver) Do we need to install ovs to test the flows

[dpdk-dev] [PATCH v7 22/25] mlx5: work around spurious compilation errors

On Mon, Jun 27, 2016 at 01:12:15PM +0100, Bruce Richardson wrote: > On Fri, Jun 24, 2016 at 03:18:01PM +0200, Nelio Laranjeiro wrote: > > From: Adrien Mazarguil > > > > Since commit "mlx5: resurrect Tx gather support", older GCC versions (such > > as 4.8.5) may complain about the following: > >

[dpdk-dev] [PATCH v7 17/25] mlx5: add support for inline send

On Mon, Jun 27, 2016 at 01:17:42PM +0100, Bruce Richardson wrote: > On Fri, Jun 24, 2016 at 03:17:56PM +0200, Nelio Laranjeiro wrote: > > From: Yaacov Hazan > > > > Implement send inline feature which copies packet data directly into WQEs > > for improved latency. The maximum packet size and the

[dpdk-dev] [PATCH v14 0/2] i40e: add floating VEB support for i40e

On 6/27/2016 8:20 AM, Zhe Tao wrote: > This patch-set add the support for floating VEB in i40e. > All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or > the floating VEB. When connect to the floating VEB a new floating VEB is > created. Now all the VFs need to connect to

[dpdk-dev] [PATCH] ixgbe:enable configuration for old ptype behavior

Hi, On 27/06/16 09:00, Ananyev, Konstantin wrote: >> The default behavior is to NOT support the old ptype behavior, >> but enabling the configuration option the old ptype style >> can be supported. >> >> Add support for old behaviour until we have a cleaner solution using >> a configuration

[dpdk-dev] [PATCH 2/3] cmdline: remove duplicated symbols from .map

Fixes: 9d41beed24b0 ("lib: provide initial versioning") Signed-off-by: Ferruh Yigit --- lib/librte_cmdline/rte_cmdline_version.map | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/librte_cmdline/rte_cmdline_version.map b/lib/librte_cmdline/rte_cmdline_version.map index c9fc18a..04bcb38

[dpdk-dev] [PATCH 1/3] ethdev: remove duplicated symbols from .map

Fixes: 19b16e2f6442 ("ethdev: add vlan type when setting ether type") Signed-off-by: Ferruh Yigit --- lib/librte_ether/rte_ether_version.map | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map index 97ed0b0..e1ccebe

[dpdk-dev] [PATCH 1/3] ethdev: remove duplicated symbols from .map

On 6/27/2016 1:57 PM, y at ecsmtp.ir.intel.com wrote: > From: Ferruh Yigit > > Fixes: 19b16e2f6442 ("ethdev: add vlan type when setting ether type") > Signed-off-by: Ferruh Yigit Send from wrong e-mail address, although patches looks OK, I don't see them in patchwork and will send again.

[dpdk-dev] [PATCH] vhost: fix missing flag reset on stop

Commit 550c9d27d143 ("vhost: set/reset device flags internally") moves the VIRTIO_DEV_RUNNING set/reset to vhost lib. But I missed one reset on stop; here fixes it. Fixes: 550c9d27d143 ("vhost: set/reset device flags internally") Reported-by: Loftus Ciara Signed-off-by: Yuanhan Liu ---

[dpdk-dev] [PATCH v2] doc: update vhost guide

Mainly on updating vhost-user part: we now support client mode. Also refine some words, and add a bit more explanation. And made an emphatic statement that you are suggested to use vhost-user instead of vhost-cuse, because we have enhanced vhost-user a lot since v2.2 (Actually, I doubt there are

[dpdk-dev] [PATCH v7 17/25] mlx5: add support for inline send

On Fri, Jun 24, 2016 at 03:17:56PM +0200, Nelio Laranjeiro wrote: > From: Yaacov Hazan > > Implement send inline feature which copies packet data directly into WQEs > for improved latency. The maximum packet size and the minimum number of Tx > queues to qualify for inline send are

[dpdk-dev] [PATCH 3/3] port: remove duplicated symbols from .map

> -Original Message- > From: Yigit, Ferruh > Sent: Monday, June 27, 2016 2:03 PM > To: dev at dpdk.org > Cc: Thomas Monjalon ; Olivier Matz > ; Dumitrescu, Cristian > > Subject: [PATCH 3/3] port: remove duplicated symbols from .map > > Fixes: 9d41beed24b0 ("lib: provide initial

[dpdk-dev] [PATCH v13 2/2] i40e: add floating VEB support in i40e

This patch add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both of

[dpdk-dev] [PATCH v13 1/2] i40e: support floating VEB config

Add the new floating related argument option in the devarg. Using this parameter, all the applications can decide whether to use legacy VEB/VEPA or floating VEB. To enable this feature, the user should pass a devargs parameter to the EAL like "-w 84:00.0,enable_floating_veb=1", and the application

[dpdk-dev] [PATCH v13 0/2] i40e: add floating VEB support for i40e

This patch-set add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both

[dpdk-dev] [PATCH v7 22/25] mlx5: work around spurious compilation errors

On Fri, Jun 24, 2016 at 03:18:01PM +0200, Nelio Laranjeiro wrote: > From: Adrien Mazarguil > > Since commit "mlx5: resurrect Tx gather support", older GCC versions (such > as 4.8.5) may complain about the following: > > mlx5_rxtx.c: In function `mlx5_tx_burst': > mlx5_rxtx.c:705:25: error:

[dpdk-dev] [PATCH] doc: update vhost guide

On Sun, Jun 26, 2016 at 08:28:12PM +, Mcnamara, John wrote: > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Friday, June 24, 2016 8:53 AM > > To: dev at dpdk.org > > Cc: Xie, Huawei ; Mcnamara, John > > ; Thomas Monjalon > 6wind.com>; > >

[dpdk-dev] [PATCH v7 15/25] mlx5: handle Rx CQE compression

On Fri, Jun 24, 2016 at 03:17:54PM +0200, Nelio Laranjeiro wrote: > Mini (compressed) CQEs are returned by the NIC when PCI back pressure is > detected, in which case the first CQE64 contains common packet information > followed by a number of CQE8 providing the rest, followed by a matching >

[dpdk-dev] [PATCH] app/test: fix PCI class probing

2016-06-24 14:34, Thomas Monjalon: > The PCI test was failing because some fake devices had no PCI class. > > Fixes: 1dbba1650c89 ("app/test: remove real PCI ids") > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH] app/test: avoid freeing mbuf twice

2016-06-25 17:11, Pablo de Lara: > In cryptodev tests, when input and output buffers were the same, > the mbuf was being freed twice, causing refcnt_atomic to be negative. > > Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests") > > Signed-off-by: Pablo de Lara Applied,

[dpdk-dev] [PATCH] app/test: avoid freeing mbufs twice in qat test

> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Monday, June 27, 2016 1:41 PM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [dpdk-dev] [PATCH] app/test: avoid freeing mbufs twice in qat test > > Test_multi_session was

[dpdk-dev] [PATCH] app/test: fix for icc compilation error

> > Icc complains about variable may be used without setting. > > > > Fixes: 97fe6461c7cbfb ("app/test: add SNOW 3G performance test) > > > > Signed-off-by: Deepak Kumar Jain > Acked-by: John Griffin Applied, thanks

[dpdk-dev] Crypto API use error

Hi, 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 0x1 -n 4 --vdev

[dpdk-dev] [PATCH] kni: fix compilation with gcc 6.1

2016-06-24 09:28, Ferruh Yigit: > On 6/23/2016 3:38 PM, Pablo de Lara wrote: > > Using gcc 6.1, in some cases, kni fails to compile > > because of unused variables: > > > > build/lib/librte_eal/linuxapp/kni/ixgbe_main.c:82:19: > > error: ?ixgbe_copyright? > > defined but not used

[dpdk-dev] [PATCH v2 5/7] mk: fix external dependencies of crypto drivers

On 06/26/2016 07:42 PM, Thomas Monjalon wrote: > When linking drivers as shared libraries, the dependencies need > to be marked as DT_NEEDED entries. > > The crypto dependencies (libsso and libIPSec) are static libraries. > To make them linked in the shared PMDs, the code must relocatable: > -

[dpdk-dev] [PATCH v4 2/2] examples/ethtool: use rte_eth_dev_get_reg_info for reg params

On 23/06/2016 14:26, zr at semihalf.com wrote: > From: Zyta Szpak > > Version 4 of fixing the fixed register width assumption. > The app was allocating too little space for 64-bit registers > which resulted in memory corruption. This commit resolves > this by getting the number of registers and

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

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 not provide register size > to the app in any way. It is

[dpdk-dev] [PATCH v2 3/7] pipeline: fix truncated dependency list

2016-06-27 09:20, Dumitrescu, Cristian: > As discussed in some other email thread, it would also make sense to replace > the ':=' operator with '+=' operator in Makefile of rte_port and rte_table as > well, do you want us to send a separate patch for this? It's a cosmetic change. Yes you can

[dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain friendly

2016-06-27 10:27, Olivier Matz: > On 06/27/2016 10:21 AM, Ananyev, Konstantin wrote: > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles > >> Move the next pointer to the first cacheline of the rte_mbuf structure > >> and move the offload values to the second cacheline to give

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

2016-06-26 10:54, Keith Wiles: > Latest clang compiler 3.8.0 on latest update of Ubuntu > creates a few more warnings on -Warray-bounds and extra > () around 'if' expressions. The goal of compiler checks is to fix the code. Why disabling these checks instead of fixing the code? If they are false

[dpdk-dev] backtracing from within the code

I've done something similar to what's described in the link below. But it's worth pointing out that it's using printf() inside a signal handler, which isn't safe. If your use case is catching SIGSEGV, for example, solutions built on printf() will usually work, but can deadlock. One way around

[dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain friendly

On 06/27/2016 10:21 AM, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles >> Sent: Saturday, June 25, 2016 4:56 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf

[dpdk-dev] [PATCH v3] cryptodev: uninline parameter parsing

> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Sunday, June 26, 2016 5:34 PM > To: Doherty, Declan; De Lara Guarch, Pablo > Cc: dev at dpdk.org > Subject: [PATCH v3] cryptodev: uninline parameter parsing > > There is no need to have this

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

On Sun, Jun 26, 2016 at 10:54:12AM -0500, Keith Wiles wrote: > Latest clang compiler 3.8.0 on latest update of Ubuntu > creates a few more warnings on -Warray-bounds and extra > () around 'if' expressions. > > Signed-off-by: Keith Wiles > --- > app/test-pmd/Makefile| 3 +++ >

[dpdk-dev] example/l2fwd-crypto: maybe an issue?

On 23/06/16 06:30, Xuelin Shi wrote: > Hi, > > Do we have assumption each core will have a separate crypto device? > > I have only one crypto device with several queues which are shared by > multi-cores. > > As I run the l2fwd-crypto, only one cparam is valid and has a good dev_id. > The other

[dpdk-dev] [PATCH] bnxt: Add Cumulus+ PCI ID

On Fri, Jun 24, 2016 at 01:31:37PM -0500, Ajit Khaparde wrote: > On Fri, Jun 24, 2016 at 6:59 AM, Bruce Richardson < > bruce.richardson at intel.com> wrote: > > > On Tue, Jun 21, 2016 at 04:58:20PM -0500, Ajit Khaparde wrote: > > > This patch adds support for Cumulus+ Ethernet adapters. > > >

[dpdk-dev] [PATCH v2] doc: update vhost guide

> -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Monday, June 27, 2016 6:20 AM > To: dev at dpdk.org > Cc: Xie, Huawei ; Mcnamara, John > ; Thomas Monjalon ; > Yuanhan Liu > Subject: [PATCH v2] doc: update vhost guide > > Mainly on updating

[dpdk-dev] [PATCH v2 3/7] pipeline: fix truncated dependency list

> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Sunday, June 26, 2016 5:42 PM > To: Panu Matilainen > Cc: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 3/7] pipeline: fix truncated dependency list > > From: Panu Matilainen > >

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

> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jasvinder Singh > Sent: Thursday, June 23, 2016 6:50 PM > To: dev at dpdk.org > Cc: Dumitrescu, Cristian > Subject: [dpdk-dev] [PATCH] ip_pipeline: add Python script file for > creating visual diagram of IP

[dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain friendly

> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles > Sent: Saturday, June 25, 2016 4:56 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] mbuf:rearrange mbuf to be more mbuf chain > friendly > > Move the next pointer to the first

[dpdk-dev] [PATCH v3] cryptodev: uninline parameter parsing

> > There is no need to have this parsing inlined in the header. > > It brings kvargs dependency to every crypto drivers. > > The functions are moved into rte_cryptodev.c. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: Pablo de Lara Applied

[dpdk-dev] [PATCH] ixgbe:enable configuration for old ptype behavior

> The default behavior is to NOT support the old ptype behavior, > but enabling the configuration option the old ptype style > can be supported. > > Add support for old behaviour until we have a cleaner solution using > a configuration option CONFIG_RTE_IXGBE_ENABLE_OLD_PTYPE_BEHAVIOUR, > which

[dpdk-dev] [PATCH v12 1/2] i40e: support floating VEB config

On Fri, Jun 24, 2016 at 12:14:14PM +0100, Ferruh Yigit wrote: > Hi Zhe, > > On 6/24/2016 9:29 AM, Zhe Tao wrote: > > Add the new floating VEB related arguments option in the devarg. > > Using this parameter, all the applications can decide whether to use legacy > > VEB/VEPA or floating VEB. > >

[dpdk-dev] [PATCH v2 1/2] i40e: fix problematic dereference

It fixes the issue reported by Coverity of 'Dereference before null check', by deleting null checks as they are not necessary at all, or move null checks before where uses it. Coverity ID 119267: Dereference before null check. Coverity ID 13301: Dereference before null check. Coverity ID 13294:

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

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 +-- drivers/net/i40e/i40e_rxtx.c | 12 2 files changed,