[lng-odp] Fwd: Marking API

Hi, FYI. Went through these old discussions while updating Traffic manager documentation and felt the technical discussion might benefit the group. Regards, Bala -- Forwarded message -- From: Barry Spinney Date: 15 April 2016 at 23:41 Subject: RE: Marking

Re: [lng-odp] Fwd: No huge pages, fall back to normal pages

mmap is failing in "odp_shm_reserve" function (odp_queue_init_global() ---> odp_shm_reserve() ---> odp_shm_reserve()) debug logs: root@ubuntu-15-10:/home/linaro/linaro/odp/example/classifier# ./odp_classifier -i eno1 -m 0 -p "ODP_PMR_SIP_ADDR:192.168.10.11::queue1" -p

[lng-odp] [Bug 2211] clang fails with 32bit build for odp_timer.c

https://bugs.linaro.org/show_bug.cgi?id=2211 Bill Fischofer changed: What|Removed |Added Assignee|ivan.khoronz...@linaro.org

[lng-odp] [PATCH] linux-generic: timer: change assert to account for padding

The tick_buf_t struct may be larger than 16 bytes when a lock char is needed so correct the ODP_STATIC_ASSERT to reflect this. This addresses bug https://bugs.linaro.org/show_bug.cgi?id=2211 when compiling with clang. Signed-off-by: Bill Fischofer ---

[lng-odp] ODP Addressing Model

The purpose of this thread is to summarize the discussions we've had over the past week on this topic and to foster discussion. The goal here is to reach consensus on a complete specification for ODP in this area that can be added to the User Guide for Monarch, as well as to form the basis for any

Re: [lng-odp] [PATCH 1/8] doc: api guide: remove incorrect include

For this series: Reviewed-and-tested-by: Bill Fischofer < bill.fischo...@linaro.org> On Mon, May 9, 2016 at 3:43 PM, Mike Holmes wrote: > The api guide should not reference the users guide > > Signed-off-by: Mike Holmes > --- >

Re: [lng-odp] [API-NEXT, RFC, 1/1] api: cpu: performance profiling start/stop

On Mon, May 9, 2016 at 1:50 AM, Yi He wrote: > Hi, Bill > > Thanks very much for your detailed explanation. I understand the > programming practise like: > > /* Firstly developer got a chance to specify core availabilities to the > * application instance. > */ >

[lng-odp] [PATCH 7/8] helper: add strong types documentation section

Signed-off-by: Mike Holmes --- helper/include/odp/helper/strong_types.h | 8 1 file changed, 8 insertions(+) diff --git a/helper/include/odp/helper/strong_types.h b/helper/include/odp/helper/strong_types.h index 777e24d..13e35a4 100644 ---

[lng-odp] [PATCH 8/8] helper: add chksum documentation section

Signed-off-by: Mike Holmes --- helper/include/odp/helper/chksum.h | 8 1 file changed, 8 insertions(+) diff --git a/helper/include/odp/helper/chksum.h b/helper/include/odp/helper/chksum.h index 78efa08..07d0be3 100644 --- a/helper/include/odp/helper/chksum.h

[lng-odp] [PATCH 6/8] helper: add linux documentation section

Signed-off-by: Mike Holmes --- helper/include/odp/helper/linux.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/helper/include/odp/helper/linux.h b/helper/include/odp/helper/linux.h index 7a6504f..e2dca35 100644 --- a/helper/include/odp/helper/linux.h +++

[lng-odp] [PATCH 5/8] doc: helper guide: add initial version

Signed-off-by: Mike Holmes --- configure.ac | 6 +- doc/Makefile.am | 2 +- doc/helper-guide/.gitignore | 1 + doc/helper-guide/Doxyfile| 45 doc/helper-guide/Makefile.am | 6 ++

[lng-odp] [PATCH 3/8] doc: api guide: look directly in api guide dir for src

Don't look in all of the doc subdir for api guide docs Signed-off-by: Mike Holmes --- doc/application-api-guide/Doxyfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/application-api-guide/Doxyfile b/doc/application-api-guide/Doxyfile index

[lng-odp] [PATCH 2/8] doc: api guide: use default cfg file name Doxyfile

Signed-off-by: Mike Holmes --- configure.ac| 2 +- doc/application-api-guide/{doxygen.cfg => Doxyfile} | 0 m4/ax_prog_doxygen.m4 | 2 +- 3 files changed, 2 insertions(+), 2 deletions(-) rename

[lng-odp] [PATCH 1/8] doc: api guide: remove incorrect include

The api guide should not reference the users guide Signed-off-by: Mike Holmes --- doc/application-api-guide/doxygen.cfg | 1 - 1 file changed, 1 deletion(-) diff --git a/doc/application-api-guide/doxygen.cfg b/doc/application-api-guide/doxygen.cfg index

Re: [lng-odp] [PATCH 2/2] doc: api guide: just use ARCH_DIR, drop WITH_ARCH

On 9 May 2016 at 11:40, Bill Fischofer wrote: > This patch applies and builds fine by itself, but when combined with your > previous patch I get this on 32-bit systems: > > bill@Ub16-32:~/linaro/mikepatch$ make > Making all in platform/linux-generic > make[1]: Entering

Re: [lng-odp] [PATCH 2/2] doc: api guide: just use ARCH_DIR, drop WITH_ARCH

This patch applies and builds fine by itself, but when combined with your previous patch I get this on 32-bit systems: bill@Ub16-32:~/linaro/mikepatch$ make Making all in platform/linux-generic make[1]: Entering directory '/home/bill/linaro/mikepatch/platform/linux-generic' CC

[lng-odp] [PATCH 2/2] doc: api guide: just use ARCH_DIR, drop WITH_ARCH

Signed-off-by: Mike Holmes --- configure.ac | 1 - doc/application-api-guide/doxygen.cfg | 2 +- m4/ax_prog_doxygen.m4 | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

As we discussed in our call, we can both play around with graphviz and see if a prettier diagram can be constructed that is perhaps clearer, possibly with two separate FSMs. On Mon, May 9, 2016 at 9:39 AM, Christophe Milard < christophe.mil...@linaro.org> wrote: > > > On 9 May 2016 at 16:28,

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

merged On 7 May 2016 at 19:16, Bill Fischofer wrote: > > > On Thu, Apr 21, 2016 at 8:32 AM, 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

Re: [lng-odp] [PATCH] linux-generic: correctly set arch as undefined

On Mon, May 9, 2016 at 6:59 AM, Mike Holmes wrote: > arch is defined as an architecture flag so that optimisations or > necessary differences can be accommodated. When we are unable to > determine the architecture it is incorrect to assign it the value of an > operating

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

On 9 May 2016 at 16:28, Bill Fischofer wrote: > > > On Mon, May 9, 2016 at 7:58 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> I am a bit confused by this diagram: It feels to me that timers and >> timeout have separate lives (even , if of course

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

On Mon, May 9, 2016 at 7:58 AM, Christophe Milard < christophe.mil...@linaro.org> wrote: > I am a bit confused by this diagram: It feels to me that timers and > timeout have separate lives (even , if of course some dependency exist). > From this diagram, it feels like these 2 separate objects

[lng-odp] ODP ARCH notes

https://collaborate.linaro.org/display/LNG/2016-05-09+ODP+ARCH -- Mike Holmes Technical Manager - Linaro Networking Group Linaro.org *│ *Open source software for ARM SoCs "Work should be fun and collaborative, the rest follows"

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

I am a bit confused by this diagram: It feels to me that timers and timeout have separate lives (even , if of course some dependency exist). >From this diagram, it feels like these 2 separate objects (timers and time out events) share states...? do they? In my head, and from the understanding I

[lng-odp] [PATCH] linux-generic: correctly set arch as undefined

arch is defined as an architecture flag so that optimisations or necessary differences can be accommodated. When we are unable to determine the architecture it is incorrect to assign it the value of an operating system. Instead lets label it correctly as a default and add an error to the user

[lng-odp] Fwd: No huge pages, fall back to normal pages

Hi All, Pleas help me in running ODP classifier example with huge pages. In ubuntu 15.10 by default interface naming as "eno1" and value in "/proc/sys/vm/nr_hugepages." is 1024. classifier example program could not able to use huge pages even though nr_hugepages value is non zero. I could able to

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

It’s the other applications (like trex, etc … old and new external applications) that will need to explicitly create N pthreads or M processes. True - as with any helper function - an application can do the same sequence of system and ODP call by itself, but it’s easier with helpers. With

Re: [lng-odp] [API-NEXT, RFC, 1/1] api: cpu: performance profiling start/stop

Hi, Bill Thanks very much for your detailed explanation. I understand the programming practise like: /* Firstly developer got a chance to specify core availabilities to the * application instance. */ odp_init_global(... odp_init_t *param->worker_cpus & ->control_cpus ... ) *So It is possible

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

The test exists already in the "running things in process mode" patch series (reviewed by Brian). Hopefully I do test what I wrote there. However his patch series does not add any test for the fork*() helper functions as these functions are removed in the last patch of the series. I still think

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

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