Re: [lng-odp] [API-NEXT PATCH] linux-generic: _fdserver: allocating data table dynamicaly

2016-09-13 Thread Christophe Milard
of course... :-)... It really should be freed at exit only, But I agree, that should be explicit. will send a v2! On 13 September 2016 at 13:25, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Tue, Sep 13, 2016 at 4:47 AM, Christophe Milard < > christophe.mil

[lng-odp] [API-NEXT PATCH] linux-generic: _fdserver: fixing comment typo

2016-09-13 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_fdserver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/platform/linux-generic/_fdserver.c b/platform/linux-generic/_fdserver.c index bf36eb2..97661d0 100644 --- a/pl

[lng-odp] [API-NEXT PATCH] linux-generic: _fdserver: allocating data table dynamicaly

2016-09-13 Thread Christophe Milard
The table containing the saved file-descriptors<->{context, key} couples is now dynamically malloc'd in the fd server process, hence avoiding the memory waste which happened in other process when the table was staticaly reserved in all processes. Signed-off-by: Christophe Milard <chris

[lng-odp] [API-NEXT PATCHv3 13/13] test: validation: drv: shmem: stress tests

2016-08-20 Thread Christophe Milard
Stress tests, randomly allocating memory are added: the test is based on a group of odp threads allocating, mapping and freeing each-other memory. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c

[lng-odp] [API-NEXT PATCHv3 12/13] test: validation: drv: shmem: testing SINGLE_VA flag

2016-08-20 Thread Christophe Milard
Basic test for different drvshm_reserve() time in conjunction with the usage of SINGLE_VA flag is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 221 + .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCHv3 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-20 Thread Christophe Milard
processes and regardless of fork time), when the required _ODP_ISHM_SINGLE_VA flag is used. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am |4 + platform/linux-generic/_ishm.c

[lng-odp] [API-NEXT PATCHv3 11/13] test: validation: drv: shmem: basic tests

2016-08-20 Thread Christophe Milard
reservation occurs both before and after odpthread creation here). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 3 +- test/common_plat/validation/drv/Makefile.am| 3 +- .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCHv3 10/13] linux-gen: drv: shm

2016-08-20 Thread Christophe Milard
The shared memory DRV interface is created, mostly as a wrapper around the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

[lng-odp] [API-NEXT PATCHv3 09/13] linux-gen: ishm: adding debug function

2016-08-20 Thread Christophe Milard
A debug function, printing the internal mem alloc status is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 125 platform/linux-generic/include/_ishm_internal.h | 1 + 2 files change

[lng-odp] [API-NEXT PATCHv3 06/13] linux-gen: fdserver: new fdserver added

2016-08-20 Thread Christophe Milard
-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_fdserver.c | 655 + .../linux-generic/include/_fdserver_internal.h | 38 ++ platform/linux-generic/i

[lng-odp] [API-NEXT PATCHv3 07/13] linux-generic: system_info: adding huge page dir

2016-08-20 Thread Christophe Milard
The Huge page information is separated and a function to get the huge page mount directory is added. This function is called at init so the information is available later on. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h

[lng-odp] [API-NEXT PATCHv3 05/13] drv: adding shared memory

2016-08-20 Thread Christophe Milard
The public definition of the shared memory allocator for the driver interface is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 231 + platform/Makefile.inc | 1 + 2 files change

[lng-odp] [API-NEXT PATCHv3 04/13] linux-generic: drv: strong typing

2016-08-20 Thread Christophe Milard
Mirrored from its couterpart on the api interface, the drv interface gets its strong typing file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 +- .../include/odp/drv/plat/strong_types.h

[lng-odp] [API-NEXT PATCHv3 03/13] linux-gen: drv: adding barrier

2016-08-20 Thread Christophe Milard
Based on API interface files. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am | 3 ++ platform/linux-generic/drv_barrier.c

[lng-odp] [API-NEXT PATCHv3 02/13] drv: adding barrier.h

2016-08-20 Thread Christophe Milard
Based on API interface file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/barrier.h | 66 ++ platform/Makefile.inc | 1 + 2 files changed, 67 insertions(+) create mode 100644 include/odp/dr

[lng-odp] [API-NEXT PATCHv3 01/13] linux-gen: cosmetic changes on barrier

2016-08-20 Thread Christophe Milard
To please check-patch before the copy to the drv interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/api/spec/barrier.h | 2 -- platform/linux-generic/include/odp/api/plat/barrier_types.h | 1 - platform/linux-g

[lng-odp] [API-NEXT PATCHv3 00/13] drv shared memory allocator

2016-08-20 Thread Christophe Milard
mode (using the usual --odph_proc flag) Thanks for reading so far... and happy review! Christophe *** BLURB HERE *** Christophe Milard (13): linux-gen: cosmetic changes on barrier drv: adding barrier.h linux-gen: drv: adding barrier linux-generic: drv: strong typing drv: adding sh

Re: [lng-odp] [API-NEXT PATCH 00/13] drv shared memory allocated

2016-08-19 Thread Christophe Milard
inux-generic' > Makefile:736: recipe for target 'check-recursive' failed > make[3]: *** [check-recursive] Error 1 > make[3]: Leaving directory '/home/bill/linaro/check-odp/ > build/odp-apply/opendataplane-1.10.1.0.git151.ga52267b/_ > build/sub/test/linux-generic' > Makefile:429: recipe

[lng-odp] [API-NEXT PATCHv3] drv: byteorder: added bitfield order

2016-08-19 Thread Christophe Milard
mostly to keep the symmetry with the API side Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/byteorder.h | 3 +++ platform/linux-generic/include/odp/drv/plat/byteorder_types.h | 10 +++--- 2 files changed, 10 inse

Re: [lng-odp] [API-NEXT PATCHv2] drv: byteorder: added bitfield order

2016-08-19 Thread Christophe Milard
God! I am just blind... sorry. sending a V3 On 19 August 2016 at 22:47, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Fri, Aug 19, 2016 at 1:17 PM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> mostly to keep the symmetry w

Re: [lng-odp] [API-NEXT PATCH 00/13] drv shared memory allocated

2016-08-19 Thread Christophe Milard
gt; Core was generated by `./drvshmem_main'. > Program terminated with signal SIGSEGV, Segmentation fault. > #0 0x0804c10b in run_test_stress (arg=0xbf8662b0) at drvshmem.c:683 > 683 CU_ASSERT(address[i] == (data & 0xFF)); > [Current thread is 1 (Thread 0x96959b40 (LWP 15773))] > (gdb)

[lng-odp] [API-NEXT PATCHv2] drv: byteorder: added bitfield order

2016-08-19 Thread Christophe Milard
mostly to keep the symmetry with the API side Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/byteorder.h | 3 +++ platform/linux-generic/include/odp/drv/plat/byteorder_types.h | 10 +++--- 2 files changed, 10 inse

[lng-odp] [API-NEXT PATCHv2 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-19 Thread Christophe Milard
processes and regardless of fork time), when the required _ODP_ISHM_SINGLE_VA flag is used. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am |4 + platform/linux-generic/_ishm.c

[lng-odp] [API-NEXT PATCHv2 13/13] test: validation: drv: shmem: stress tests

2016-08-19 Thread Christophe Milard
Stress tests, randomly allocating memory are added: the test is based on a group of odp threads allocating, mapping and freeing each-other memory. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c

[lng-odp] [API-NEXT PATCHv2 11/13] test: validation: drv: shmem: basic tests

2016-08-19 Thread Christophe Milard
reservation occurs both before and after odpthread creation here). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 3 +- test/common_plat/validation/drv/Makefile.am| 3 +- .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCHv2 12/13] test: validation: drv: shmem: testing SINGLE_VA flag

2016-08-19 Thread Christophe Milard
Basic test for different drvshm_reserve() time in conjunction with the usage of SINGLE_VA flag is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 221 + .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCHv2 10/13] linux-gen: drv: shm

2016-08-19 Thread Christophe Milard
The shared memory DRV interface is created, mostly as a wrapper around the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

[lng-odp] [API-NEXT PATCHv2 06/13] linux-gen: fdserver: new fdserver added

2016-08-19 Thread Christophe Milard
-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_fdserver.c | 655 + .../linux-generic/include/_fdserver_internal.h | 38 ++ platform/linux-generic/i

[lng-odp] [API-NEXT PATCHv2 07/13] linux-generic: system_info: adding huge page dir

2016-08-19 Thread Christophe Milard
The Huge page information is separated and a function to get the huge page mount directory is added. This function is called at init so the information is available later on. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h

[lng-odp] [API-NEXT PATCHv2 09/13] linux-gen: ishm: adding debug function

2016-08-19 Thread Christophe Milard
A debug function, printing the internal mem alloc status is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 125 platform/linux-generic/include/_ishm_internal.h | 1 + 2 files change

[lng-odp] [API-NEXT PATCHv2 05/13] drv: adding shared memory

2016-08-19 Thread Christophe Milard
The public definition of the shared memory allocator for the driver interface is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 231 + platform/Makefile.inc | 1 + 2 files change

[lng-odp] [API-NEXT PATCHv2 04/13] linux-generic: drv: strong typing

2016-08-19 Thread Christophe Milard
Mirrored from its couterpart on the api interface, the drv interface gets its strong typing file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 +- .../include/odp/drv/plat/strong_types.h

[lng-odp] [API-NEXT PATCHv2 03/13] linux-gen: drv: adding barrier

2016-08-19 Thread Christophe Milard
Based on API interface files. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am | 3 ++ platform/linux-generic/drv_barrier.c

[lng-odp] [API-NEXT PATCHv2 02/13] drv: adding barrier.h

2016-08-19 Thread Christophe Milard
Based on API interface file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/barrier.h | 66 ++ platform/Makefile.inc | 1 + 2 files changed, 67 insertions(+) create mode 100644 include/odp/dr

[lng-odp] [API-NEXT PATCHv2 01/13] linux-gen: cosmetic changes on barrier

2016-08-19 Thread Christophe Milard
To please check-patch before the copy to the drv interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/api/spec/barrier.h | 2 -- platform/linux-generic/include/odp/api/plat/barrier_types.h | 1 - platform/linux-g

[lng-odp] [API-NEXT PATCHv2 00/13] drv shared memory allocator

2016-08-19 Thread Christophe Milard
he definition a a series of tests to test the driver interface shared memory. These tests are runnable in thread or process mode (using the usual --odph_proc flag) Thanks for reading so far... and happy review! Christophe Christophe Milard (13): linux-gen: cosmetic changes on barrier drv: adding b

[lng-odp] [API-NEXT PATCH 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-19 Thread Christophe Milard
processes and regardless of fork time), when the required _ODP_ISHM_SINGLE_VA flag is used. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am |4 + platform/linux-generic/_ishm.c

[lng-odp] [API-NEXT PATCH 10/13] linux-gen: drv: shm

2016-08-19 Thread Christophe Milard
The shared memory DRV interface is created, mostly as a wrapper around the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

[lng-odp] [API-NEXT PATCH 03/13] linux-gen: drv: adding barrier

2016-08-19 Thread Christophe Milard
Based on API interface files. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am | 3 ++ platform/linux-generic/drv_barrier.c

[lng-odp] [API-NEXT PATCH 04/13] linux-generic: drv: strong typing

2016-08-19 Thread Christophe Milard
Mirrored from its couterpart on the api interface, the drv interface gets its strong typing file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 +- .../include/odp/drv/plat/strong_types.h

[lng-odp] [API-NEXT PATCH 02/13] drv: adding barrier.h

2016-08-19 Thread Christophe Milard
Based on API interface file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/barrier.h | 66 ++ platform/Makefile.inc | 1 + 2 files changed, 67 insertions(+) create mode 100644 include/odp/dr

[lng-odp] [API-NEXT PATCH 06/13] linux-gen: fdserver: new fdserver added

2016-08-19 Thread Christophe Milard
-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_fdserver.c | 655 + .../linux-generic/include/_fdserver_internal.h | 38 ++ platform/linux-generic/i

[lng-odp] [API-NEXT PATCH 00/13] drv shared memory allocated

2016-08-19 Thread Christophe Milard
erface shared memory. These tests are runnable in thread or process mode (using the usual --odph_proc flag) Thanks for reading so far... and happy review! Christophe Christophe Milard (13): linux-gen: cosmetic changes on barrier drv: adding barrier.h linux-gen: drv: adding barrier linux-

[lng-odp] [API-NEXT PATCH 09/13] linux-gen: ishm: adding debug function

2016-08-19 Thread Christophe Milard
A debug function, printing the internal mem alloc status is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 125 platform/linux-generic/include/_ishm_internal.h | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 07/13] linux-generic: system_info: adding huge page dir

2016-08-19 Thread Christophe Milard
The Huge page information is separated and a function to get the huge page mount directory is added. This function is called at init so the information is available later on. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h

[lng-odp] [API-NEXT PATCH 05/13] drv: adding shared memory

2016-08-19 Thread Christophe Milard
The public definition of the shared memory allocator for the driver interface is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 231 + platform/Makefile.inc | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 01/13] linux-gen: cosmetic changes on barrier

2016-08-19 Thread Christophe Milard
To please check-patch before the copy to the drv interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/api/spec/barrier.h | 2 -- platform/linux-generic/include/odp/api/plat/barrier_types.h | 1 - platform/linux-g

[lng-odp] [API-NEXT PATCH 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-19 Thread Christophe Milard
processes and regardless of fork time), when the required _ODP_ISHM_SINGLE_VA flag is used. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am |4 + platform/linux-generic/_ishm.c

[lng-odp] [API-NEXT PATCH 13/13] test: validation: drv: shmem: stress tests

2016-08-19 Thread Christophe Milard
Stress tests, randomly allocating memory are added: the test is based on a group of odp threads allocating, mapping and freeing each-other memory. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c

[lng-odp] [API-NEXT PATCH 06/13] linux-gen: fdserver: new fdserver added

2016-08-19 Thread Christophe Milard
-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_fdserver.c | 655 + .../linux-generic/include/_fdserver_internal.h | 38 ++ platform/linux-generic/i

[lng-odp] [API-NEXT PATCH 12/13] test: validation: drv: shmem: testing SINGLE_VA flag

2016-08-19 Thread Christophe Milard
Basic test for different drvshm_reserve() time in conjunction with the usage of SINGLE_VA flag is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 221 + .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCH 11/13] test: validation: drv: shmem: basic tests

2016-08-19 Thread Christophe Milard
reservation occurs both before and after odpthread creation here). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 3 +- test/common_plat/validation/drv/Makefile.am| 3 +- .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCH 10/13] linux-gen: drv: shm

2016-08-19 Thread Christophe Milard
The shared memory DRV interface is created, mostly as a wrapper around the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

[lng-odp] [API-NEXT PATCH 09/13] linux-gen: ishm: adding debug function

2016-08-19 Thread Christophe Milard
A debug function, printing the internal mem alloc status is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 125 platform/linux-generic/include/_ishm_internal.h | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 07/13] linux-generic: system_info: adding huge page dir

2016-08-19 Thread Christophe Milard
The Huge page information is separated and a function to get the huge page mount directory is added. This function is called at init so the information is available later on. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h

[lng-odp] [API-NEXT PATCH 05/13] drv: adding shared memory

2016-08-19 Thread Christophe Milard
The public definition of the shared memory allocator for the driver interface is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 231 + platform/Makefile.inc | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 04/13] linux-generic: drv: strong typing

2016-08-19 Thread Christophe Milard
Mirrored from its couterpart on the api interface, the drv interface gets its strong typing file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 +- .../include/odp/drv/plat/strong_types.h

[lng-odp] [API-NEXT PATCH 03/13] linux-gen: drv: adding barrier

2016-08-19 Thread Christophe Milard
Based on API interface files. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am | 3 ++ platform/linux-generic/drv_barrier.c

[lng-odp] [API-NEXT PATCH 02/13] drv: adding barrier.h

2016-08-19 Thread Christophe Milard
Based on API interface file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/barrier.h | 66 ++ platform/Makefile.inc | 1 + 2 files changed, 67 insertions(+) create mode 100644 include/odp/dr

[lng-odp] [API-NEXT PATCH 00/13] drv shared memory allocated

2016-08-19 Thread Christophe Milard
erface shared memory. These tests are runnable in thread or process mode (using the usual --odph_proc flag) Thanks for reading so far... and happy review! Christophe Christophe Milard (13): linux-gen: cosmetic changes on barrier drv: adding barrier.h linux-gen: drv: adding barrier linux-

[lng-odp] [API-NEXT PATCH 01/13] linux-gen: cosmetic changes on barrier

2016-08-19 Thread Christophe Milard
To please check-patch before the copy to the drv interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/api/spec/barrier.h | 2 -- platform/linux-generic/include/odp/api/plat/barrier_types.h | 1 - platform/linux-g

Re: [lng-odp] [API-NEXT PATCH 13/13] test: validation: drv: shmem: stress tests

2016-08-19 Thread Christophe Milard
Fischofer <bill.fischo...@linaro.org> wrote: > This test appears to require that it be run with sudo. Is that > intentional? If so that should be well documented. > > On Fri, Aug 19, 2016 at 10:48 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >&g

Re: [lng-odp] [API-NEXT PATCH 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-19 Thread Christophe Milard
t; Coverity flagged a number of issues against this module (as well as > _fdserver.c). These should be addressed now rather than later: > > On Fri, Aug 19, 2016 at 10:48 AM, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> A new ODP internal mem

[lng-odp] [API-NEXT PATCH 08/13] linux-gen: ishm: internal shared memory allocator (ishm) added

2016-08-19 Thread Christophe Milard
processes and regardless of fork time), when the required _ODP_ISHM_SINGLE_VA flag is used. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am |4 + platform/linux-generic/_ishm.c

[lng-odp] [API-NEXT PATCH 11/13] test: validation: drv: shmem: basic tests

2016-08-19 Thread Christophe Milard
reservation occurs both before and after odpthread creation here). Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 3 +- test/common_plat/validation/drv/Makefile.am| 3 +- .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCH 02/13] drv: adding barrier.h

2016-08-19 Thread Christophe Milard
Based on API interface file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/barrier.h | 66 ++ platform/Makefile.inc | 1 + 2 files changed, 67 insertions(+) create mode 100644 include/odp/dr

[lng-odp] [API-NEXT PATCH 12/13] test: validation: drv: shmem: testing SINGLE_VA flag

2016-08-19 Thread Christophe Milard
Basic test for different drvshm_reserve() time in conjunction with the usage of SINGLE_VA flag is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 221 + .../common_plat/validation/drv/dr

[lng-odp] [API-NEXT PATCH 13/13] test: validation: drv: shmem: stress tests

2016-08-19 Thread Christophe Milard
Stress tests, randomly allocating memory are added: the test is based on a group of odp threads allocating, mapping and freeing each-other memory. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../common_plat/validation/drv/drvshmem/drvshmem.c

[lng-odp] [API-NEXT PATCH 10/13] linux-gen: drv: shm

2016-08-19 Thread Christophe Milard
The shared memory DRV interface is created, mostly as a wrapper around the internal memory allocator, ishm. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

[lng-odp] [API-NEXT PATCH 01/13] linux-gen: cosmetic changes on barrier

2016-08-19 Thread Christophe Milard
To please check-patch before the copy to the drv interface. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/api/spec/barrier.h | 2 -- platform/linux-generic/include/odp/api/plat/barrier_types.h | 1 - platform/linux-g

[lng-odp] [API-NEXT PATCH 06/13] linux-gen: fdserver: new fdserver added

2016-08-19 Thread Christophe Milard
-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_fdserver.c | 655 + .../linux-generic/include/_fdserver_internal.h | 38 ++ platform/linux-generic/i

[lng-odp] [API-NEXT PATCH 09/13] linux-gen: ishm: adding debug function

2016-08-19 Thread Christophe Milard
A debug function, printing the internal mem alloc status is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/_ishm.c | 125 platform/linux-generic/include/_ishm_internal.h | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 07/13] linux-generic: system_info: adding huge page dir

2016-08-19 Thread Christophe Milard
The Huge page information is separated and a function to get the huge page mount directory is added. This function is called at init so the information is available later on. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp_internal.h

[lng-odp] [API-NEXT PATCH 05/13] drv: adding shared memory

2016-08-19 Thread Christophe Milard
The public definition of the shared memory allocator for the driver interface is added. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/shm.h | 231 + platform/Makefile.inc | 1 + 2 files change

[lng-odp] [API-NEXT PATCH 04/13] linux-generic: drv: strong typing

2016-08-19 Thread Christophe Milard
Mirrored from its couterpart on the api interface, the drv interface gets its strong typing file. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 +- .../include/odp/drv/plat/strong_types.h

[lng-odp] [API-NEXT PATCH 03/13] linux-gen: drv: adding barrier

2016-08-19 Thread Christophe Milard
Based on API interface files. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am | 3 ++ platform/linux-generic/drv_barrier.c

[lng-odp] [API-NEXT PATCH 00/13] drv shared memory allocated

2016-08-19 Thread Christophe Milard
erface shared memory. These tests are runnable in thread or process mode (using the usual --odph_proc flag) Thanks for reading so far... and happy review! Christophe Christophe Milard (13): linux-gen: cosmetic changes on barrier drv: adding barrier.h linux-gen: drv: adding barrier linux-

[lng-odp] [API-NEXT PATCH] drv: byteorder: added bitfield order

2016-08-19 Thread Christophe Milard
mostly to keep the symmetry with the API side Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- include/odp/drv/spec/byteorder.h | 3 +++ platform/linux-generic/include/odp/drv/plat/byteorder_types.h | 6 +- 2 files changed, 8 insertions

[lng-odp] [API-NEXT PATCH] doc: application-api-guide: excluding drv interface

2016-08-19 Thread Christophe Milard
The input files parsed to generate the ODP platform specific API doc included (wrongly) part of the drv interface specification. This led to warning on undefined symbols. Fixed by restricting imput files to API only. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --

Re: [lng-odp] [PATCH v2 1/6] linux-generic: internal odp_cpu_pause()

2016-08-17 Thread Christophe Milard
Isn't this obsolete and replaced by : [PATCH v3 00/12] arch improvements ? Christophe On 16 August 2016 at 15:28, Maxim Uvarov wrote: > ping. Please do one more review. > > Maxim. > > On 08/04/16 22:30, Brian Brooks wrote: > >> Signed-off-by: Brian Brooks

[lng-odp] [API-NEXT PATCHv2] test: drv: atomic renamed to drvatomic

2016-08-17 Thread Christophe Milard
the drv interface is hence renamed drvatomic. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- since v1: Fixed DRV prefix in macro (Maxim) test/common_plat/m4/configure.m4 | 2 +- test/common_plat/validation/drv/Makefile.am| 2 +- test/commo

[lng-odp] [PATCHv3] linux-gen: doc: defining the ODP thread

2016-08-09 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> Signed-off-by: Mike Holmes <mike.hol...@linaro.org> --- doc/platform-api-guide/Doxyfile | 1 + doc/platform-api-guide/Makefile.am | 8 - platform/linux-generic/doc/platform_speci

Re: [lng-odp] [PATCH] doc/implementers add internal prefix recommendation

2016-08-04 Thread Christophe Milard
On 4 August 2016 at 16:08, Mike Holmes wrote: > Promote coding style that reduces the chance of collisions with > applications. > > Signed-off-by: Mike Holmes > --- > doc/implementers-guide/implementers-guide.adoc | 7 +++ > 1 file changed, 7

Re: [lng-odp] [PATCH 1/6] linux-generic: internal odp_cpu_pause()

2016-08-04 Thread Christophe Milard
Hi, I have problems applying these patches: ... anyone has tried? Can you grab them from the list and apply them, Brian? Christophe. git am ~/incoming/PATCH_1-6_linux-generic_internal_odp_cpu_pause_.mbox Applying: linux-generic: internal odp_cpu_pause() fatal: corrupt patch at line 33 Patch

Re: [lng-odp] [PATCH] linux-gen: doc: defining the ODP thread

2016-08-04 Thread Christophe Milard
Doxygen looks like your field :) > > Thank you, > Maxim. > > > > On 08/03/16 22:17, Christophe Milard wrote: >> >> ping >> >> >> On 29 July 2016 at 19:33, Bill Fischofer <bill.fischo...@linaro.org> >> wrote: >>> >>> >>>

[lng-odp] [API-NEXT PATCH] test: drv: atomic renamed to drvatomic

2016-08-04 Thread Christophe Milard
the drv interface is hence renamed drvatomic. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 2 +- test/common_plat/validation/drv/Makefile.am| 2 +- test/common_plat/validation/drv/atomic/.gitignore | 1 -

[lng-odp] [API-NEXT PATCH] linux-gen: drv: byteorder: fixed macro typo

2016-08-04 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp/drv/plat/byteorder_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/include/odp/drv/plat/byteorder_types.h b/platform/linux-generic/inclu

Re: [lng-odp] [PATCH] linux-gen: doc: defining the ODP thread

2016-08-03 Thread Christophe Milard
ping On 29 July 2016 at 19:33, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Thu, Jul 28, 2016 at 6:59 AM, Christophe Milard > <christophe.mil...@linaro.org> wrote: >> >> Defining an odp thread for the linux generic implementation of ODP. &g

Re: [lng-odp] [PATCHv3] test: update in readme file

2016-08-03 Thread Christophe Milard
ping On 22 July 2016 at 14:51, Christophe Milard <christophe.mil...@linaro.org> wrote: > mostly updating paths following new test structure > > Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> > --- > test/README | 9 + > 1 file changed

Re: [lng-odp] [PATCH] linux-gen: doc: defining the ODP thread

2016-07-29 Thread Christophe Milard
need comes. > > On Thu, Jul 28, 2016 at 6:59 AM, Christophe Milard > <christophe.mil...@linaro.org> wrote: >> >> Defining an odp thread for the linux generic implementation of ODP. >> >> Signed-off-by: Christophe Milard <christ

[lng-odp] [API-NEXT PATCHv2 5/5] test: validation: drv: atomic tests

2016-07-29 Thread Christophe Milard
Adding tests for atomic operations performed on the driver (drv) interface. Based on the north API tests. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/validation/drv/atomic/.gitignore | 1 + test/common_plat/validation/drv/atomic/Makefile.am

[lng-odp] [API-NEXT PATCHv2 4/5] test: validation: drv: adding the new drv interface

2016-07-29 Thread Christophe Milard
The new structure for validation tests aiming the drv interface of ODP is added (around a first DRV atomic test empty skeleton) Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/m4/configure.m4 | 6 ++-- test/common_plat/vali

[lng-odp] [API-NEXT PATCHv2 3/5] linux-generic: drv: fixed macro typo

2016-07-29 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/include/odp/drv/byteorder.h | 28 +++--- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/platform/linux-generic/include/odp/drv/byteorder.h b/platform/linux-generic/i

[lng-odp] [API-NEXT PATCHv2 1/5] linux-generic: make drv interface part of the distribution

2016-07-28 Thread Christophe Milard
Make sure that the file odp_drv.h is part if a distribution, as odp_api.h. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- platform/linux-generic/Makefile.am | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/platform/linux-generic/Makefile.am b/pl

[lng-odp] [PATCH] linux-gen: doc: defining the ODP thread

2016-07-28 Thread Christophe Milard
Defining an odp thread for the linux generic implementation of ODP. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- .../include/odp/api/platform_specific.dox | 46 ++ 1 file changed, 46 insertions(+) create mode 100644 platform/linux-g

[lng-odp] [API-NEXT PATCH 5/5] test: validation: drv: atomic tests

2016-07-28 Thread Christophe Milard
Adding tests for atomic operations performed on the driver (drv) interface. Based on the north API tests. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- test/common_plat/validation/drv/atomic/.gitignore | 1 + test/common_plat/validation/drv/atomic/Makefile.am

Re: [lng-odp] [PATCH v3 01/12] linux-generic: internal cache line size

2016-07-26 Thread Christophe Milard
ian Brooks <brian.bro...@linaro.org> The commit msg should say prefixed with "_odp" not just "_". Maybe you can change that when merging, Maxim. I need that first patch in api-next to submit some first driver interface test. Reviewed-by: Christophe Milard <christophe.m

Re: [lng-odp] [PATCH v3 00/12] arch improvements

2016-07-26 Thread Christophe Milard
EFS SPLIT_STRING SSCANF_TO_KSTRTO 0001-arm-odp_cpu_cycles-for-arm.patch has style problems, please review. On 26 July 2016 at 10:13, Christophe Milard <christophe.mil...@linaro.org> wrote: > I am confused about patch 3 which seems to remove arch specific for > arm and patch 6 which reint

Re: [lng-odp] [PATCH v3 00/12] arch improvements

2016-07-26 Thread Christophe Milard
I am confused about patch 3 which seems to remove arch specific for arm and patch 6 which reintroduce it. Probably something I don't understand... can you explain? Is this series working fine on ARM? The API function: uint64_t odp_cpu_hz_current(int id ODP_UNUSED) Is still defined in the arch

Re: [lng-odp] [PATCH v2] linux-generic: fixup cache line size API

2016-07-25 Thread Christophe Milard
Brian: can you join the arch call this afternoon (your morning). would be nice to clarify what we want to do here... Thanks Christophe. On 22 July 2016 at 19:25, Brian Brooks <brian.bro...@linaro.org> wrote: > On 07/22 08:39:38, Christophe Milard wrote: >> On 19 July 2016 at 17:2

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