Re: [lng-odp] [PATCH] example: introducing multi-threaded traffic management case

2016-12-30 Thread Barry Spinney
FYI. I am still looking at this patch, but it is fairly large and is taking quite a few hours. I am hoping to be able to spend a bunch more hours on it tomorrow and/or possibly Sunday/Monday. I have been out of the office a fair bit, plus we are dealing with a serious skiing accident that inv

Re: [lng-odp] [PATCH v3 2/2] test: pktio_ordered: add test script

2016-12-30 Thread Bill Fischofer
Trying to run this I get: bill@Ubuntu15:~/linaro/review/test/common_plat/performance$ ./odp_pktio_ordered_run.sh using PCAP_IN = ./udp64.pcap _ishm.c:859:_odp_ishm_reserve():No huge pages, fall back to normal pages. check: /proc/sys/vm/nr_hugepages. PKTIO: initialized loop interface. PKTIO: init

[lng-odp] [API-NEXT PATCHv8 5/5] linux-gen: modules: adding initial file to load modules

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

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

2016-12-30 Thread Christophe Milard
The parsing of the odp.conf configuration file is added. The file is searched first in filename specified in the environment variable $ODP_SYSCONFIG_FILE (where the special string "none" prevent any file loading) The file is then searched in the user home directory (~) and then is the $prefix/etc d

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

2016-12-30 Thread Christophe Milard
The tests should not be affected by any system or user ODP configuration file. The ODP_SYSCONFIG_FILE environment variables is therefore set to "none" in TESTS_ENVIRONMENT. Tests which need specific a configuration file will have to overwrite this setting. Note that tests ran manually (not using ma

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

2016-12-30 Thread Christophe Milard
The linux implementation for the enumerator class registration function, enumerator instance registration function, devio and driver registration functions (stub) Signed-off-by: Christophe Milard --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

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

2016-12-30 Thread Christophe Milard
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 --- include/odp/drv/spec/driver.h | 293

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

2016-12-30 Thread Christophe Milard
Since V7: -changed include order in drv_driver.c (Anders) -correction of package name for Fedora install in DEPENDENCIES (Anders) -user config file changed from ./odp.conf to ~/.odp.conf: then the user config file and the usage of the environment variable are clearly distinct (Anders) -fi

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

2016-12-30 Thread Savolainen, Petri (Nokia - FI/Espoo)
This patch Reviewed-by: Petri Savolainen > -Original Message- > From: Christophe Milard [mailto:christophe.mil...@linaro.org] > Sent: Friday, December 30, 2016 1:59 PM > To: francois.o...@linaro.org; forrest@linaro.org; > mike.hol...@linaro.org; yi...@linaro.org; bill.fischo...@lina

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

2016-12-30 Thread Christophe Milard
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 --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 86 ++ .../common_plat/validation/drv/drvshmem/drvshmem.h | 1 + 2 files

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

2016-12-30 Thread Christophe Milard
Stress tests for the random size allocator (buddy allocator in linux-generic) are added here. Signed-off-by: Christophe Milard --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 177 + .../common_plat/validation/drv/drvshmem/drvshmem.h | 1 + 2 files changed, 178 inse

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

2016-12-30 Thread Christophe Milard
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 --- .../common_plat/validation/drv/drvshmem/drvshmem.c | 92 ++ .../common_plat/validation/drv/drvshmem/drvshmem.h | 1 + 2 files chan

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

2016-12-30 Thread Christophe Milard
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 --- platform/linux-generic/drv_shm.c | 44 1 file changed, 44 insertions(+

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

2016-12-30 Thread Christophe Milard
_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 --- platform/linux-generic/Makefile.am | 2 + platform/linux-generic/_ishm.c

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

2016-12-30 Thread Christophe Milard
Adding functions to create and destroy memory pools (from which memory can be allocated and freed) are added. These functions enable the usage of small memory amount (compared to drvshm_reserve() whose granularity is the page size). The usage of this pool guatantees that allocated memory is sharabl

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

2016-12-30 Thread Christophe Milard
Since V2: -comment and doxygen fixes (Petri) Since V1: -Common south pool (for all drivers) suppressed. (Petri) -function to create and destroy the pools exposed to the interface, so that each user create his own pools (Petri) -support for fixed size allocator added (Petri, Bill) -Creation

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

2016-12-30 Thread Anders Roxell
On 2016-12-30 10:09, Christophe Milard wrote: > 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"];

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

2016-12-30 Thread Maxim Uvarov
On 30 December 2016 at 12:01, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia-bell-labs.com> wrote: > > > +=== Known Issues > > > + > > > + https://bugs.linaro.org/show_bug.cgi?id=2765[Bug 2765] > > > +The fixed storage overhead for supporting the new pool structures is > > large

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

2016-12-30 Thread Anders Roxell
On 2016-12-30 10:09, Christophe Milard wrote: > 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 Mil

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

2016-12-30 Thread Savolainen, Petri (Nokia - FI/Espoo)
> > > >> + uint64_t pool_size; > >> + > >> + /** Minimum alloc size application will request from pool > (bytes)*/ > >> + uint64_t min_alloc; > > > > Since this is a driver interface: ... size *driver* will request ... > > It is not limited to drivers: other driver elements such as > e

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

2016-12-30 Thread Christophe Milard
On 29 December 2016 at 10:19, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >> Christophe Milard >> Sent: Thursday, December 29, 2016 9:57 AM >> To: mike.hol...@linaro.org; bill.fischo...@linaro

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

2016-12-30 Thread Savolainen, Petri (Nokia - FI/Espoo)
> > +=== Known Issues > > + > > + https://bugs.linaro.org/show_bug.cgi?id=2765[Bug 2765] > > +The fixed storage overhead for supporting the new pool structures is > large > > +(over a quarter of a gigabyte). This may have an impact on smaller > systems. > > + > > > I think we need merge this

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

2016-12-30 Thread Christophe Milard
Hopefully, I understood what you meant :-) Sent a v7 with this updtade as well... /Christophe. On 29 December 2016 at 14:39, Maxim Uvarov wrote: > On 12/29/16 16:12, Christophe Milard wrote: >> I have now updated the travis file (second comment) and had the >> DEPENDENCIES file updated since the

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

2016-12-30 Thread Christophe Milard
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 Milard --- configure.ac

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

2016-12-30 Thread Christophe Milard
The tests should not be affected by any system or user ODP configuration file. The ODP_SYSCONFIG_FILE environment variables is therefore set to "none" in TESTS_ENVIRONMENT. Tests which needs specific a configuration file will have to overwrite this setting. Note that tests ran manually (not using m

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

2016-12-30 Thread Christophe Milard
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 --- .travis.yml |

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

2016-12-30 Thread Christophe Milard
The linux implementation for the enumerator class registration function, enumerator instance registration function, devio and driver registration functions (stub) Signed-off-by: Christophe Milard --- include/odp_drv.h | 1 + platform/linux-generic/Makefile.am

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

2016-12-30 Thread Christophe Milard
Since V6: -more inforamtion added in the DEPENDENCIES file for libconf installation (Maxim) Since V5: -name and comment changes as suggested by Maxim in https://lists.linaro.org/pipermail/lng-odp/2016-December/027400.html -update .travis.yml Since V4: -typo fix (Thanks Yi!) -rebased. Si

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

2016-12-30 Thread Christophe Milard
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 --- include/odp/drv/spec/driver.h | 293