Re: [lng-odp] [API-NEXT PATCH v3] api: pool: additional packet length configuration

2017-06-22 Thread Nikhil Agarwal
This concept for sub-pools within a pool is adding unnecessary complexity to pool configurations. What is the purpose of this? We don't support having multiple size buffers in same pool. In case application wants to attach multiple pools to same pktio, we can introduce an API to attach pool to

Re: [lng-odp] [PATCH v2] build: fix conditional compilation of sources

2017-06-22 Thread Brian Brooks
Hi Maxim, Can you please land this in api-next as well as master since we need to rebase the scalable scheduler patch series against this patch? Thanks, Brian On 06/22 17:05:38, Brian Brooks wrote: > Explicitly add all arch//* files to respective _SOURCES > variables instead of using @ARCH_DIR@

Re: [lng-odp] [PATCH v2] build: fix conditional compilation of sources

2017-06-22 Thread Dmitry Eremin-Solenikov
On 23.06.2017 01:05, Brian Brooks wrote: > Explicitly add all arch//* files to respective _SOURCES > variables instead of using @ARCH_DIR@ substitution. > > This patch fixes the broken build for ARM, PPC, and MIPS > introduced by [1] and the similar issue reported while > testing [2]. > > From

[lng-odp] [PATCH v2] build: fix conditional compilation of sources

2017-06-22 Thread Brian Brooks
Explicitly add all arch//* files to respective _SOURCES variables instead of using @ARCH_DIR@ substitution. This patch fixes the broken build for ARM, PPC, and MIPS introduced by [1] and the similar issue reported while testing [2]. >From the Autoconf manual [3]: You can't put a configure

Re: [lng-odp] [PATCH API-NEXT v1 1/1] linux-generic: queue: modular queue interface

2017-06-22 Thread Honnappa Nagarahalli
On 22 June 2017 at 02:48, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >> Honnappa Nagarahalli >> Sent: Thursday, June 22, 2017 12:20 AM >> To: Maxim Uvarov

Re: [lng-odp] Need to standardize tools version

2017-06-22 Thread Honnappa Nagarahalli
On 22 June 2017 at 02:18, Maxim Uvarov wrote: > Yes we can document that. That would be greatly helpful, knowing exactly what are the check points. If Travis passes patch can be send to mailing > list. We should continuously improve Travis scripts to add more build and >

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Honnappa Nagarahalli
On 22 June 2017 at 10:30, Maxim Uvarov wrote: > On 06/22/17 17:55, Brian Brooks wrote: >> On 06/22 10:27:01, Savolainen, Petri (Nokia - FI/Espoo) wrote: >>> I was asking to make sure that performance impact has been checked also >>> when timers are not used, e.g. l2fwd

Re: [lng-odp] [API-NEXT PATCH v9 4/6] linux-gen: sched scalable: add a concurrent queue

2017-06-22 Thread Bill Fischofer
On Thu, Jun 22, 2017 at 11:46 AM, Brian Brooks wrote: >> > > The first is built only for ARM and the second for the rest. Would there >> > >be a way to build both always ? >> > For ARMv7a and ARMv8a, you could build both versions. You really want to >> > use the LL/SC

Re: [lng-odp] [API-NEXT PATCHv7 1/5] api: feature: add odp feature bits

2017-06-22 Thread Maxim Uvarov
Merged. On 06/13/17 16:16, Savolainen, Petri (Nokia - FI/Espoo) wrote: > For the entire patch set: > > Reviewed-by: Petri Savolainen > > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Bill >> Fischofer >>

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Maxim Uvarov
On 06/22/17 19:54, Brian Brooks wrote: > On 06/22 19:44:45, Maxim Uvarov wrote: >> On 06/22/17 19:19, Brian Brooks wrote: >>> On 06/22 19:06:01, Maxim Uvarov wrote: On 06/22/17 17:17, Brian Brooks wrote: > On 06/22 11:13:57, Maxim Uvarov wrote: >> On 22 June 2017 at 06:24, Brian

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Brian Brooks
On 06/22 19:44:45, Maxim Uvarov wrote: > On 06/22/17 19:19, Brian Brooks wrote: > > On 06/22 19:06:01, Maxim Uvarov wrote: > >> On 06/22/17 17:17, Brian Brooks wrote: > >>> On 06/22 11:13:57, Maxim Uvarov wrote: > On 22 June 2017 at 06:24, Brian Brooks wrote: > >

Re: [lng-odp] [API-NEXT PATCH v9 4/6] linux-gen: sched scalable: add a concurrent queue

2017-06-22 Thread Brian Brooks
> > > The first is built only for ARM and the second for the rest. Would there > > >be a way to build both always ? > > For ARMv7a and ARMv8a, you could build both versions. You really want to > > use the LL/SC version on these architectures. > > > > For architectures without double-word LL/SC,

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Maxim Uvarov
On 06/22/17 19:19, Brian Brooks wrote: > On 06/22 19:06:01, Maxim Uvarov wrote: >> On 06/22/17 17:17, Brian Brooks wrote: >>> On 06/22 11:13:57, Maxim Uvarov wrote: On 22 June 2017 at 06:24, Brian Brooks wrote: > Explicitly add all arch//* files to respective

[lng-odp] [Linaro/odp] 0d24ad: arch: arm: add CPU global time

2017-06-22 Thread GitHub
Branch: refs/heads/api-next Home: https://github.com/Linaro/odp Commit: 0d24adacfe17c8e0e1348f19cd8b75b64cb13ccf https://github.com/Linaro/odp/commit/0d24adacfe17c8e0e1348f19cd8b75b64cb13ccf Author: Brian Brooks Date: 2017-06-20 (Tue, 20 Jun 2017)

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Brian Brooks
On 06/22 19:06:01, Maxim Uvarov wrote: > On 06/22/17 17:17, Brian Brooks wrote: > > On 06/22 11:13:57, Maxim Uvarov wrote: > >> On 22 June 2017 at 06:24, Brian Brooks wrote: > >> > >>> Explicitly add all arch//* files to respective _SOURCES > >>> variables instead of using

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Brian Brooks
On 06/22 18:30:47, Maxim Uvarov wrote: > On 06/22/17 17:55, Brian Brooks wrote: > > On 06/22 10:27:01, Savolainen, Petri (Nokia - FI/Espoo) wrote: > >> I was asking to make sure that performance impact has been checked also > >> when timers are not used, e.g. l2fwd performance before and after

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Maxim Uvarov
On 06/22/17 17:17, Brian Brooks wrote: > On 06/22 11:13:57, Maxim Uvarov wrote: >> On 22 June 2017 at 06:24, Brian Brooks wrote: >> >>> Explicitly add all arch//* files to respective _SOURCES >>> variables instead of using @ARCH_DIR@ substitution. >>> >>> This patch fixes

[lng-odp] [PATCH v1 1/1] linux-generic: crypto: adapt HMAC code to OpenSSL 1.1.x

2017-06-22 Thread Github ODP bot
From: Dmitry Eremin-Solenikov OpenSSL 1.1.x has changed HMAC API in an incompatible way. Let's adapt to it by providing version-dependent wrapper around HMAC calculation. Signed-off-by: Dmitry Eremin-Solenikov

[lng-odp] [PATCH v1 0/1] linux-generic: crypto: adapt HMAC code to OpenSSL 1.1.x

2017-06-22 Thread Github ODP bot
OpenSSL 1.1.x has changed HMAC API in an incompatible way. Let's adapt to it by providing version-dependent wrapper around HMAC calculation. Signed-off-by: Dmitry Eremin-Solenikov dmitry.ereminsoleni...@linaro.org Reviewed-and-tested-by: Brian Brooks brian.bro...@arm.com

Re: [lng-odp] [MONARCH_LTS PATCH] linux-gen: socket: remove limits for maximum RX/TX burst size

2017-06-22 Thread Maxim Uvarov
added to monarch_lts branch. And also added monarch_lts branch to github. Which looks like I forgot to add and nobody noticed. Best regards, Maxim. On 06/20/17 10:13, Matias Elo wrote: > Remove unnecessary limits for maximum RX/TX burst size. > > Fixes:

[lng-odp] [Linaro/odp] 6dac46: git_hash: handle git worktree

2017-06-22 Thread GitHub
Branch: refs/heads/monarch_lts Home: https://github.com/Linaro/odp Commit: 6dac46d9c8dd8cc1f27638215704effcf6269baf https://github.com/Linaro/odp/commit/6dac46d9c8dd8cc1f27638215704effcf6269baf Author: Nicolas Morey-Chaisemartin Date: 2016-11-25 (Fri, 25 Nov

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Maxim Uvarov
On 06/22/17 17:55, Brian Brooks wrote: > On 06/22 10:27:01, Savolainen, Petri (Nokia - FI/Espoo) wrote: >> I was asking to make sure that performance impact has been checked also when >> timers are not used, e.g. l2fwd performance before and after the change. It >> would be also appropriate to

[lng-odp] [Bug 2857] Bus error on not enough hdd space (Ubuntu VM, aarch64)

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2857 Bill Fischofer changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS ---

[lng-odp] [Bug 2861] Remove redundant loop_support parameter in pktio capability

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2861 Bill Fischofer changed: What|Removed |Added CC|

[lng-odp] [Bug 2895] odp_crypto_operation() does not work with multi-segment packets

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2895 Bill Fischofer changed: What|Removed |Added Resolution|--- |FIXED

[lng-odp] [Bug 2903] validation: add test case for PMR LD_VNI

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2903 --- Comment #2 from Bill Fischofer --- Ping to Bala. Please update whether this still is on the "to do" list. -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 2938] make file deps failure

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2938 Bill Fischofer changed: What|Removed |Added CC|

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Brian Brooks
On 06/22 10:27:01, Savolainen, Petri (Nokia - FI/Espoo) wrote: > I was asking to make sure that performance impact has been checked also when > timers are not used, e.g. l2fwd performance before and after the change. It > would be also appropriate to test impact in the worst case: l2fwd type >

[lng-odp] [Bug 2941] test bug for debug

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2941 Bill Fischofer changed: What|Removed |Added Resolution|--- |FIXED

[lng-odp] [Bug 2976] IP headers checksum functions are incorrect

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2976 --- Comment #5 from Bill Fischofer --- Ping to remind to look at this as part of IPsec implementation. -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 2988] ODP exposes symbols outside of odp*/_odp* namespace

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2988 --- Comment #5 from Bill Fischofer --- Dmitry, can you suggest a patch to address this? -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 3017] Travis: time main test out of boundaries

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3017 Bill Fischofer changed: What|Removed |Added CC|

[lng-odp] [Bug 3039] Socket pktio recv fails on large number of packet

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3039 --- Comment #6 from Bill Fischofer --- Will bring up question of issuing Monarch LTS service patch at next week's LNG-SC call. -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 3051] ./configure has to fail on not recognized option

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3051 Bill Fischofer changed: What|Removed |Added CC|

[lng-odp] [Bug 2995] compile from valid install failed

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2995 Bill Fischofer changed: What|Removed |Added Ever confirmed|0 |1

[lng-odp] [Bug 3066] Cross compile broken for ARMv8

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3066 Bill Fischofer changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] [PATCH v2 0/9] Rework the way ODP links with other libraries

2017-06-22 Thread Maxim Uvarov
Hello Matias, was your commend addressed in v2? Thank you, Maxim. On 06/14/17 00:59, Github ODP bot wrote: > Currently configure tests add all libraries to AM_LDFLAGS or LIBS, thus > making all libraries link against everything. For example libodp-linux ends > up depending on libcunit.

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Brian Brooks
On 06/22 11:13:57, Maxim Uvarov wrote: > On 22 June 2017 at 06:24, Brian Brooks wrote: > > > Explicitly add all arch//* files to respective _SOURCES > > variables instead of using @ARCH_DIR@ substitution. > > > > This patch fixes the broken build for ARM, PPC, and MIPS > >

Re: [lng-odp] [PATCH API-NEXT v1 1/1] linux-generic: crypto: adapt HMAC code to OpenSSL 1.1.x

2017-06-22 Thread Brian Brooks
On 06/15 11:00:07, Github ODP bot wrote: > From: Dmitry Eremin-Solenikov > > OpenSSL 1.1.x has changed HMAC API in an incompatible way. Let's adapt > to it by providing version-dependent wrapper around HMAC calculation. I am using OpenSSL 1.1.x on some

Re: [lng-odp] [PATCH v2] linux-gen: time: fix ARM compile for GCC 4.8

2017-06-22 Thread Maxim Uvarov
just merged it with Brians review. Tested ifdefs yesterday. Maxim. On 22 June 2017 at 15:49, Bill Fischofer wrote: > On Thu, Jun 22, 2017 at 4:17 AM, Petri Savolainen > wrote: > > Use __aarch64__ instead of __ARM_ARCH, since it's

[lng-odp] [Bug 3066] Cross compile broken for ARMv8

2017-06-22 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3066 --- Comment #1 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/09a7800c4c4a093fb962e362952f9cf562d2fc98 refs/heads/master 2017-06-22T15:40:43+03:00 Petri Savolainen petri.savolai...@linaro.org linux-gen: time: fix

Re: [lng-odp] [PATCH v2] linux-gen: time: fix ARM compile for GCC 4.8

2017-06-22 Thread Bill Fischofer
On Thu, Jun 22, 2017 at 4:17 AM, Petri Savolainen wrote: > Use __aarch64__ instead of __ARM_ARCH, since it's backwards > compatible between GCC versions. > > Fixes bug https://bugs.linaro.org/show_bug.cgi?id=3066 > > Signed-off-by: Petri Savolainen

[lng-odp] [Linaro/odp] ddf2ac: travis: back to stable image until issue is resolv...

2017-06-22 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: ddf2ac38b1288283f7b3d7c9e16e36f392685866 https://github.com/Linaro/odp/commit/ddf2ac38b1288283f7b3d7c9e16e36f392685866 Author: Maxim Uvarov Date: 2017-06-22 (Thu, 22 Jun 2017)

Re: [lng-odp] [PATCH API-NEXT v1 1/1] linux-generic: crypto: adapt HMAC code to OpenSSL 1.1.x

2017-06-22 Thread Dmitry Eremin-Solenikov
On 15.06.2017 11:00, Github ODP bot wrote: > From: Dmitry Eremin-Solenikov > > OpenSSL 1.1.x has changed HMAC API in an incompatible way. Let's adapt > to it by providing version-dependent wrapper around HMAC calculation. > > Signed-off-by: Dmitry

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Savolainen, Petri (Nokia - FI/Espoo)
I was asking to make sure that performance impact has been checked also when timers are not used, e.g. l2fwd performance before and after the change. It would be also appropriate to test impact in the worst case: l2fwd type application + a periodic 1sec timeout. Timer is on, but timeouts come

[lng-odp] [PATCH v1 0/1] travis: back to stable image until issue is resolved

2017-06-22 Thread Github ODP bot
after update tap pktio hangs. Back to stable image untill we solve failure. Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org github /** Email created from pull request 55 (muvarov:master_travis_deprecated) ** https://github.com/Linaro/odp/pull/55 **

Re: [lng-odp] [API-NEXT PATCH v9 4/6] linux-gen: sched scalable: add a concurrent queue

2017-06-22 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: Ola Liljedahl [mailto:ola.liljed...@arm.com] > Sent: Wednesday, June 21, 2017 7:31 PM > To: Savolainen, Petri (Nokia - FI/Espoo) ; > Brian Brooks ; lng-odp@lists.linaro.org > Cc: nd > Subject:

[lng-odp] [PATCH v2] linux-gen: time: fix ARM compile for GCC 4.8

2017-06-22 Thread Petri Savolainen
Use __aarch64__ instead of __ARM_ARCH, since it's backwards compatible between GCC versions. Fixes bug https://bugs.linaro.org/show_bug.cgi?id=3066 Signed-off-by: Petri Savolainen --- platform/linux-generic/arch/arm/odp_cpu_arch.c | 4 ++-- 1 file changed, 2

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-22 Thread Maxim Uvarov
Petri, do you want to test performance before patch inclusion? Maxim. On 21 June 2017 at 21:52, Honnappa Nagarahalli < honnappa.nagaraha...@linaro.org> wrote: > We have not run any performance application. In our Linaro connect > meeting, we presented numbers on how it improves the timer

Re: [lng-odp] [PATCH] build: fix conditional compilation of sources

2017-06-22 Thread Maxim Uvarov
On 22 June 2017 at 06:24, Brian Brooks wrote: > Explicitly add all arch//* files to respective _SOURCES > variables instead of using @ARCH_DIR@ substitution. > > This patch fixes the broken build for ARM, PPC, and MIPS > introduced by [1] and the similar issue reported

[lng-odp] [CLOUD-DEVv2] ddf: examples: add ddf module skeleton and ddf application

2017-06-22 Thread Bogdan Pricope
Signed-off-by: Bogdan Pricope --- example/Makefile.am | 4 +- example/ddf_app/Makefile.am | 10 example/ddf_app/odp_ddf_app.c | 49 +++ example/ddf_ifs/Makefile.am | 27 +++

[lng-odp] [CLOUD-DEVv2] ddf: examples: ddf module skeleton

2017-06-22 Thread Bogdan Pricope
This patch contains a ddf module skeleton (example) and a ddf model application. Was designed to highlight (print) DDF operations/steps. Module is loaded at init_global time through odp configuration file (modules section). Usage: export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/odp/lib/.libs

Re: [lng-odp] [PATCH API-NEXT v1 1/1] linux-generic: queue: modular queue interface

2017-06-22 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Honnappa Nagarahalli > Sent: Thursday, June 22, 2017 12:20 AM > To: Maxim Uvarov > Cc: lng-odp-forward > Subject: Re: [lng-odp] [PATCH

Re: [lng-odp] Need to standardize tools version

2017-06-22 Thread Savolainen, Petri (Nokia - FI/Espoo)
If you write bug free, C99 compliant code, it's actually pretty rare to bump into compiler issues. The issues are mostly raised by different error check levels of different compilers, and thus may catch actual bugs. Every now and then, we bump into false positives, but there's usually an easy

Re: [lng-odp] Need to standardize tools version

2017-06-22 Thread Maxim Uvarov
Yes we can document that. If Travis passes patch can be send to mailing list. We should continuously improve Travis scripts to add more build and targets. We can add hooks to compile it and run on different hardware which we care about. Problem with Travis is that we do not control it. Images can

[lng-odp] [API-NEXTv2] api: ipsec: reorganize odp_ipsec_sa_param_t structure based on SA direction

2017-06-22 Thread Bogdan Pricope
Signed-off-by: Bogdan Pricope --- include/odp/api/spec/ipsec.h | 114 --- 1 file changed, 63 insertions(+), 51 deletions(-) diff --git a/include/odp/api/spec/ipsec.h b/include/odp/api/spec/ipsec.h index e602e4b..5373ede 100644

[lng-odp] [API-NEXTv2] api: ipsec: reorganize odp_ipsec_sa_param_t structure

2017-06-22 Thread Bogdan Pricope
Reorganize odp_ipsec_sa_param_t structure to highlight inbound only and outbound only members. To be applied on top of Petri's v3 of 'IPSEC packet event' v2: Set names for inbound only and outbound only struct members Bogdan Pricope (1): api: ipsec: reorganize odp_ipsec_sa_param_t structure