Re: [lng-odp] [API-NEXT PATCHv5 1/5] drv: adding driver registration interface (stub)

On 22 December 2016 at 21:40, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 12/19/16 13:50, Christophe Milard wrote: >> The enumerator class, enumerator instance, devio and driver registration >> functions prototypes (and a draft of their parameters) are >> define

[lng-odp] [API-NEXT PATCHv2 6/6] test: drv: shm: adding buddy allocation stress tests

Stress tests for the random size allocator (buddy allocator in linux-generic) are added here. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 177 + .../common_plat/validation/drv/drvshmem/drvs

[lng-odp] [API-NEXT PATCHv2 5/6] test: drv: shm: adding basic fixed size allocation tests

Basic tests for odpdrv_shm_pool are added here, creating a fixed size pool and performing basic alloc/free on it Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 86 ++ .../common_plat/validati

[lng-odp] [API-NEXT PATCHv2 4/6] test: drv: shm: adding basic buddy allocation tests

Basic tests for odpdrv_shm_pool are added here, creating a buddy pool and performing basic alloc/free on it Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 92 ++ .../common_plat/validation/drv/dr

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

_ishm now provides functions to create/destroy pools for buddy/slab memory allocation, as well as functions to allocated/release memory from the created pools. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + pl

[lng-odp] [API-NEXT PATCHv2 3/6] linux-gen: drv: shm: adding pool allocator

Adding functions to create memory pools and allocate / free memory from the created pools. These functions calls their _ishm conterpart, of course. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/drv_shm.

[lng-odp] [API-NEXT PATCHv2 2/6] drv: adding odpdrv_shm_pool functions

is sharable between ODP threads. (using malloc would not work when ODP threads are linux processes). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 97 ++ .../linux-generic/include/odp/drv/plat/shm_types.

[lng-odp] [API-NEXT PATCHv2 0/6] small memory amount allocator for drv shm

, drivers... (PS:I guess this concept will be needed as well on the north (API) interface when we start looking into having things working "in process mode") Christophe Milard (6): linux-gen: _ishm: adding buddy and slab allocation drv: adding odpdrv_shm_pool functions linux-gen: drv: s

Re: [lng-odp] [PATCH v3] helper: do not break odp_term_global protocol

should not call > this function but leave it to the main application thread/process. > > Signed-off-by: Stanislaw Kardach <k...@semihalf.com> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > helper/linux.c | 4 > 1 file changed, 4 deletion

Re: [lng-odp] [PATCH] helper: do not break odp_term_global protocol

Still the same: Applying: helper: do not break odp_term_global protocol error: patch failed: helper/linux.c:42 error: helper/linux.c: patch does not apply Patch failed at 0001 helper: do not break odp_term_global protocol The copy of the patch that failed is found in: .git/rebase-apply/patch When

Re: [lng-odp] [PATCH] helper: do not break odp_term_global protocol

This seems a good correction. However I get the following when applying this patch: git am ~/incoming/lng-odp_PATCH_helper_do_not_break_odp_term_global_protocol.mbox Applying: helper: do not break odp_term_global protocol error: patch failed: helper/linux.c:42 error: helper/linux.c: patch does

Re: [lng-odp] [API-NEXT PATCH 1/2] linux-gen: _ishm: exporting/importing user len and flags

Ok. thx On 27 December 2016 at 16:21, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > I merged it with my review. > > Maxim. > > On 27 December 2016 at 17:57, Christophe Milard > <christophe.mil...@linaro.org> wrote: >> >> It needs revi

Re: [lng-odp] [API-NEXT PATCH 1/2] linux-gen: _ishm: exporting/importing user len and flags

> > On 12/21/16 14:11, Christophe Milard wrote: >> The size of the shared memory and its user flag set, as given at reserve >> time, are exported and imported so that odp_shm_info() return proper >> values on imported blocks >> >> Signed-off-by: Christophe Milard <chr

Re: [lng-odp] api for small buffer allocations

On 27 December 2016 at 09:58, Savolainen, Petri (Nokia - FI/Espoo) <petri.savolai...@nokia-bell-labs.com> wrote: > > >> -Original Message- >> From: Christophe Milard [mailto:christophe.mil...@linaro.org] >> Sent: Friday, December 23, 2016 3:36 PM >> To:

Re: [lng-odp] api for small buffer allocations

On 23 December 2016 at 09:15, Savolainen, Petri (Nokia - FI/Espoo) <petri.savolai...@nokia-bell-labs.com> wrote: > > >> -Original Message- >> From: Christophe Milard [mailto:christophe.mil...@linaro.org] >> Sent: Thursday, December 22, 2016 3:13 PM >&g

Re: [lng-odp] api for small buffer allocations

revert to HTML from time to time. > > > > From: Christophe Milard [mailto:christophe.mil...@linaro.org] > Sent: Thursday, December 22, 2016 11:24 AM > To: Savolainen, Petri (Nokia - FI/Espoo) > <petri.savolai...@nokia-bell-labs.com>; Mike Holmes <mike.hol..

[lng-odp] api for small buffer allocations

Hi, I am trying to sum up what was said at the arch call yesterday, regarding memory allocation to figure out how to rewrite my "buddy allocator for drv interface" patch series. - Some voices seemed to say that a single allocation would be best, regardless of the size being allocated (mostly at

[lng-odp] [API-NEXT PATCHv2] linux-gen: _ishm: fixing typos

Fixing a set of iritating typos. just in comments. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- Since V1: another typo found by Bill platform/linux-generic/_ishm.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff

[lng-odp] [API-NEXT PATCH 5/5] test: drv: shm: adding small allocation stress tests

stress tests added for allocation of small memory units using odpdrv_shm_sreserve() and odpdrv_shm_sfree(). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 164 + .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCH 4/5] test: drv: shm: adding small allocation tests

Tests for odpdrv_shm_sreserve() and odpdrv_shm_sfree are added, testing the ishm small memory unit allocator. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 77 ++ .../common_plat/validati

[lng-odp] [API-NEXT PATCH 3/5] linux-gen: drv: shm: adding buddy allocator

as process case) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/drv_shm.c | 44 +- .../linux-generic/include/odp_config_internal.h| 10 + platform/linux-generic/include/odp_internal.h | 4 ++ platform

[lng-odp] [API-NEXT PATCH 1/5] linux-gen: _ishm: adding buddy allocation

_ishm now provides a function to create a pool for buddy memory allocation, as well as functions to allocated/release memory from the created pool. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform

[lng-odp] [API-NEXT PATCH 2/5] drv: adding odpdrv_shm_sreserve and odpdrv_shm_sfree()

The two functions are meant to be used fo small amount of memory allocation. They are quicker, lighter and have a smaller granularity than the odpdrv_shm_reserve() / odpdrv_shm_free_*() sets of function, but also lacks some of their functionality (lookup, export, flags). Signed-off-by: Christophe

[lng-odp] [API-NEXT PATCH 0/5] small memory amount allocator for drv shm

it for building list of things such as enumerators, devio, drivers... (PS:I guess this concept will be needed as well on the north (API) interface when we start looking into having things working "in process mode") Christophe Milard (5): linux-gen: _ishm: adding buddy allocation d

[lng-odp] [API-NEXT PATCH] linux-gen: _ishm: fixing typos

Fixing a set of iritating typos. just in comments. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/platform/linux-generic/_ishm.c b/platform

[lng-odp] [API-NEXT PATCH 2/2] test: shm: checking exported vs imported block length

Checking that the block size returned by odp_shm_info() matches the exported block length. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/linux-generic/validation/api/shmem/shmem_odp2.c | 8 1 file changed, 8 insertions(+) diff --git a/test/linux-g

[lng-odp] [API-NEXT PATCH 1/2] linux-gen: _ishm: exporting/importing user len and flags

The size of the shared memory and its user flag set, as given at reserve time, are exported and imported so that odp_shm_info() return proper values on imported blocks Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c

Re: [lng-odp] [API-NEXT PATCHv2] linux-gen: _ishm: unlinking files asap for cleaner termination

Ping. I think this patch would ease cleaning up the Huge pages. If reviewed, it is candidate for "next" and master. Christophe. On 6 December 2016 at 18:25, Christophe Milard <christophe.mil...@linaro.org > wrote: > _ishm now unlinks the created files as soon as possi

[lng-odp] [RFC API-NEXT PATCH 5/5] test: drv: shm: adding small allocation stress tests

stress tests added for allocation of small memory units using odpdrv_shm_sreserve() and odpdrv_shm_sfree(). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 164 + .../common_plat/validation/drv/dr

[lng-odp] [RFC API-NEXT PATCH 4/5] test: drv: shm: adding small allocation tests

Tests for odpdrv_shm_sreserve() and odpdrv_shm_sfree are added, testing the ishm small memory unit allocator. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 77 ++ .../common_plat/validati

[lng-odp] [RFC API-NEXT PATCH 1/5] linux-gen: _ishm: adding buddy allocation

_ishm now provides a function to create a pool for buddy memory allocation, as well as functions to allocated/release memory from the created pool. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform

[lng-odp] [RFC API-NEXT PATCH 2/5] drv: adding odpdrv_shm_sreserve and odpdrv_shm_sfree()

The two functions are meant to be used fo small amount of memory allocation. They are quicker, lighter and have a smaller granularity than the odpdrv_shm_reserve() / odpdrv_shm_free_*() sets of function, but also lacks some of their functionality (lookup, export, flags). Signed-off-by: Christophe

[lng-odp] [RFC API-NEXT PATCH 3/5] linux-gen: drv: shm: adding buddy allocator

as process case) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/drv_shm.c | 44 +- .../linux-generic/include/odp_config_internal.h| 10 + platform/linux-generic/include/odp_internal.h | 4 ++ platform

[lng-odp] [RFC API-NEXT PATCH 0/5] small memory amount allocator for drv shm

it for building list of things such as enumerators, devio, drivers... (PS:I guess this concept will be needed as well on the north (API) interface when we start looking into having things working "in process mode") Christophe Milard (5): linux-gen: _ishm: adding buddy allocation d

[lng-odp] [API-NEXT PATCHv5 4/5] test: preventing odp.conf loading for tests

(not using make check) may be affected by configuration files. Setting ODP_SYSCONFIG_FILE to an appropriate value (possibly "none") may be required. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/Makefile.inc | 4 +++- 1 file changed, 3 insertions(+),

[lng-odp] [API-NEXT PATCHv5 5/5] linux-gen: drv_drivers: loading modules from config file

The shared objects listed in the ODP configuration files are loaded at init time. The odp configuration file list the shared objects to be loaded as shown in the following example: module = { modules = ["enumerator1.so", "driver1.so"]; }; Signed-off-by: Christophe Mi

[lng-odp] [API-NEXT PATCHv5 3/5] linux-gen: init: adding configuration file parsing

The parsing of the odp.conf configuration file is added. The file is searched first in the local directory (.) and then is the $prefix/etc directory. This requires libconfig (sudo apt-get install libconfig-dev) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- DEPEND

[lng-odp] [API-NEXT PATCHv5 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

The linux implementation for the enumerator class registration function, enumerator instance registration function, devio and driver registration functions (stub) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + pl

[lng-odp] [API-NEXT PATCHv5 0/5] driver initialisation framework

int an error message). odp_driver_register() is of course part of the driver interface (south). Christophe Milard (5): drv: adding driver registration interface (stub) linux-gen: adding enum, devio and driver registration interface (stub) linux-gen: init: adding configuration file parsing test:

[lng-odp] [API-NEXT PATCHv5 1/5] drv: adding driver registration interface (stub)

The enumerator class, enumerator instance, devio and driver registration functions prototypes (and a draft of their parameters) are defined, the goal being to define the registration framework only. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/dr

Re: [lng-odp] [API-NEXT PATCHv4 1/5] drv: adding driver registration interface (stub)

, Yi > > On 8 December 2016 at 17:20, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> The enumerator class, enumerator instance, devio and driver registration >> functions prototypes (and a draft of their parameters) are >> defined, the g

Re: [lng-odp] [API-NEXT PATCHv4 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

;> >> >> On 8 December 2016 at 17:20, Christophe Milard < >> christophe.mil...@linaro.org> wrote: >> >>> The linux implementation for the enumerator class registration function, >>> enumerator instance registration function, >>>

Re: [lng-odp] [API-NEXT PATCH] linux-generic: ishm: set up len for block correctly

...continued first part got sent for some reason... -should a size be given on import (for a check) On 15 December 2016 at 14:05, Christophe Milard < christophe.mil...@linaro.org> wrote: > I guess Petri is God when it comes to API: > questions are: > - should the size ret

Re: [lng-odp] [API-NEXT PATCH] linux-generic: ishm: set up len for block correctly

I guess Petri is God when it comes to API: questions are: - should the size return as a result of the shm_get_info be the real size or just the requested size? On 15 December 2016 at 13:54, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 12/15/16 10:47, Christophe Mil

Re: [lng-odp] [API-NEXT PATCH] linux-generic: ishm: set up len for block correctly

On 14 December 2016 at 20:33, Maxim Uvarov wrote: > Size might be not passed to that function if is is read > from file. But it has to be set to provide valid number > in odp_shm_info(). > > Signed-off-by: Maxim Uvarov > --- >

Re: [lng-odp] [PATCH 4/6] helper: select os in configure

the problems that follows) Chrstophe On 13 December 2016 at 09:01, Nicolas Morey-Chaisemartin <nmo...@kalray.eu> wrote: > > > Le 12/13/2016 à 08:46 AM, Christophe Milard a écrit : > > On 12 December 2016 at 15:52, Mike Holmes <mike.hol...@linaro.org> > wrote: >

Re: [lng-odp] [PATCH 4/6] helper: select os in configure

On 12 December 2016 at 15:52, Mike Holmes wrote: > Signed-off-by: Mike Holmes > --- > configure.ac | 14 ++ > helper/Makefile.am | 2 +- > 2 files changed, 15 insertions(+), 1 deletion(-) > > diff --git a/configure.ac

Re: [lng-odp] [PATCH 6/6] helper: take default os from odp platform

On 12 December 2016 at 15:52, Mike Holmes wrote: > Signed-off-by: Mike Holmes > --- > configure.ac | 4 ++-- > platform/linux-generic/m4/configure.m4 | 2 ++ > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff

Re: [lng-odp] [PATCH 2/6] helper: remove unused linux specific apis

they were old test for linux only. process and thread omplementation of odp threads are tested in odpthread.c On 12 December 2016 at 19:51, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 12/12/16 21:08, Christophe Milard wrote: > > Why that question? The files are deleted,

[lng-odp] [RFC API-NEXT PATCHv3] test: adding test for driver loading

the result of a long series of trial and errors with autotools. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- Note: must be applied on top of: [API-NEXT PATCHv4 0/5] driver initialisation framework Since V2: includes a needed change for V4 of "driver initialisation frame

[lng-odp] [RFC API-NEXT PATCHv2] test: adding test for driver loading

the result of a long series of trial and errors with autotools. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- Note: must be applied on top of: [API-NEXT PATCHv4 0/5] driver initialisation framework Since V1: updates for V4 of "driver initialisation framework"

[lng-odp] [API-NEXT PATCHv4 5/5] linux-gen: drv_drivers: loading modules from config file

The shared objects listed in the ODP configuration files are loaded at init time. The odp configuration file list the shared objects to be loaded as shown in the following example: module = { modules = ["enumerator1.so", "driver1.so"]; }; Signed-off-by: Christophe Mi

[lng-odp] [API-NEXT PATCHv4 3/5] linux-gen: init: adding configuration file parsing

The parsing of the odp.conf configuration file is added. The file is searched first in the local directory (.) and then is the $prefix/etc directory. This requires libconfig (sudo apt-get install libconfig-dev) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- DEPEND

[lng-odp] [API-NEXT PATCHv4 4/5] test: preventing odp.conf loading for tests

(not using make check) may be affected by configuration files. Setting ODP_SYSCONFIG_FILE to an appropriate value (possibly "none") may be required. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/Makefile.inc | 4 +++- 1 file changed, 3 insertions(+),

[lng-odp] [API-NEXT PATCHv4 0/5] driver initialisation framework

parate patch as many questions remains. -All libdl tests removed: libdl is assumed to always be on linux (Maxim) Since V1: -enum names prefixed by ODPDRV (Yi) -better commit message for last patch (Christophe) -typo fix (Christophe) Christophe Milard (5): drv: adding driver registration inte

[lng-odp] [API-NEXT PATCHv4 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

The linux implementation for the enumerator class registration function, enumerator instance registration function, devio and driver registration functions (stub) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + pl

[lng-odp] [API-NEXT PATCHv4 1/5] drv: adding driver registration interface (stub)

The enumerator class, enumerator instance, devio and driver registration functions prototypes (and a draft of their parameters) are defined, the goal being to define the registration framework only. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/dr

Re: [lng-odp] [API-NEXT PATCHv3 0/5] driver initialisation framework

Ooops Clang. forgot this one will go through this. thanks... On 7 December 2016 at 13:21, Bill Fischofer <bill.fischo...@linaro.org> wrote: > On Wed, Dec 7, 2016 at 1:19 AM, Christophe Milard > <christophe.mil...@linaro.org> wrote: > > > > On 7 December

Re: [lng-odp] [API-NEXT PATCHv3 0/5] driver initialisation framework

n extra dependency is added? Thanks for your attention, Christophe > On Tue, Dec 6, 2016 at 8:23 AM, Christophe Milard > <christophe.mil...@linaro.org> wrote: > > Since V2: > > -function odp_load_driver removed. replaced by config file. (Petri, FF) > > -configur

Re: [lng-odp] [PATCH] configure: add option to enable/disable helper debug prints

On 6 December 2016 at 15:28, Nicolas Morey-Chaisemartin wrote: > As a side effect, debug print are turned off by default > > Signed-off-by: Nicolas Morey-Chaisemartin > --- > configure.ac | 14 ++ > helper/Makefile.am | 3 ++- > 2 files

[lng-odp] [API-NEXT PATCHv2] linux-gen: _ishm: unlinking files asap for cleaner termination

_ODP_ISHM_EXPORT), whose names shall be seen in the file system. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- Since V1: -flag size reduced to 1 bit (maxim) -typo fix (Maxim) platform/linux-generic/_ishm.c | 14 -- 1 file changed, 12 insertions

[lng-odp] [RFC API-NEXT PATCH] test: adding test for driver loading

the result of a long series of trial and errors with autotools. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- Note: must be applied on top of: [API-NEXT PATCHv3 0/5] driver initialisation framework Is this good enough to go in the test structure? I assumed this is

[lng-odp] [API-NEXT PATCHv3 5/5] linux-gen: drv_drivers: loading modules from config file

The shared objects listed in the ODP configuration files are loaded at init time. The odp configuration file list the shared objects to be loaded as shown in the following example: module = { modules = ["enumerator1.so", "driver1.so"]; }; Signed-off-by: Christophe Mi

[lng-odp] [API-NEXT PATCHv3 4/5] test: preventing odp.conf loading for tests

(not using make check) may be affected by configuration files. Setting ODP_SYSCONFIG_FILE to an appropriate value (possibly "none") may be required. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/Makefile.inc | 4 +++- 1 file changed, 3 insertions(+),

[lng-odp] [API-NEXT PATCHv3 0/5] driver initialisation framework

odp_driver_register() is of course part of the driver interface (south). Christophe Milard (5): drv: adding driver registration interface (stub) linux-gen: adding enum, devio and driver registration interface (stub) linux-gen: init: adding configuration file parsing test: preventing odp.co

[lng-odp] [API-NEXT PATCHv3 1/5] drv: adding driver registration interface (stub)

The enumerator class, enumerator instance, devio and driver registration functions prototypes (and a draft of their parameters) are defined, the goal being to define the registration framework only. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/dr

[lng-odp] [API-NEXT PATCHv3 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

The linux implementation for the enumerator class registration function, enumerator instance registration function, devio and driver registration functions (stub) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + pl

[lng-odp] [API-NEXT PATCH] linux-gen: shared_memory: remove flag forcing mlock()

The _ishm flag _ODP_ISHM_LOCK is no longer set when doing shm_reserve(), hence enabling non-root user to exceed the 64MB mlock memory limit (ulimit). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/odp_shared_memory.c | 3 --- 1 file chan

Re: [lng-odp] API-NEXT: mlock for > 64 kb area

If you want to re-open that question, schedule it ay tomorrow (tuesday) arch call, as I won't be bale to attend neither the arch call nor the sync call today due to conflicting compulsory ericsson meetings. Thanks, Christophe On 4 December 2016 at 12:39, Christophe Milard <christophe.

Re: [lng-odp] API-NEXT: mlock for > 64 kb area

The original patch set had a flag for mlock . Petri requested mlock for any allocation . I mentioned the ulimit issue at the arch call if I remember right. I still have a flag on the south API. On 3 Dec 2016 20:24, "Maxim Uvarov" wrote: > Hello Christophe, > > Found

[lng-odp] [API-NEXT PATCH] linux-gen: _ishm: unlinking files asap for cleaner termination

_ODP_ISHM_EXPORT), whose names shall be seen in the file system. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/platform/linux-generic/_ishm.c b/platform/linux-g

Re: [lng-odp] [API-NEXT PATCH] test: drvshm: removing invalid test

Savolainen <petri.savolai...@nokia.com> > > >> -----Original Message- >> From: Christophe Milard [mailto:christophe.mil...@linaro.org] >> Sent: Tuesday, November 29, 2016 4:46 PM >> To: maxim.uva...@linaro.com; Savolainen, Petri (Nokia - FI/Espoo) >> <petri

Re: [lng-odp] [API-NEXT PATCH] linux-gen: _fdserver: request sigterm if parent dies

ping This should be straightforward to review and would help (partly) avoiding hanging processes when killing ODP (e.g. ctrl-C) Thanks On 2016-11-25 15:39, Christophe Milard wrote: > _fdserver now request SIGTERM if parent process (ODP instantiation > process) dies, hence avoiding it to

Re: [lng-odp] Driver registration: object version control

On 29 November 2016 at 20:00, Francois Ozog <francois.o...@linaro.org> wrote: > > > On 29 November 2016 at 19:03, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> >> >> On 29 November 2016 at 18:13, Francois Ozog <francois.o...@lin

Re: [lng-odp] Driver registration: object version control

On 29 November 2016 at 18:13, Francois Ozog <francois.o...@linaro.org> wrote: > > > On 29 November 2016 at 16:44, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> >> >> On 29 November 2016 at 13:22, Francois Ozog <francois.o...@l

Re: [lng-odp] Driver registration: object version control

On 29 November 2016 at 13:22, Francois Ozog <francois.o...@linaro.org> wrote: > Hi, > > comments inline > > On 29 November 2016 at 10:58, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> Hi, >> >> In our last meeting, yesterday, w

[lng-odp] [API-NEXT PATCH] test: drvshm: removing invalid test

The test trying to free a unallocated address is now erroneous since ODP is using ishm too: unknown drvshm addresses may well be valid ODP addresses, in which case the test frees someone else memory! The test is simply suppressed. Signed-off-by: Christophe Milard <christophe.mil...@linaro.

[lng-odp] Driver registration: object version control

Hi, In our last meeting, yesterday, we agreed on the following objects: 1) enumerator class 2) enumerator 3) enumerated_device 4) devio 5) driver and to come: pktio_interface... At registration time, the driver have to tell: - The enumerator class it expects devices from (string E) - The devio

[lng-odp] [PATCH 2/2] performance: odp_pktio_perf: adding missing term functions

Both term function were missing for the ODP main process. Added here. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/performance/odp_pktio_perf.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/common_plat/performance/odp_pktio_perf.c

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

is the best, If you have better approach, please suggest (better patch welcome!) Christophe Milard (2): linux-gen: packet_io: handling term_global while in CLOSE_PENDING performance: odp_pktio_perf: adding missing term functions platform/linux-generic/odp_packet_io.c| 6 -- test

Re: [lng-odp] [API-NEXT PATCH 0/3] _ishm: multiple blocks using identical names

ping On 9 November 2016 at 14:28, Bill Fischofer <bill.fischo...@linaro.org> wrote: > For this series: > > Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org> > > On Tue, Nov 8, 2016 at 3:49 AM, Christophe Milard > <christophe.mil...@linaro.org> wrote

[lng-odp] [API-NEXT PATCH] linux-gen: _fdserver: request sigterm if parent dies

_fdserver now request SIGTERM if parent process (ODP instantiation process) dies, hence avoiding it to become orphan and reattached to the init process. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_fdserver.c | 6 ++ 1 file changed, 6 inse

Re: [lng-odp] [PATCH] linux-gen: _fdserver: request sigterm if parent dies

Please ignore: this is meant for API-NEXt!: sending a new one! On 25 November 2016 at 15:01, Christophe Milard <christophe.mil...@linaro.org> wrote: > _fdserver now request SIGTERM if parent process (ODP instantiation > process) dies, hence avoiding it to become orphan an

[lng-odp] [PATCH] linux-gen: _fdserver: request sigterm if parent dies

_fdserver now request SIGTERM if parent process (ODP instantiation process) dies, hence avoiding it to become orphan and reattached to the init process. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_fdserver.c | 6 ++ 1 file changed, 6 inse

Re: [lng-odp] [API-NEXT PATCHv9 00/15] using _ishm as north API mem allocator

uva...@linaro.org> wrote: > I forgot to say that I see back ground process only after make check. And > run single process works. > > On 25 November 2016 at 12:23, Christophe Milard > <christophe.mil...@linaro.org> wrote: >> >> On 24 November 2016 at 21:07, Maxim Uv

Re: [lng-odp] [API-NEXT PATCHv9 00/15] using _ishm as north API mem allocator

andle_request():drop {ctx=1, key=11}->fd=16 _fdserver.c:321:_odp_fdserver_deregister_fd():FD client deregister: pid=7625 key=8 _fdserver.c:513:handle_request():drop {ctx=1, key=8}->fd=13 erachmi@erachmi-ericsson:~/linaro/ODP/odp$ > > On 11/24/16 19:22, Christophe Milard wrote: >>

[lng-odp] [API-NEXT PATCHv9 15/15] doc: updating docs for the shm interface extension

Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- doc/users-guide/users-guide.adoc | 68 ++-- 1 file changed, 66 insertions(+), 2 deletions(-) diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc index 6

[lng-odp] [API-NEXT PATCHv9 14/15] linux_gen: _ishm: decreasing the number of error messages when no huge pages

Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/platform/linux-generic/_ishm.c b/platform/linux-generic/_ishm.c index b00f8f1..7d9c213

[lng-odp] [API-NEXT PATCHv9 13/15] linux-gen: _ishm: cleaning remaining block at odp_term_global

Remaining (forgotten, not freed) blocks are gathered and related files cleaned when odp_term_global() is called. An error message is also issued so the application writters get to know about these blocks Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-g

[lng-odp] [API-NEXT PATCHv9 12/15] test: linux-gen: api: shmem: test sharing memory between ODP instances

e test procedure. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/linux-generic/validation/api/shmem/.gitignore | 3 +- .../linux-generic/validation/api/shmem/Makefile.am | 22 ++-- .../validation/api/shmem/shmem_linux.c | 140 +++-- .

[lng-odp] [API-NEXT PATCHv9 11/15] linux-gen: shm: add flag and function to share memory between ODP instances

Implemented by calling the related functions from _ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/odp_shared_memory.c | 31 +++--- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/platform/linux-g

[lng-odp] [API-NEXT PATCHv9 09/15] test: api: shmem: new proper tests for shm API

ing trying to provoque race conditions. The new shmem tests do not assume pthreads any longer and are runnable in process mode. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/validation/api/shmem/shmem.c | 687 -- test/common_plat/vali

[lng-odp] [API-NEXT PATCHv9 10/15] linux-gen: _ishm: adding function to map memory from other ODP

-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 160 +++- platform/linux-generic/include/_ishm_internal.h | 5 + 2 files changed, 136 insertions(+), 29 deletions(-) diff --git a/platform/linux-generic/_ishm.c b/platform

[lng-odp] [API-NEXT PATCHv9 07/15] api: shm: add flags to shm_reserve and function to find external mem

at odp_shm_reserve() time the memory block becomes visible to other ODP instances. The function odp_shm_import() is added: this function enables to reserve block of memories exported by other ODP instances (using the ODP_SHM_EXPORT flag). Signed-off-by: Christophe Milard <christophe.mil...@linaro.

[lng-odp] [API-NEXT PATCHv9 08/15] linux-gen: shm: new ODP_SHM_SINGLE_VA flag implementation

This flag guarentess the unicity the the block address on all ODP threads. The patch just exposes the _ODP_ISHM_SINGLE_VA flag of the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/odp_shared_memory.c | 1 +

[lng-odp] [API-NEXT PATCHv9 06/15] linux-gen: push internal flag definition

long as we have the current IPC, removing the file would break compilation. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_shm_internal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/platform/linux

[lng-odp] [API-NEXT PATCHv9 05/15] linux-gen: use ishm as north API mem allocator

-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h | 5 - platform/linux-generic/odp_init.c | 19 - platform/linux-generic/odp_shared_memory.c | 406 ++--- .../validation/api/shmem/shmem_l

[lng-odp] [API-NEXT PATCHv9 04/15] linux-gen: _ishm: allow memory alloc/free at global init/term

for the main process, hence making the ishm_reserve() and ishm_free() functions available at init_global/term_global time. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 189 + 1 file change

[lng-odp] [API-NEXT PATCHv9 03/15] linux-gen: _ishm: create description file for external memory sharing

ying block attributes. This file is meant to be used by processes external to ODP willing to share this memory. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 74 ++--- platform/linux-generic/include/_ishm_inte

[lng-odp] [API-NEXT PATCHv9 02/15] linux-gen: init: removing possible obsolete ODP files at startup

as the PID is unique and therefore, there cannot be another ODP instance with the same PID. This patch does this cleanup at startup. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/odp_init.c | 60 +++ 1 file change

<    1   2   3   4   5   6   7   8   9   10   >