Re: [lng-odp] [PATCHv7 0/3] changelog: summary of changes for odp v1.12.0.0

2016-11-28 Thread Maxim Uvarov
Anders, Bogdan, do you have any other comments or you can set your review-by? Maxim. On 11/24/16 23:24, Maxim Uvarov wrote: v7: more accurate define disable option (Anders) Bill Fischofer (1): changelog: summary of changes for odp v1.12.0.0 Maxim Uvarov (2): configure.ac: disable

Re: [lng-odp] [API-NEXT PATCH] api: ipsec: added IPSEC API

2016-11-28 Thread Bogdan Pricope
Hi Petri, 1. I am not a big fan of idea that application level data stored per SA needs to be accessed per packet: for example for asynchronous processing you need to know if processing is inbound or outbound, tunnel or transport and also to update stats through this mechanism:

Re: [lng-odp] [PATCH] test: linux-gen: fix termination in mmap_vlan_ins

2016-11-28 Thread Maxim Uvarov
please review. This need to be in nearest release. Maxim. On 11/25/16 17:06, Maxim Uvarov wrote: exit variable has to be volatile to be visible from thread. Without that background process is not killed with default optimization level. Signed-off-by: Maxim Uvarov ---

Re: [lng-odp] A topic related to bug 2622

2016-11-28 Thread Elo, Matias (Nokia - FI/Espoo)
On 28 Nov 2016, at 12:19, Yi He > wrote: Thanks Matias Very clearly documented: “ * Event ordering is based on the * received event(s), but also other (newly allocated or stored) events are * ordered when enqueued within the same ordered context. “

[lng-odp] [PATCH] platform: linux-generic: reading cpu affinity from cpuset

2016-11-28 Thread Balakrishna Garapati
Fixes bug: https://bugs.linaro.org/show_bug.cgi?id=2472 Signed-off-by: Balakrishna Garapati --- With this new proposal cpu affinity is read correctly especialy when using cgroups otherwise wrong cpu mask is set. platform/linux-generic/odp_cpumask.c | 69

Re: [lng-odp] [PATCH v2] travis.yml: add coverity support

2016-11-28 Thread Mike Holmes
ping, are you happy to merge this or do I respin it ? I think you are the only other user so far unless Christophe gets chance to try it On 18 November 2016 at 14:15, Maxim Uvarov wrote: > ok, let's leave that. Maybe later we will have some code in user guide >

Re: [lng-odp] [PATCHv7 0/3] changelog: summary of changes for odp v1.12.0.0

2016-11-28 Thread Bogdan Pricope
./configure --help | grep ABI --disable-abi-compat disables ABI compatible mode, enables inline code in header files while in CHANGELOG: +`--enable-abi-compat=yes`:: and +`--enable-abi-compat=no`:: but no: --disable-abi-compat I guess is common practice and --disable-abi-compat is

[lng-odp] [PATCHv2] platform: linux-generic: reading cpu affinity from cpuset

2016-11-28 Thread Balakrishna Garapati
With this new proposal cpu affinity is read correctly especially when using cgroups otherwise wrong cpu mask is set. Fixes bug: https://bugs.linaro.org/show_bug.cgi?id=2472 Signed-off-by: Balakrishna Garapati --- platform/linux-generic/odp_cpumask.c | 69

[lng-odp] [PATCHv3] platform: linux-generic: reading cpu affinity from cpuset

2016-11-28 Thread Balakrishna Garapati
With this new proposal cpu affinity is read correctly especially when using cgroups otherwise wrong cpu mask is set. Fixes bug: https://bugs.linaro.org/show_bug.cgi?id=2472 Signed-off-by: Balakrishna Garapati --- v1 to v2: added Description of the issue to the

Re: [lng-odp] lng-odp at a text only list.

2016-11-28 Thread Mike Holmes
On 24 November 2016 at 05:28, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia-bell-labs.com> wrote: > Could bugzilla spam a dedicated lng-odp-bugs / lng-odp-reports / etc list ? > > We'd need only one strictly plain text list - the one that humans use for > patches. > That is a

Re: [lng-odp] [API-NEXT PATCHv4 1/7] api: random: replace use_entropy with odp_rand_kind parameter

2016-11-28 Thread Bill Fischofer
On Mon, Nov 28, 2016 at 2:38 AM, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > > From: Bill Fischofer [mailto:bill.fischo...@linaro.org] > Sent: Monday, November 28, 2016 6:25 AM > To: Savolainen, Petri (Nokia - FI/Espoo) >

Re: [lng-odp] [PATCH 0/2] calling term functions from pktio_perf

2016-11-28 Thread Bill Fischofer
For this series: Reviewed-by: Bill Fischofer On Fri, Nov 25, 2016 at 11:14 AM, Christophe Milard wrote: > The term functions were not called in odp_pktio_perf.c. > Fixing this (Patch 2) showed another problem: closing pktio > in

Re: [lng-odp] [API-NEXT PATCH 3/3] linux-gen: add interests query (iquery) scheduler

2016-11-28 Thread Bill Fischofer
Attempting to compile this with clang I get: odp_schedule_iquery.c:170:3: error: redefinition of typedef 'sched_thread_local_t' is a C11 feature [-Werror,-Wtypedef-redefinition] } sched_thread_local_t; ^ odp_schedule_iquery.c:111:35: note: previous definition is here typedef struct

Re: [lng-odp] [API-NEXT PATCHv4 1/7] api: random: replace use_entropy with odp_rand_kind parameter

2016-11-28 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: Bill Fischofer [mailto:bill.fischo...@linaro.org] Sent: Monday, November 28, 2016 6:25 AM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [API-NEXT PATCHv4 1/7] api: random: replace use_entropy with

Re: [lng-odp] A topic related to bug 2622

2016-11-28 Thread Elo, Matias (Nokia - FI/Espoo)
On 28 Nov 2016, at 7:58, Yi He > wrote: Hi, Bill, Matias For this bug related to the validation scheduler test program, with errors like: scheduler.c:878 - bctx->sequence == seq I saw a potential problem in the validation scheduler program code

Re: [lng-odp] A topic related to bug 2622

2016-11-28 Thread Yi He
Thanks Matias Very clearly documented: “ * Event ordering is based on the * received event(s), but also other (newly allocated or stored) events are * ordered when enqueued within the same ordered context. “ One question is that for scheduled events, their order are very clear determined by

Re: [lng-odp] [API-NEXT PATCH] api: ipsec: added IPSEC API

2016-11-28 Thread Nikhil Agarwal
-Original Message- From: Savolainen, Petri (Nokia - FI/Espoo) [mailto:petri.savolai...@nokia-bell-labs.com] Sent: Friday, November 25, 2016 6:40 PM To: Nikhil Agarwal ; lng-odp@lists.linaro.org Subject: RE: [lng-odp] [API-NEXT PATCH] api: ipsec: added IPSEC API