Re: [lng-odp] [PATCH] helper: remove unused odph_linux_process_fork API

2016-05-09 Thread Christophe Milard
till think these should be removed (see separate answer to Petri). Christophe. On 3 May 2016 at 14:27, Mike Holmes <mike.hol...@linaro.org> wrote: > > > On 3 May 2016 at 07:39, Christophe Milard <christophe.mil...@linaro.org> > wrote: > >> There is a test for th

Re: [lng-odp] [PATCH] helper: remove unused odph_linux_process_fork API

2016-05-09 Thread Christophe Milard
I am not sure I really agree with that, Petri: as of today, none of the written validation tests, performance tests nor example needed to do the difference: of course, that does not mean it will never be needed, but I don't think we should be adding (nor keeping) unused code promoting a behaviour

Re: [lng-odp] [GIT PULL ODPv5] running things in process mode

2016-05-03 Thread Christophe Milard
Thanks for the time you took for it, Brian! On 3 May 2016 at 15:56, Brian Brooks <brian.bro...@linaro.org> wrote: > v5 LGTM (looks good to me). Thanks Christophe! > > Reviewed-by: Brian Brooks <brian.bro...@linaro.org> > > On 05/03 11:44:47, Christophe Milard wrote: &

Re: [lng-odp] [PATCH] helper: remove unused odph_linux_process_fork API

2016-05-03 Thread Christophe Milard
There is a test for these new functions in the patch series. Hope this is what you meant by "it". Christophe On 3 May 2016 at 13:35, Mike Holmes <mike.hol...@linaro.org> wrote: > > > On 3 May 2016 at 07:32, Christophe Milard <christophe.mil...@linaro.org>

Re: [lng-odp] [PATCH] helper: remove unused odph_linux_process_fork API

2016-05-03 Thread Christophe Milard
Please don't do that: It will just create an extra rebase on my "running things in process mode " patch series which replaces all this (and therefore delete these also) Christophe. On 2 May 2016 at 14:01, Krishna Garapati wrote: > > > On 2 May 2016 at 13:47,

[lng-odp] [GIT PULL ODPv5] running things in process mode

2016-05-03 Thread Christophe Milard
ode (2016-05-03 11:39:44 +0200) -------- Christophe Milard (36): helpers: adding command line argument parsing validation: common: adding command line argument parsing validation: shmem: adding command line argument parsing h

Re: [lng-odp] [GIT PULL ODPv4] running things in process mode

2016-05-02 Thread Christophe Milard
Hi Brian I have commented below, but it looks we need to talk. would you have time after the linaro sync call today, for a HO? On 29 April 2016 at 23:49, Brian Brooks <brian.bro...@linaro.org> wrote: > On 04/29 13:29:03, Christophe Milard wrote: > > On 29 April 2016 at 00:4

[lng-odp] [PATCH] configure.ac: remove duplicated line

2016-04-29 Thread Christophe Milard
Removed an instance AM_CONDITIONAL for test_example, as this line was twice in the file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- configure.ac | 1 - 1 file changed, 1 deletion(-) diff --git a/configure.ac b/configure.ac index f88c36b..f138031

Re: [lng-odp] [GIT PULL ODPv4] running things in process mode

2016-04-29 Thread Christophe Milard
On 29 April 2016 at 00:48, Brian Brooks <brian.bro...@linaro.org> wrote: > On 04/28 17:18:36, Christophe Milard wrote: > > Since v3: > > -fixed rebase error (Christophe) > > -rebased > > Thanks for the rebase. test_in_process_mode_v4 merged cleanly into > or

Re: [lng-odp] RFC: inter-process dynamic shared memory support.

2016-04-29 Thread Christophe Milard
Thanks, Barry, I definitively think that is a good candidate for the arch call. Even if I am not 100% sure I fully understand the whole picture, I this text raise a few questions: 1) is this plain user space code, or are there related kernel modules involved? I am guessing user space, but please

Re: [lng-odp] [PATCH] validation: lock: tuning the iteration number

2016-04-28 Thread Christophe Milard
ping On 21 April 2016 at 15:32, Christophe Milard <christophe.mil...@linaro.org> wrote: > fixing: https://bugs.linaro.org/show_bug.cgi?id=2108 > > The no_lock_functional_test does not really tests the ODP functionality: > Instead, it actually checks that race conditions can

[lng-odp] [GIT PULL ODPv4] running things in process mode

2016-04-28 Thread Christophe Milard
he git repository at: https://git.linaro.org/people/christophe.milard/odp.git test_in_process_mode_v4 for you to fetch changes up to d833f72880758a12885775847a6bda558253: helper: removing dead code (2016-04-28 17:04:14 +0200) -------- Christo

Re: [lng-odp] [PATCH v2] DEPENDENCIES: add codespell for checkpatch

2016-04-28 Thread Christophe Milard
Got cheated by google mail! this is fine! Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> On 28 April 2016 at 07:52, Christophe Milard <christophe.mil...@linaro.org> wrote: > How come your v2 just shows the diff compared to v1? and not the complete > patch again?

Re: [lng-odp] [API-NEXT PATCH 1/2] doc: support ascidoctor

2016-04-28 Thread Christophe Milard
why are these patches submitted against api-next? they are not related to any api changes as I can see? (you are the one that know these stuffs: I guess you have a good reason that I am just missing...:-) ) Christophe On 27 April 2016 at 23:59, Mike Holmes wrote: >

Re: [lng-odp] [PATCH v2] DEPENDENCIES: add codespell for checkpatch

2016-04-27 Thread Christophe Milard
How come your v2 just shows the diff compared to v1? and not the complete patch again? Has the first patch been merged already? (looks like your v2 has to be applied in v1) other than that, for the patch contents, you can apply my reviewed by on the fix (Reviewed-by: Christophe Milard

[lng-odp] [GIT PULL ODPv3] running things in process mode

2016-04-27 Thread Christophe Milard
st_in_process_mode_v3 for you to fetch changes up to 71e38c2d49b4e4b7649ddebcb69fb680f46139cc: helper: removing dead code (2016-04-27 14:11:58 +0200) -------- Christophe Milard (36): helpers: adding command line argument parsing

Re: [lng-odp] [API-NEXT PATCH] validation: crypto: verify odp crypto capability

2016-04-22 Thread Christophe Milard
On 22 April 2016 at 16:14, Balakrishna Garapati < balakrishna.garap...@linaro.org> wrote: > test update to verify the crypto capability fucntionality > > Signed-off-by: Balakrishna Garapati > --- > test/validation/crypto/odp_crypto_test_inp.c | 28 >

Re: [lng-odp] shmem in process mode.

2016-04-22 Thread Christophe Milard
On 21 April 2016 at 18:30, Gary S. Robertson <ga...@cybertechsystemsinc.com> wrote: > > > On 04/21/2016 02:03 AM, Christophe Milard wrote: > > Hi, > Indeed, if the main ODP instantiation process performs a huge virtual > space reservation, e.g something like:

[lng-odp] [GIT PULL ODPv2] running things in process mode

2016-04-21 Thread Christophe Milard
ople/christophe.milard/odp.git test_in_process_mode_v2 for you to fetch changes up to 53a81edc1c77dc7aac08d97cd28e8cc5f7d9d842: helper: removing dead code (2016-04-21 17:26:26 +0200) -------- Christophe Milard (56): helpers: adding command line arg

[lng-odp] [PATCH] validation: lock: tuning the iteration number

2016-04-21 Thread Christophe Milard
iterations performed). The solution taken here is actually to tune the critical section running time (currentely to ITER_MPLY_FACTOR=3 times the time needed to note the first race condition). This means that the test will take longer to run with valgrind, but will remain short without valgrind. Signed-off-b

[lng-odp] shmem in process mode.

2016-04-21 Thread Christophe Milard
Hi, This is partly an answer to Gary's mail included below. it is about shmem allocation in process mode. I don't think I really agree with you, Gary: As Ola, I believe it is important (for performance) that shmem areas are seen at the same address in their virtual space on different ODP threads,

Re: [lng-odp] [GIT PULL ODP] running things in process mode

2016-04-19 Thread Christophe Milard
e). Thread creation is so common task for any app that those are >> surely needed. Implementation of the agnostic and not agnostic functions >> may share the core parts of the code. >> >> >> >> -Petri >> >> >> >> >> >> *From:

Re: [lng-odp] [GIT PULL ODP] running things in process mode

2016-04-19 Thread Christophe Milard
On 19 April 2016 at 09:46, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 19.04.2016 10:40, Christophe Milard wrote: > >> Thanks for your comment, Petri. You are welcome to point out the coding >> guideline faults you have seen, so I don't miss them. I will re

Re: [lng-odp] [GIT PULL ODP] running things in process mode

2016-04-19 Thread Christophe Milard
de"), since not all apps want to obscure whether process or >> pthread is used. Opaque thread type helps our purposes to test everything >> on both pthread and process models, but other apps are likely to care only >> one model. >> >> -Petri >> >>

Re: [lng-odp] [GIT PULL ODP] running things in process mode

2016-04-19 Thread Christophe Milard
rocess models, but other apps are likely to care only > one model. > > -Petri > > > > > -Original Message- > > From: EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > > Sent: Monday, April 18, 2016 5:00 PM > > To: lng-odp@lists.linaro

[lng-odp] [GIT PULL ODP] running things in process mode

2016-04-18 Thread Christophe Milard
dead code (2016-04-18 15:25:48 +0200) -------- Christophe Milard (56): helpers: adding command line argument parsing validation: common: adding command line argument parsing validation: shmem: adding command line argumen

Re: [lng-odp] [PATCH] linux-generic: test: shmem: coverity fix

2016-04-14 Thread Christophe Milard
Ping: time for merge? On 4 April 2016 at 18:23, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Mon, Apr 4, 2016 at 4:29 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> Fixes: https://bugs.linaro.org/show_bug.cgi?id=2148 (CID 15939

Re: [lng-odp] [PATCHv2] linux-generic: test: shmem: close fifo

2016-04-14 Thread Christophe Milard
Ping... On 1 April 2016 at 11:47, Christophe Milard <christophe.mil...@linaro.org> wrote: > Fixes: https://bugs.linaro.org/show_bug.cgi?id=2147 (CID 159394) > The fifo is closed at end of test > > Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> > ---

Re: [lng-odp] [PATCH] platform/Makefile.inc: remove incorrect install to share dir

2016-04-12 Thread Christophe Milard
On 2016-04-11 22:36, Anders Roxell wrote: > Remove libodp-linux.la from /usr/share/... its already installed > in /usr/lib/ > > Signed-off-by: Anders Roxell <anders.rox...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > platform

Re: [lng-odp] [PATCH] linux-generic: pktio_run_tap: failure to setup tap != test fail

2016-04-07 Thread Christophe Milard
On 2016-04-04 23:30, Zi Shen Lim wrote: > When we fail to setup tap for any reason, simply report > TEST_SKIPPED and move on. It is not an actual test failure. > > Signed-off-by: Zi Shen Lim <zlim@gmail.com> Reviewed-by: Christophe Milard <christophe.mil...@linaro.or

[lng-odp] [PATCHv6] linux-generic: test: shmem: atomic check+open fifo

2016-04-06 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2146 (CID 159395) The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/test/shmem/shmem_linux.

Re: [lng-odp] [PATCHv5] linux-generic: test: shmem: atomic check+open fifo

2016-04-06 Thread Christophe Milard
t 02:18, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Tue, Apr 5, 2016 at 3:03 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> Fixes: https://bugs.linaro.org/show_bug.cgi?id=2146 (CID 159395) >> The open system call is direc

[lng-odp] [PATCHv5] linux-generic: test: shmem: atomic check+open fifo

2016-04-05 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2146 (CID 159395) The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v4: test after loop (Maxim) since v3:

Re: [lng-odp] [PATCH 2/2] doc: install docs

2016-04-04 Thread Christophe Milard
On 2016-03-30 14:34, Mike Holmes wrote: > Signed-off-by: Mike Holmes <mike.hol...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > doc/implementers-guide/Makefile.am | 1 + > doc/process-guide/Makefile.am | 2 ++ > doc/users-g

Re: [lng-odp] [PATCH 1/2] doc: rebuild only when required

2016-04-04 Thread Christophe Milard
On 2016-03-30 14:34, Mike Holmes wrote: > Signed-off-by: Mike Holmes <mike.hol...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > doc/Makefile.inc | 3 +++ > doc/implementers-guide/Makefile.am | 3 +-- > doc/process

[lng-odp] [PATCH] linux-generic: test: shmem: string size fix

2016-04-04 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2149 (CID 159392) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/test/shmem/shmem_linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/test/shmem/shmem_l

[lng-odp] [PATCHv4] linux-generic: test: shmem: atomic check+open fifo

2016-04-04 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2146 (CID 159395) The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v3: nb_sec incremented in loop rather then a

[lng-odp] [PATCH] linux-generic: test: shmem: coverity fix

2016-04-04 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2148 (CID 159393) The if statement introduced here is redundant with the previous CU_ASSERT_FATAL, but avoids the coverity warning. (coverity probably misses the longjump in CU_ASSERT_FATAL) Signed-off-by: Christophe Milard <christophe.

[lng-odp] [PATCHv2] linux-generic: test: shmem: close fifo

2016-04-01 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2147 (CID 159394) The fifo is closed at end of test Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v1: bug URL added in commit msg (Anders) platform/linux-generic/test/shmem/shmem_odp.c | 1 + 1 file changed, 1 ins

[lng-odp] [PATCHv3] linux-generic: test: shmem: atomic check+open fifo

2016-04-01 Thread Christophe Milard
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2146 (CID 159395) The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v2: bug URL added (Anders) since v1: change

[lng-odp] [PATCH] linux-generic: test: shmem: close fifo

2016-04-01 Thread Christophe Milard
Fix for bug 2147, CID 159394: The fifo is closed at end of test Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/test/shmem/shmem_odp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/platform/linux-generic/test/shmem/shmem_odp.c b/platform

Re: [lng-odp] [API-NEXT PATCHv2 1/2] drv: adding compiler.h

2016-03-30 Thread Christophe Milard
ping for merge... On 18 February 2016 at 02:25, Bill Fischofer <bill.fischo...@linaro.org> wrote: > For this series: > > Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> > > On Wed, Feb 17, 2016 at 3:32 AM, Christophe Milard < > ch

[lng-odp] [PATCHv2] linux-generic: test: shmem: atomic check+open fifo

2016-03-30 Thread Christophe Milard
Fix for bug 2146, CID 159395: The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v1: changed loop to avoid open() line duplication (Maxim) platform/linux-generi

[lng-odp] [PATCH] linux-generic: test: shmem: atomic check+open fifo

2016-03-29 Thread Christophe Milard
Fix for bug 2146, CID 159395: The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/test/shmem/shmem_linux.c | 5 +++-- 1 file changed, 3 insertions

Re: [lng-odp] [PATCH 1/2] linux-generic: test: Mimic the test validation directory structure

2016-03-29 Thread Christophe Milard
On 2016-03-21 14:34, Mike Holmes wrote: I have problem applying this patch: Applying: linux-generic: test: Mimic the test validation directory structure error: patch failed: platform/linux-generic/m4/configure.m4:24 error: platform/linux-generic/m4/configure.m4: patch does not apply error: patch

Re: [lng-odp] [PATCHv2 0/2] linux test for shared memory

2016-03-25 Thread Christophe Milard
t;> +++ b/platform/linux-generic/test/shmem/shmem_linux.c >> @@ -1,4 +1,4 @@ >> -/* Copyright (c) 2014, Linaro Limited >> +/* Copyright (c) 2016, Linaro Limited >> * All rights reserved. >> * >> * SPDX-License-Identifier: BSD-3-Clause >> >>

Re: [lng-odp] [PATCHv2 2/2] linux-generic: test: ODP_SHM_PROC test

2016-03-22 Thread Christophe Milard
Maxim. Would you mind changing these dates when merging? Otherwise wait next week. ... no computer here (Cambodia). No much Internet either. Thanks. See you next week from Sweden On 23 Mar 2016 06:35, "Mike Holmes" <mike.hol...@linaro.org> wrote: > > > On 2 March 2016 at

Re: [lng-odp] [API-NEXT PATCHv3 3/3] doc: userguide: add PktIO chapter to ODP User Guide

2016-03-07 Thread Christophe Milard
ace capability. Queue type is defined by the > + * input mode. The default value is 1. */ > + unsigned num_queues; > + > + /** Queue parameters for creating input queues in ODP_PKTIN_MODE_QUEUE > + * or ODP_PKTIN_MODE_SCHED modes. Scheduler parameters are cons

Re: [lng-odp] [API-NEXT PATCHv2 3/3] doc: userguide: add PktIO chapter to ODP User Guide

2016-03-03 Thread Christophe Milard
. Fix what you can: This text is much better that the emptiness that was before. On 3 March 2016 at 17:12, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Thu, Mar 3, 2016 at 4:00 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> On 2016-

Re: [lng-odp] [API-NEXT PATCHv2 3/3] doc: userguide: add PktIO chapter to ODP User Guide

2016-03-03 Thread Christophe Milard
On 2016-03-02 04:35, Bill Fischofer wrote: > Signed-off-by: Bill Fischofer > --- > doc/users-guide/users-guide-pktio.adoc | 605 > + > doc/users-guide/users-guide.adoc | 2 + > 2 files changed, 607 insertions(+) > create mode

[lng-odp] [PATCHv2 2/2] linux-generic: test: ODP_SHM_PROC test

2016-03-02 Thread Christophe Milard
Adding a platform side test to test sharing memory between ODP and linux. Also tests that the shared memory device name (under /dev/shm/) is scoped. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> -

[lng-odp] [PATCHv2 1/2] linux-generic: test: adding odp includes

2016-03-02 Thread Christophe Milard
A new Makefile.inc is created, defining common includes that linux specific tests will need if they need to build with ODP. Tests just acting as wrappers around platform agnostic tests will not need these defines, whereas tests building ODP executable will. Signed-off-by: Christophe Milard

[lng-odp] [PATCHv2 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
were just wrappers to generic platform-agnostic test) *tests that the shared memory device name (under /dev/shm/) is scoped (under linux) Christophe Milard (2): linux-generic: test: adding odp includes linux-generic: test: ODP_SHM_PROC test platform/linux-generic/m4/configure.m4

Re: [lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
ed-by: Bill Fischofer <bill.fischo...@linaro.org> > > On Wed, Mar 2, 2016 at 7:30 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> shmem_odp should not be run alone: it is forked and execv'd by shmem >> linux. If you run make check, only shmem_linux is

Re: [lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
t; > On Wed, Mar 2, 2016 at 7:19 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> Make shmem_odp will hang alone: it needs its linux counterpart. Make >> should fix that for you. I assume as soon as we sort out why you have this >> make pro

Re: [lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
other comments :-) Many thanks for looking at it! On 2 March 2016 at 14:06, Christophe Milard <christophe.mil...@linaro.org> wrote: > shouldn't you have --enable-test-vald as well? > > If I run: > > ./bootstrap && ./configure > --with-cunit-path=/home/erachm

Re: [lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
ake > > Result: nothing built in platforms/linux-generic/test/shmem > > If I go into that directory and type make then shmem_odp is built, but > when I try to run it it hangs. > > On Wed, Mar 2, 2016 at 6:32 AM, Christophe Milard < > christophe.mil...@linaro.org> w

Re: [lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-02 Thread Christophe Milard
I manually make the test, running it seems to hang. Haven't > investigated further. > > On Tue, Mar 1, 2016 at 10:34 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> (Note: requires: linux-generic: shmem: odp scope in /dev/shmem) >> Addin

Re: [lng-odp] [API-NEXT PATCH 1/3] doc: images: add gv file for pktio state machine

2016-03-02 Thread Christophe Milard
p patching, run "git am --abort". erachmi@erachmi-ericsson:~/linaro/ODP/odp$ On 2 March 2016 at 09:12, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 03/02/16 10:20, Christophe Milard wrote: > >> Hi Bill, >> I am probably missing something obvious,

Re: [lng-odp] [API-NEXT PATCH 1/3] doc: images: add gv file for pktio state machine

2016-03-01 Thread Christophe Milard
Hi Bill, I am probably missing something obvious, but that Makefile does not exist for me... Does this require some previous patch that have made their way to the repo yet? erachmi@erachmi-ericsson:~/linaro/ODP/odp$ git am

[lng-odp] [PATCH 2/2] linux-generic: test: ODP_SHM_PROP test

2016-03-01 Thread Christophe Milard
Adding a platform side test to test sharing memory between ODP and linux. Also tests that the shared memory device name (under /dev/shm/) is scoped. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/m4/configure.m4 | 1 + platform

[lng-odp] [PATCH 1/2] linux-generic: test: adding odp includes

2016-03-01 Thread Christophe Milard
A new Makefile.inc is created, defining common includes that linux specific tests will need if they need to build with ODP. Tests just acting as wrappers around platform agnostic tests will not need these defines, whereas tests building ODP executable will. Signed-off-by: Christophe Milard

[lng-odp] [PATCH 0/2] linux test for shared memory

2016-03-01 Thread Christophe Milard
were just wrappers to generic platform-agnostic test) *tests that the shared memory device name (under /dev/shm/) is scoped (under linux). Christophe Milard (2): linux-generic: test: adding odp includes linux-generic: test: ODP_SHM_PROP test platform/linux-generic/m4/configure.m4

Re: [lng-odp] [PATCH v2] configure.ac: distribute the AC_CONFIG_FILES

2016-02-29 Thread Christophe Milard
On 2016-02-26 17:29, Mike Holmes wrote: > Signed-off-by: Mike Holmes <mike.hol...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > v2 >Sort lists suggested by Maxim > > configure.ac| 44

Re: [lng-odp] [API-NEXT v3] doc: user-guide documentation for classification

2016-02-28 Thread Christophe Milard
Hi Bala. Just sent you a patch that you can apply on the top of this to correct a few formating issues.I assume you will agree. You can sent V6 with my review (see patch comments), as this document makes sense to me. It would be nice to add a section on writting PMRs (with code exemples) in the

Re: [lng-odp] [API-NEXT PATCH 1/3] drv: adding byteorder.h

2016-02-25 Thread Christophe Milard
On 25 February 2016 at 22:43, Mike Holmes <mike.hol...@linaro.org> wrote: > On 25 February 2016 at 08:17, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> I guess this is stuck untill BKK discussion...? >> > > I hope not. > > >>

Re: [lng-odp] [API-NEXT PATCH 1/3] drv: adding byteorder.h

2016-02-25 Thread Christophe Milard
I guess this is stuck untill BKK discussion...? On 18 February 2016 at 12:37, Christophe Milard < christophe.mil...@linaro.org> wrote: > A complete -partly working- driver prototype was sent as in RFC: > https://lists.linaro.org/pipermail/lng-odp/2016-January/019162.html >

Re: [lng-odp] [API-NEXT v3] doc: user-guide documentation for classification

2016-02-25 Thread Christophe Milard
On 25 February 2016 at 09:56, Balasubramanian Manoharan < bala.manoha...@linaro.org> wrote: > Signed-off-by: Balasubramanian Manoharan > --- > v3: Incorporates classification user guide to main document > Adds Practical example section > v2: Incorporates review

Re: [lng-odp] [PATCH] configure.ac: distribute the AC_CONFIG_FILES

2016-02-25 Thread Christophe Milard
On 2016-02-24 17:08, Mike Holmes wrote: > Signed-off-by: Mike Holmes <mike.hol...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > configure.ac| 44 > doc/m4/configure.m4

Re: [lng-odp] [PATCH V2 2/2] linux-generic: test: add process mode scheduler test

2016-02-25 Thread Christophe Milard
On 24 February 2016 at 22:24, Mike Holmes wrote: > Add new platform test infrastructure to run the scheduler performance > test in process mode. > > Signed-off-by: Mike Holmes > --- > platform/linux-generic/m4/configure.m4 | 1 + >

Re: [lng-odp] [PATCH V2 1/2] linux-generic: test: move validation tests to their own directory

2016-02-25 Thread Christophe Milard
On 24 February 2016 at 22:24, Mike Holmes wrote: > Simplify adding platform performance tests by mirroring the test directory > structure and moving the validation tests to their own directory under > platform also. > > Signed-off-by: Mike Holmes

[lng-odp] [PATCHv2] linux-generic: shmem: odp scope in /dev/shmem

2016-02-23 Thread Christophe Milard
The name of the shared memory (created with linux shm_open() when the ODP_SHM_PROC flag is given) is now prefixed with "/odp--" so as to name scope ODP shared mem in /dev/shmem. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischo

[lng-odp] [PATCH] linux-generic: shmem: odp scope in /dev/shmem

2016-02-23 Thread Christophe Milard
The name of the shared memory (created with linux shm_open() when the ODP_SHM_PROC flag is given) is now prefixed with "/opd--" so as to name scope ODP shared mem in /dev/shmem. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp

Re: [lng-odp] [API-NEXT PATCHv2] linux-generic: Makefile: fix broken include path

2016-02-22 Thread Christophe Milard
t; Signed-off-by: Anders Roxell <anders.rox...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > platform/linux-generic/Makefile.am | 2 +- > platform/linux-generic/arch/linux/odp/{ => api}/cpu_arch.h | 0 &g

[lng-odp] [PATCHv3 3/4] doc: user-guide: fixing other typos

2016-02-22 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> --- doc/users-guide/users-guide.adoc | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/users-guide/users-guide.adoc b/doc

[lng-odp] [PATCHv3 1/4] doc: glossary: defining ODP thread more precisely

2016-02-22 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/glossary.adoc | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/doc/glossary.adoc b/doc/glossary.adoc index 8e514ee..5408b1f 100644 --- a/doc/glossary.adoc +++ b/doc/glossar

[lng-odp] [PATCHv3 0/4] shmem docs

2016-02-22 Thread Christophe Milard
since v2: ODP thread definition changed. (Petri) since v1: fixed escape around '+' char. (Bill) Documentation updates documenting shmem and related items. Christophe Milard (4): doc: glossary: defining ODP thread more precisely doc: user-guide: shmem doc: user-guide: fixing other typos

[lng-odp] [PATCHv3 4/4] doc: removing invalid characters

2016-02-22 Thread Christophe Milard
linux "tree" command output contains non printable character 160 which made its way into the asciidoc. Removed and replaced by space here. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> ---

[lng-odp] [PATCHv3 2/4] doc: user-guide: shmem

2016-02-22 Thread Christophe Milard
section regarding shared memo added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> --- doc/users-guide/users-guide.adoc | 130 +++ 1 file changed, 130 inserti

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

2016-02-18 Thread Christophe Milard
thread (i.e a linux process calling +odp_init_local()+ will be refered as ODP thread, not ODP process). If this is not correct... what should it be :-) Christophe. On 18 February 2016 at 10:29, Christophe Milard < christophe.mil...@linaro.org> wrote: > > > On 18 Februar

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

2016-02-18 Thread Christophe Milard
On 18 February 2016 at 09:50, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > > > > > *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, February 18, 2016 10:40 AM > *To:* Savolainen, Petri (Nokia

Re: [lng-odp] [PATCHv2 2/4] doc: user-guide: shmem

2016-02-18 Thread Christophe Milard
On 18 February 2016 at 09:29, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 02/18/16 11:06, Christophe Milard wrote: > >> section regarding shared memo added. >> >> Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> >> Reviewed-

Re: [lng-odp] [PATCH API-NEXTv2] doc: user-guide documentation for classification

2016-02-18 Thread Christophe Milard
On 18 February 2016 at 08:54, Balasubramanian Manoharan < bala.manoha...@linaro.org> wrote: > Adds user-guide documentation for classification module > > Signed-off-by: Balasubramanian Manoharan > --- > v2: Incorporates review comments from Christophe > >

[lng-odp] [PATCHv2 4/4] doc: removing invalid characters

2016-02-18 Thread Christophe Milard
linux "tree" command output contains non printable character 160 which made its way into the asciidoc. Removed and replaced by space here. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> ---

[lng-odp] [PATCHv2 3/4] doc: user-guide: fixing other typos

2016-02-18 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> --- doc/users-guide/users-guide.adoc | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/users-guide/users-guide.adoc b/doc

[lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

2016-02-18 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> --- doc/glossary.adoc | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/doc/glossary.adoc b/doc/glossary.adoc index 8e51

[lng-odp] [PATCHv2 2/4] doc: user-guide: shmem

2016-02-18 Thread Christophe Milard
section regarding shared memo added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Reviewed-and-tested-by: Bill Fischofer <bill.fischo...@linaro.org> --- doc/users-guide/users-guide.adoc | 129 +++ 1 file changed, 129 inserti

[lng-odp] [PATCHv2 0/4] shmem docs

2016-02-18 Thread Christophe Milard
since v1: fixed escape around '+' char. (Bill) Documentation updates documenting shmem and related items. Christophe Milard (4): doc: glossary: defining ODP thread more precisely doc: user-guide: shmem doc: user-guide: fixing other typos doc: removing invalid characters doc

Re: [lng-odp] [API-NEXT PATCH 2/3] linux-generic: byte-order: split common part

2016-02-17 Thread Christophe Milard
bill/linaro/chrisdoc/.git/rebase-apply/patch > When you have resolved this problem, run "git am --continue". > If you prefer to skip this patch, run "git am --skip" instead. > To restore the original branch and stop patching, run "git am --abort". > > &

[lng-odp] [PATCH 4/4] doc: removing invalid characters

2016-02-17 Thread Christophe Milard
linux "tree" command output contains non printable character 160 which made its way into the asciidoc. Removed and replaced by space here. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/implementers-guide/implementers-guide.adoc | 34 +

[lng-odp] [PATCH 3/4] doc: user-guide: fixing other typos

2016-02-17 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/users-guide/users-guide.adoc | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc index 07d8949..bc23b72 100644 --- a/doc/users

[lng-odp] [PATCH 2/4] doc: user-guide: shmem

2016-02-17 Thread Christophe Milard
section regarding shared memo added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/users-guide/users-guide.adoc | 129 +++ 1 file changed, 129 insertions(+) diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide

[lng-odp] [PATCH 1/4] doc: glossary: defining ODP thread more precisely

2016-02-17 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/glossary.adoc | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/doc/glossary.adoc b/doc/glossary.adoc index 8e514ee..becaf16 100644 --- a/doc/glossary.adoc +++ b/doc/glossary.adoc @@ -9,9

[lng-odp] [API-NEXT PATCH 2/3] linux-generic: byte-order: split common part

2016-02-17 Thread Christophe Milard
The files defining byte order and related functions are split so that a common part can be re-used for other odp interfaces. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 8 ++ platform/linux-generic/include/o

[lng-odp] [API-NEXT PATCH 3/3] linux-generic: drv: adding byteorder.h

2016-02-17 Thread Christophe Milard
byteorder.h is created for the driver interface, using the common part (common to both ODP application interface and ODP driver interface). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-g

[lng-odp] [API-NEXT PATCH 1/3] drv: adding byteorder.h

2016-02-17 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/byteorder.h | 176 +++ 1 file changed, 176 insertions(+) create mode 100644 include/odp/drv/spec/byteorder.h diff --git a/include/odp/drv/spec/byteorder.h b/inclu

[lng-odp] [API-NEXT PATCHv2 2/2] linux-generic: drv: adding compiler.h

2016-02-17 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am| 4 +++ platform/linux-generic/include/odp/drv/compiler.h | 34 +++ 3 files changed, 39 inse

[lng-odp] [API-NEXT PATCHv2 1/2] drv: adding compiler.h

2016-02-17 Thread Christophe Milard
The file is, of course, largely inspired from its equivalent on the application interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/compiler.h | 51 + 1 file changed, 51 insertions(+) create mode

Re: [lng-odp] [API-NEXT PATCH 2/2] linux-generic: drv: adding compiler.h

2016-02-17 Thread Christophe Milard
Missing Makefile update. will send a v2 :-) On 17 February 2016 at 09:44, Christophe Milard < christophe.mil...@linaro.org> wrote: > Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> > --- > include/odp_drv.h | 1 + > platfo

[lng-odp] [API-NEXT PATCH 1/2] drv: adding compiler.h

2016-02-16 Thread Christophe Milard
The file is, of course, largely inspired from its equivalent on the application interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/compiler.h | 51 + 1 file changed, 51 insertions(+) create mode

<    5   6   7   8   9   10   11   12   13   14   >