[lng-odp] [Bug 2828] CID 174662: Bad sizeof() in odp_schedule_sp.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2828 Yi He changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS --- Comment #3 from Yi

[lng-odp] [PATCH 2/2] api: New API to support timer capacity

2017-01-18 Thread Kevin Wang
Implement a new internal function timer_res_init() to detect the max timer resolution without overrun at the ODP init stage. It will check timer resolution from 1ms to 100us, 10us...1ns until the timer is overrun. --- example/generator/odp_generator.c | 8 ++-

[lng-odp] [PATCH 1/2] api: New API to support timer capacity

2017-01-18 Thread Kevin Wang
Currently, user needs to decide the timer resolution before creating a timer pool. But sometimes it will cause timer overrun as the system can't support such high resolution. So a new API is required to expose the timer capacity to the user. --- include/odp/api/spec/timer.h | 21

[lng-odp] [API-NEXT PATCH] linux-gen: align: fix round up power of two

2017-01-18 Thread Yi He
Fix Bug https://bugs.linaro.org/show_bug.cgi?id=2828 incorrect sizeof expression (BAD_SIZEOF) of ODP_ROUNDUP_POWER_2(literal constant) usage. Signed-off-by: Yi He --- platform/linux-generic/include/odp_align_internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[lng-odp] [Bug 2828] CID 174662: Bad sizeof() in odp_schedule_sp.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2828 Yi He changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[lng-odp] [Bug 2826] CID 174664: Unchecked return in pool.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2826 Bill Fischofer changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS

[lng-odp] [PATCH] linux-generic: pool: add odp_pool_capability() rc check

2017-01-18 Thread Bill Fischofer
Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2826 by adding an explicit check of the rc from odp_pool_capability() for consistency with other code that calls this routine. Signed-off-by: Bill Fischofer --- platform/linux-generic/odp_pool.c | 3 ++- 1 file

[lng-odp] [PATCH v3 3/4] helper: move thread implementation under platform

2017-01-18 Thread Mike Holmes
Only the portable api is built by default, use --enable-helper-extn to enable non portable APIs for a helper platform Signed-off-by: Mike Holmes --- configure.ac | 17 +- example/Makefile.inc | 2 +-

Re: [lng-odp] [PATCH 7/8] abi: queue: added initial definitions

2017-01-18 Thread Brian Brooks
On 01/18 14:57:02, Petri Savolainen wrote: > Initially, use the same default ABI file for all architectures. > Default values match those defined in odp-linux implementation > to minimize changes in this phase. Removed an unused type > definition, documentation of that need to be still removed >

[lng-odp] [PATCH v3 4/4] helper: remove dependence on test dir

2017-01-18 Thread Mike Holmes
There is no reason that the helpers need to depend on the test directory so remove the offending links Signed-off-by: Mike Holmes --- helper/test/Makefile.am | 26 --- helper/test/chksum.c| 18 helper/test/cuckootable.c

[lng-odp] [PATCH v3 2/4] helper: cleanup Linux rename to thread

2017-01-18 Thread Mike Holmes
Remove the OS specific Linux filename and use the name of the generic apis it contains which is for odp_threads Signed-off-by: Mike Holmes --- example/classifier/odp_classifier.c | 2 +- example/generator/odp_generator.c | 2 +-

[lng-odp] [PATCH v3 1/4] configure: use helper configure.m4

2017-01-18 Thread Mike Holmes
Don't keep helper data in the main configure, split it to an m4 just like the other sub dirs Signed-off-by: Mike Holmes --- configure.ac | 2 -- helper/m4/configure.m4 | 3 +++ 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure.ac

[lng-odp] [PATCH v3 0/4] remove Linux specifics and dependence on test dir

2017-01-18 Thread Mike Holmes
Starting with some clean up, rename the Linux specific files and then move them to a platform specific directory. A new configure option is introduced "with-helper-platform", this defaults to match the existing odp selector "with-platform" so that there is no operational change. With the move

[lng-odp] [Bug 2830] CID 168956: Storage leak in cuckoo.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2830 --- Comment #1 from Bill Fischofer --- Also occurs here: 466unsigned num; 467int ret = 0; 468struct timeval start, end; 469double add_time = 0; 470 471fflush(stdout); 472table

[lng-odp] [Bug 2831] New: CID 168994: Unchecked return in mmap_unmap_sock()

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2831 Bug ID: 2831 Summary: CID 168994: Unchecked return in mmap_unmap_sock() Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux

[lng-odp] [Bug 2830] New: CID 168956: Storage leak in cuckoo.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2830 Bug ID: 2830 Summary: CID 168956: Storage leak in cuckoo.c Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux Status:

[lng-odp] [Bug 2829] New: CID 168965: Unused value in helper iptablelookup

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2829 Bug ID: 2829 Summary: CID 168965: Unused value in helper iptablelookup Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux

[lng-odp] [Bug 2828] CID 174662: Bad sizeof() in odp_schedule_sp.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2828 --- Comment #1 from Bill Fischofer --- Also appears here: 350static inline void add_tail(sched_cmd_t *cmd) 351{ 352prio_queue_t *prio_queue; 353int group= cmd->s.group; 354int prio =

[lng-odp] [Bug 2828] New: CID 174662: Bad sizeof() in odp_schedule_sp.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2828 Bug ID: 2828 Summary: CID 174662: Bad sizeof() in odp_schedule_sp.c Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux Status:

[lng-odp] [Bug 2827] New: CID 174663: Unchecked return in _ishm.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2827 Bug ID: 2827 Summary: CID 174663: Unchecked return in _ishm.c Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux Status:

[lng-odp] [Bug 2826] New: CID 174664: Unchecked return in pool.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2826 Bug ID: 2826 Summary: CID 174664: Unchecked return in pool.c Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux Status:

[lng-odp] [Bug 2825] New: CID 174667: Resource leak in _ishm

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2825 Bug ID: 2825 Summary: CID 174667: Resource leak in _ishm Product: OpenDataPlane - linux- generic reference Version: api-next Hardware: Other OS: Linux Status:

[lng-odp] [Bug 2790] CID 173382: Memory - illegal accesses /test/common_plat/performance/odp_l2fwd.c

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2790 Bill Fischofer changed: What|Removed |Added Status|IN_PROGRESS |RESOLVED

[lng-odp] [Bug 2798] ODP random and other crypto functions (possibly) not thread safe?

2017-01-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2798 --- Comment #5 from Bill Fischofer --- Merged Commit ID 46e088c2f56d1463a598b4ac23308e19bf2b9031 -- You are receiving this mail because: You are on the CC list for the bug.

Re: [lng-odp] [PATCH 0/8] First ABI files

2017-01-18 Thread Mike Holmes
Sorry it is not there, I had that with Yi a while back it took an hour to appear on the list and patchworks, for direct recipients it was fine On 18 January 2017 at 13:23, Mike Holmes wrote: > I see it on patch works, I think it is this serise >

Re: [lng-odp] [PATCH 0/8] First ABI files

2017-01-18 Thread Mike Holmes
I see it on patch works, I think it is this serise http://patches.opendataplane.org/project/lng-odp/list/?series=3 On 18 January 2017 at 13:12, Bill Fischofer wrote: > Petri, did Part 1/8 of this series not get posted? It doesn't seem to > be on either the mailing list

Re: [lng-odp] [PATCH 0/8] First ABI files

2017-01-18 Thread Bill Fischofer
Petri, did Part 1/8 of this series not get posted? It doesn't seem to be on either the mailing list or patchworks. Bill On Wed, Jan 18, 2017 at 6:56 AM, Petri Savolainen wrote: > Added initial version of couple of ABI specification files. These files define >

Re: [lng-odp] ODP crypto support with HW offload

2017-01-18 Thread Mike Holmes
On 18 January 2017 at 11:20, Bill Fischofer wrote: > +cc ODP mailing list. These sort of discussions belong on the mailing > list as that's where those most knowledgeable and impacted by API > changes will see it. > > ODP implementations are free to batch/burst

Re: [lng-odp] ODP crypto support with HW offload

2017-01-18 Thread Bill Fischofer
+cc ODP mailing list. These sort of discussions belong on the mailing list as that's where those most knowledgeable and impacted by API changes will see it. ODP implementations are free to batch/burst individual odp_crypto_operation() calls but extensions to support application configuration of

Re: [lng-odp] [PATCH] update API version number from v1.12.0.0 to v1.13.0

2017-01-18 Thread Maxim Uvarov
Merged, Maxim. On 01/18/17 16:47, Mike Holmes wrote: > subject v1.13.0.0 <- extra 0 > > But looks good to me if that can be changed on commit > thanks. > On 18 January 2017 at 07:23, Maxim Uvarov wrote: >> >> Signed-off-by: Maxim Uvarov >

Re: [lng-odp] [NEXT PATCHv2] changelog: summary of changes for odp v1.13.0.0

2017-01-18 Thread Maxim Uvarov
Merged, Maxim. On 01/18/17 18:29, Mike Holmes wrote: > On 13 January 2017 at 11:14, Bill Fischofer wrote: >> Signed-off-by: Bill Fischofer > > Reviewed-by: Mike Holmes > >> --- >> CHANGELOG | 178 >>

Re: [lng-odp] [NEXT PATCHv2] changelog: summary of changes for odp v1.13.0.0

2017-01-18 Thread Mike Holmes
On 13 January 2017 at 11:14, Bill Fischofer wrote: > Signed-off-by: Bill Fischofer Reviewed-by: Mike Holmes > --- > CHANGELOG | 178 > ++ > 1 file

Re: [lng-odp] [API-NEXT PATCHv8 1/6] linux-gen: _ishm: adding buddy and slab allocation

2017-01-18 Thread Maxim Uvarov
replied to wrong thread. Applied v8 to api-next. Maxim. On 01/18/17 08:59, Yi He wrote: > Reviewed the difference between v7 and v8, tested with make check, for > this series: > > Reviewed-and-tested-by: Yi He > > > > > On 18 January 2017 at 02:17,

Re: [lng-odp] [API-NEXT PATCHv7 1/6] linux-gen: _ishm: adding buddy and slab allocation

2017-01-18 Thread Maxim Uvarov
Merged, Maxim. On 01/13/17 10:33, Yi He wrote: > For this patch series: > > Reviewed-and-tested-by: Yi He > > > > On 13 January 2017 at 15:55, Christophe Milard > wrote: > >> _ishm now provides functions to create/destroy pools for buddy/slab

Re: [lng-odp] [PATCH] update API version number from v1.12.0.0 to v1.13.0

2017-01-18 Thread Maxim Uvarov
On 01/18/17 16:47, Mike Holmes wrote: > subject v1.13.0.0 <- extra 0 ok, but I prefer merge changelog before that patch. Is changelog commit good for you? (I'm ok with changelog.) Maxim. > > But looks good to me if that can be changed on commit > > On 18 January 2017 at 07:23, Maxim Uvarov

Re: [lng-odp] [PATCH] update API version number from v1.12.0.0 to v1.13.0

2017-01-18 Thread Mike Holmes
subject v1.13.0.0 <- extra 0 But looks good to me if that can be changed on commit On 18 January 2017 at 07:23, Maxim Uvarov wrote: > > Signed-off-by: Maxim Uvarov Reviewed-by: Mike Holmes > --- > configure.ac | 6

[lng-odp] [PATCH 0/8] First ABI files

2017-01-18 Thread Petri Savolainen
Added initial version of couple of ABI specification files. These files define types/values that are implementation specific in the API spec. For ABI compatibility, application and implementation need to use these definitions (based on the target architecture). ABI is defined by CPU ISA

[lng-odp] [PATCH 8/8] abi: shm: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Signed-off-by: Petri Savolainen --- .../arch/arm32-linux/odp/api/abi/shared_memory.h | 7 +

[lng-odp] [PATCH 6/8] abi: pool: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Signed-off-by: Petri Savolainen --- example/timer/odp_timer_test.c | 2 +-

[lng-odp] [PATCH 4/8] abi: crypto: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Signed-off-by: Petri Savolainen --- include/odp/arch/arm32-linux/odp/api/abi/crypto.h | 7 +

[lng-odp] [PATCH 2/8] abi: buffer: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Signed-off-by: Petri Savolainen --- include/odp/arch/arm32-linux/odp/api/abi/buffer.h | 7 +

[lng-odp] [PATCH 3/8] abi: classifier: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Removed unused type definitions, documentation of those need to be still removed for API spec. Signed-off-by: Petri Savolainen

[lng-odp] [PATCH 5/8] abi: packet: added initial definitions

2017-01-18 Thread Petri Savolainen
Initially, use the same default ABI file for all architectures. Default values match those defined in odp-linux implementation to minimize changes in this phase. Signed-off-by: Petri Savolainen --- include/odp/arch/arm32-linux/odp/api/abi/packet.h | 7

Re: [lng-odp] [PATCH] travis: better comments to generate token in the travis file

2017-01-18 Thread Christophe Milard
On 18 January 2017 at 13:26, Maxim Uvarov wrote: > On 01/18/17 16:14, Christophe Milard wrote: >> Signed-off-by: Christophe Milard >> --- >> .travis.yml | 18 ++ >> 1 file changed, 18 insertions(+) >> >> diff --git

Re: [lng-odp] [PATCH] travis: better comments to generate token in the travis file

2017-01-18 Thread Maxim Uvarov
On 01/18/17 16:14, Christophe Milard wrote: > Signed-off-by: Christophe Milard > --- > .travis.yml | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index 03e61b1..909f0a3 100644 > --- a/.travis.yml > +++

[lng-odp] [PATCH] update API version number from v1.12.0.0 to v1.13.0

2017-01-18 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov --- configure.ac | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 3a209597..67d60d24 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ AC_PREREQ([2.5]) # Set correct API

[lng-odp] [PATCH] travis: better comments to generate token in the travis file

2017-01-18 Thread Christophe Milard
Signed-off-by: Christophe Milard --- .travis.yml | 18 ++ 1 file changed, 18 insertions(+) diff --git a/.travis.yml b/.travis.yml index 03e61b1..909f0a3 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,11 +6,29 @@ # pushing to github/master will