Re: [lng-odp] [API-NEXT PATCHv3 1/5] linux-generic: sockets: implement pktio statistics counters

2015-11-19 Thread Mike Holmes
On 18 November 2015 at 04:41, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > platform/linux-generic/Makefile.am | 2 + > .../linux-generic/include/odp_packet_io_internal.h | 11 ++ > platform/linux-generic/include/odp_packet_socket.h | 6 + > platform/linux-gener

Re: [lng-odp] [PATCH v2] doc: implementers-guide: convert to ODP standard layout

2015-11-19 Thread Bill Fischofer
On Thu, Nov 19, 2015 at 12:10 PM, Mike Holmes wrote: > Signed-off-by: Mike Holmes > --- > v2 spelling found by Bill > > doc/implementers-guide/implementers-guide.adoc | 267 > + > 1 file changed, 186 insertions(+), 81 deletions(-) > > diff --git a/doc/implementers-guide/

Re: [lng-odp] [API-NEXT PATCH 0/5] netmap pktio multi queue support

2015-11-19 Thread Mike Holmes
mike@mike-desktop:~/git/odp$ git st On branch api-next Your branch is ahead of 'origin/api-next' by 1 commit. (use "git push" to publish your local commits) nothing to commit, working directory clean mike@mike-desktop:~/git/odp$ git am ~/incoming/matias/lng-odp_API-NEXT_PATCH_1-5_linux-generi

Re: [lng-odp] [API-NEXT PATCH v3 6/7] linux-generic: pktio: dummy multi-queue pktio

2015-11-19 Thread Mike Holmes
ls -l ~/incoming/petri total 112 -rw--- 1 mike mike 6191 Nov 19 13:55 lng-odp_API-NEXT_PATCH_v3_1-7_api_pktio_added_pktio_capability_struct.mbox -rw--- 1 mike mike 13307 Nov 19 13:55 lng-odp_API-NEXT_PATCH_v3_2-7_api_pktio_added_multiple_pktio_input_queues.mbox -rw--- 1 mike mike 694

[lng-odp] [Bug 1854] CID 152857: Error handling issues: odp_pktio_term_global

2015-11-19 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1854 Maxim Uvarov changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[lng-odp] [PATCH v2] doc: implementers-guide: convert to ODP standard layout

2015-11-19 Thread Mike Holmes
Signed-off-by: Mike Holmes --- v2 spelling found by Bill doc/implementers-guide/implementers-guide.adoc | 267 + 1 file changed, 186 insertions(+), 81 deletions(-) diff --git a/doc/implementers-guide/implementers-guide.adoc b/doc/implementers-guide/implementers-guide.ad

[lng-odp] [API-NEXT PATCH 4/5] linux-generic: netmap: add multi queue support

2015-11-19 Thread Matias Elo
Added multi queue support to netmap pktio. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_netmap.h | 27 +- platform/linux-generic/pktio/netmap.c | 367 ++--- 2 files changed, 341 insertions(+), 53 deletions(-) diff --git a/platform/linux-g

[lng-odp] [API-NEXT PATCH 5/5] linux-generic: netmap: odp_pktio_recv() from all pktin queues

2015-11-19 Thread Matias Elo
Receive packets from all pktin queues. Improves backward compatibility. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_netmap.h | 1 + platform/linux-generic/pktio/netmap.c | 23 +- 2 files changed, 23 insertions(+), 1 deletion(-) diff

[lng-odp] [API-NEXT PATCH 2/5] linux-generic: netmap: add odp_pktio_start()

2015-11-19 Thread Matias Elo
Added initial version of odp_pktio_start() to netmap. Removed unnecessary global mmap_desc variable. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_netmap.h | 2 + platform/linux-generic/pktio/netmap.c | 59 -- 2 files changed, 35 insert

[lng-odp] [API-NEXT PATCH 3/5] linux-generic: netmap: add odp_pktio_capability()

2015-11-19 Thread Matias Elo
Implemented odp_pktio_capability() function in netmap. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_netmap.h | 2 ++ platform/linux-generic/pktio/netmap.c | 11 ++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/platform/linux-gene

[lng-odp] [API-NEXT PATCH 1/5] linux-generic: pktio: add RSS helper functions

2015-11-19 Thread Matias Elo
Added functions for fetching, configuring, and printing NIC RSS configurations. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_socket.h | 18 +++ platform/linux-generic/pktio/socket.c | 164 + 2 files changed, 182 insertions(+) diff --g

[lng-odp] [API-NEXT PATCH 0/5] netmap pktio multi queue support

2015-11-19 Thread Matias Elo
This patch set adds initial multi queue support to netmap pktio. Applies on top of the '[API-NEXT PATCH v3 0/7] Multi-queue packet io APIs' patch set. Matias Elo (5): linux-generic: pktio: add RSS helper functions linux-generic: netmap: add odp_pktio_start() linux-generic: netmap: add odp_

[lng-odp] [Bug 1615] odp_timer fails in CI with Segmentation fault

2015-11-19 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1615 Mike Holmes changed: What|Removed |Added Assignee|ola.liljed...@linaro.org|ivan.khoronz...@linaro.org -- You are receiving

[lng-odp] [Bug 1894] make distcheck does not catch inability to build stand alone apps

2015-11-19 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1894 Mike Holmes changed: What|Removed |Added CC||mike.hol...@linaro.org Status|UNCONF

[lng-odp] [Bug 1879] Ordered queues occasionally not restoring order properly

2015-11-19 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1879 Mike Holmes changed: What|Removed |Added CC||mike.hol...@linaro.org Status|IN_PRO

[lng-odp] [Bug 1890] socket pktios drop all packets on tx

2015-11-19 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1890 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED CC|

[lng-odp] [API-NEXT PATCH v3 7/7] test: l2fwd: use multi-queue pktio in direct mode

2015-11-19 Thread Petri Savolainen
Modified the application to support for multiple queues per interface. Enabled inequal worker thread counts. Signed-off-by: Petri Savolainen --- test/performance/odp_l2fwd.c | 414 --- 1 file changed, 352 insertions(+), 62 deletions(-) diff --git a/test/p

[lng-odp] [API-NEXT PATCH v3 6/7] linux-generic: pktio: dummy multi-queue pktio

2015-11-19 Thread Petri Savolainen
Implemented multi-queue API with dummy single queue support. Enables testing the API and provides stubbs for actual multi-queue implementations. Not optimized. Signed-off-by: Petri Savolainen --- .../include/odp/plat/packet_io_types.h | 14 +- .../linux-generic/include/odp_packet_io

[lng-odp] [API-NEXT PATCH v3 4/7] api: pktio: added multiple pktio output queues

2015-11-19 Thread Petri Savolainen
Added output queue configuration parameters and functions for setting up multiple output queues. Added also a function to query the number of output queues. Signed-off-by: Petri Savolainen --- include/odp/api/packet_io.h| 71 ++ .../include/odp/plat/pa

[lng-odp] [API-NEXT PATCH v3 5/7] api: pktio: added direct send to pktio output queue

2015-11-19 Thread Petri Savolainen
Added odp_pktio_send_queue for direct packet send to a pktio output queue. Signed-off-by: Petri Savolainen --- include/odp/api/packet_io.h | 39 +++ 1 file changed, 31 insertions(+), 8 deletions(-) diff --git a/include/odp/api/packet_io.h b/include/odp/api/pa

[lng-odp] [API-NEXT PATCH v3 3/7] api: pktio: added direct queue receive

2015-11-19 Thread Petri Savolainen
Added odp_pktio_recv_queue for direct packet receive from pktio input queues. Signed-off-by: Petri Savolainen --- include/odp/api/packet_io.h | 33 - 1 file changed, 28 insertions(+), 5 deletions(-) diff --git a/include/odp/api/packet_io.h b/include/odp/api/packe

[lng-odp] [API-NEXT PATCH v3 1/7] api: pktio: added pktio capability struct

2015-11-19 Thread Petri Savolainen
Added capability structure and a function to query it. Capability limits are used for advanced pktio configuration, like configuration of multiple input/output queues. Signed-off-by: Petri Savolainen --- include/odp/api/packet_io.h | 23 +++ 1 file changed, 23 insertions(+)

[lng-odp] [API-NEXT PATCH v3 2/7] api: pktio: added multiple pktio input queues

2015-11-19 Thread Petri Savolainen
Added input queue configuration parameters and functions to setup multiple input queue and hashing. Added also functions to query the number of queues and queue handles. Direct receive does use new odp_pktin_queue_t handle type. Signed-off-by: Petri Savolainen --- include/odp/api/packet_io.h

[lng-odp] [API-NEXT PATCH v3 0/7] Multi-queue packet io APIs

2015-11-19 Thread Petri Savolainen
This patch set adds APIs for multi-queue packet IO. It does not remove or modify existing API calls, so that multi-queue can be verified first with couple of apps and implementations. Single queue APIs (e.g. odp_pktio_inq_setdef()) and potentially plain _recv() and _send() should be removed once

Re: [lng-odp] [PATCH 3/3 v3] linux-generic: cpumask: warn that CPU0 is used by control and worker thread

2015-11-19 Thread Ivan Khoronzhuk
sent v3 with proposed corrections. On 18.11.15 20:34, Ivan Khoronzhuk wrote: On 18.11.15 20:25, Ola Liljedahl wrote: On 18 November 2015 at 18:09, Ivan Khoronzhuk wrote: 3patch On 10.11.15 17:36, Ivan Khoronzhuk wrote: By default all control threads on CPU0, and odp_cpumask_default_cont

Re: [lng-odp] [PATCH 1/1] linux-generic: config: increase ODP_CONFIG_SCHED_GRPS to 256

2015-11-19 Thread Bill Fischofer
That's a reasonable use case. The main issue is that the scheduler needs a rewrite to properly handle scheduler groups in cases like this. Basically, the scheduler should not be scanning queues that it can know aren't eligible for this core, so the organization of the scheduling queues themselves

[lng-odp] [PATCH v3 3/3] linux-generic: cpumask: warn that CPU0 is used by control and worker thread

2015-11-19 Thread Ivan Khoronzhuk
By default, all control threads execute on CPU 0 and odp_cpumask_default_control() returns this CPU. For reasons of performance, control and worker CPU's shouldn't overlap but for some scenarios it can be desirable to utilize all CPU's for worker threads. Thus we leave the decision of CPU allocatio

[lng-odp] [PATCH v6 2/3] linux-generic: odp_timer: warn if tick is late

2015-11-19 Thread Ivan Khoronzhuk
If tick is late then application should be warned about this. It means that actual timer resolution is worse than expected. The default timer resolution is set 10ms, that is equal to jiffy on most systems. The default resolution is set bearing in mind that on a CPU runs maximum two threads that ide

[lng-odp] [PATCH v4 1/3] example: timer: warn if timeout less than resolution

2015-11-19 Thread Ivan Khoronzhuk
The period shouldn't be less than timer resolution. If it's less there is no guarantee about timeout accuracy, it can be executed right after setting it or after up to timer resolution timeout or even be set late, in which case application will be stopped with error. So it's better to warn user tha

[lng-odp] [PATCH v3 0/3] add warnings to improve timer usage

2015-11-19 Thread Ivan Khoronzhuk
This series is intended to help user to set timer in frames that OS allows him. The timer resolution shouldn't be better than system allows, minimum timeout shouldn't be more than resolution, and warn application that CPU0, that is used for tick notification handling, can have impact on worker thre

Re: [lng-odp] [PATCH 1/1] linux-generic: config: increase ODP_CONFIG_SCHED_GRPS to 256

2015-11-19 Thread Wallen, Carl (Nokia - FI/Espoo)
Hi, We have a use case where we need to create a queue for each core running, i.e. we need to create as many sched-groups (each with one unique bit set in the thrmask) as we have cores in the system and then create as many queues and tie them to each sched-group. This way we can create queues f

Re: [lng-odp] [PATCH 1/1] linux-generic: config: increase ODP_CONFIG_SCHED_GRPS to 256

2015-11-19 Thread Bill Fischofer
No problem with picking a "good" number, but curious as to what the use case is for this proposed increase. On Thu, Nov 19, 2015 at 5:58 AM, Carl Wallen wrote: > Increase the ODP_CONFIG_SCHED_GRPS define from 16 to 256 to support a > larger number of scheduling groups by default. > > Signed-off-

Re: [lng-odp] [API-NEXT PATCH v2] api: atomic: added atomic_lock_free_u64

2015-11-19 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: EXT Ola Liljedahl [mailto:ola.liljed...@linaro.org] > Sent: Monday, November 16, 2015 6:01 PM > To: Savolainen, Petri (Nokia - FI/Espoo) > Cc: LNG ODP Mailman List > Subject: Re: [lng-odp] [API-NEXT PATCH v2] api: atomic: added > atomic_lock_free_u64 > > On 1

[lng-odp] [PATCH] odp.spec: make docs conditional for fedora only

2015-11-19 Thread Riku Voipio
Centos build fails to grab neccesary packages: 00:01:29.829 Getting requirements for ./pkg/rpm/odp.spec 00:01:29.832 --> automake-1.13.4-3.el7.noarch 00:01:30.035 --> autoconf-2.69-11.el7.noarch 00:01:30.037 --> libtool-2.4.2-20.el7.aarch64 00:01:30.040 --> 1:openssl-devel-1.0.1e-42.el7.9.aarc

[lng-odp] [PATCHv2 1/3] validation: ability to specify test install directory

2015-11-19 Thread Stuart Haslam
Previously if a build had been configured with validation tests enabled (--enable-test-vald) a subsequent "make install" would install the test binaries and scripts into /usr/local/bin without any way of separating them from the rest of the installed artefacts. This patch changes the default behavi

[lng-odp] [PATCHv2 3/3] Makefile.am: run tests during distcheck

2015-11-19 Thread Stuart Haslam
The distcheck target is intended to ensure that a built distribution will actually work. As part of the process it runs "make check" and "make installcheck", but these don't actually do much useful testing without the appropriate flags having been passed during configure. Signed-off-by: Stuart Has

[lng-odp] [PATCHv2 2/3] linux-generic: validation: add run-test script for post install testing

2015-11-19 Thread Stuart Haslam
Add a run-test script that replicates some of the "make check" functionality without depending on a whole load of Makefile gubbins. This makes it easier to perform on-target testing following a cross-build (as we want to do in CI) and gives a simple way to verify that all the required components do

Re: [lng-odp] [API-NEXT PATCHv2 5/5] validation: implement pktio statistics counters

2015-11-19 Thread Ivan Khoronzhuk
On 18.11.15 11:34, Maxim Uvarov wrote: Signed-off-by: Maxim Uvarov --- test/validation/pktio/pktio.c | 134 ++ 1 file changed, 134 insertions(+) diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index 6320b77..9f05392 100644

Re: [lng-odp] [PATCH] validation: pktio: test batch receive

2015-11-19 Thread Ivan Khoronzhuk
On 19.11.15 13:54, Ivan Khoronzhuk wrote: On 19.11.15 13:51, Stuart Haslam wrote: On Thu, Nov 19, 2015 at 01:43:23PM +0200, Ivan Khoronzhuk wrote: Sruart, Maxim It will likely have conflicts with time API that gets rid of cycles dependency. It should be rebased on api-next or api-next merg

[lng-odp] [PATCH 1/1] linux-generic: config: increase ODP_CONFIG_SCHED_GRPS to 256

2015-11-19 Thread Carl Wallen
Increase the ODP_CONFIG_SCHED_GRPS define from 16 to 256 to support a larger number of scheduling groups by default. Signed-off-by: Carl Wallen --- platform/linux-generic/include/odp/config.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/include/odp/c

Re: [lng-odp] [PATCH] validation: pktio: test batch receive

2015-11-19 Thread Ivan Khoronzhuk
On 19.11.15 13:51, Stuart Haslam wrote: On Thu, Nov 19, 2015 at 01:43:23PM +0200, Ivan Khoronzhuk wrote: Sruart, Maxim It will likely have conflicts with time API that gets rid of cycles dependency. It should be rebased on api-next or api-next merged in master and then rebased. OK, I'll wa

Re: [lng-odp] [PATCH] validation: pktio: test batch receive

2015-11-19 Thread Stuart Haslam
On Thu, Nov 19, 2015 at 01:43:23PM +0200, Ivan Khoronzhuk wrote: > Sruart, Maxim > > It will likely have conflicts with time API that gets rid of cycles > dependency. > It should be rebased on api-next or api-next merged in master and then > rebased. > OK, I'll wait for it to get a Reviewed-by

Re: [lng-odp] [PATCH] validation: pktio: test batch receive

2015-11-19 Thread Ivan Khoronzhuk
Sruart, Maxim It will likely have conflicts with time API that gets rid of cycles dependency. It should be rebased on api-next or api-next merged in master and then rebased. On 18.11.15 18:59, Stuart Haslam wrote: Modify the tests that currently transmit packets in batches to also receive packe

Re: [lng-odp] [PATCH 4/5] validation: classification: add odp_cls_cos_pool_set() api

2015-11-19 Thread Bala Manoharan
On 19 November 2015 at 16:43, Stuart Haslam wrote: > On Tue, Nov 17, 2015 at 11:46:52AM +0530, Balasubramanian Manoharan wrote: >> Assigns a packet pool to CoS using odp_cls_cos_pool_set() api. >> > > This tests that you can assign a pool to a CoS but since it uses the > same default_pool it doesn

Re: [lng-odp] [PATCH 3/5] linux-generic: classification: implements odp_cls_cos_pool_set() api

2015-11-19 Thread Bala Manoharan
Hi, Pls find comments inline.. On 19 November 2015 at 16:24, Stuart Haslam wrote: > On Tue, Nov 17, 2015 at 11:46:51AM +0530, Balasubramanian Manoharan wrote: >> Adds support for configuring packet pool to a class-of-service. >> linux-generic packet parser is enhanced to parse a packet directly

Re: [lng-odp] [PATCH] validation: remove strict dependency on CUnit 2.1-3

2015-11-19 Thread Ivan Khoronzhuk
On 19.11.15 12:22, Stuart Haslam wrote: This makes it a bit easier to get a new system set up for running the unit tests. There has always been a dependency on using a version of CUnit >= 2.1-3, using an older version would result in an error: error: initialization discards ‘const’ qualifier f

Re: [lng-odp] [PATCH 4/5] validation: classification: add odp_cls_cos_pool_set() api

2015-11-19 Thread Stuart Haslam
On Tue, Nov 17, 2015 at 11:46:52AM +0530, Balasubramanian Manoharan wrote: > Assigns a packet pool to CoS using odp_cls_cos_pool_set() api. > This tests that you can assign a pool to a CoS but since it uses the same default_pool it doesn't actually test that it's possible to deliver packets into

Re: [lng-odp] [PATCH 1/5] api: classification: add odp_cls_cos_pool_set() api

2015-11-19 Thread Stuart Haslam
On Tue, Nov 17, 2015 at 11:46:49AM +0530, Balasubramanian Manoharan wrote: > odp_cls_cos_pool_set() api is used to configure packet pool to CoS. > The packets arriving at the CoS is allocated from the configured pool > > Signed-off-by: Balasubramanian Manoharan > --- > include/odp/api/classifica

Re: [lng-odp] [PATCH 3/5] linux-generic: classification: implements odp_cls_cos_pool_set() api

2015-11-19 Thread Stuart Haslam
On Tue, Nov 17, 2015 at 11:46:51AM +0530, Balasubramanian Manoharan wrote: > Adds support for configuring packet pool to a class-of-service. > linux-generic packet parser is enhanced to parse a packet directly from > a memory location rather than from odp_packet_t. > > packet receive code is modif

[lng-odp] [PATCH] example: timer: free resources while termination

2015-11-19 Thread Ivan Khoronzhuk
Example should free resources in right order when terminates. Also it should have correct error path. Signed-off-by: Ivan Khoronzhuk --- example/timer/odp_timer_test.c | 46 ++ 1 file changed, 38 insertions(+), 8 deletions(-) diff --git a/example/timer/od

Re: [lng-odp] [lng] [API-NEXT PATCH v2 7/7] api: time: make local time to be wall time

2015-11-19 Thread Ivan Khoronzhuk
ping. On 13.11.15 11:28, Ivan Khoronzhuk wrote: Petri, Additional API calls like resolution and delay can be added after this patch. The global API can be added after this patch and it will be absolutely similar except global time source (in linux-generic it's the same source). I've added this

[lng-odp] [PATCH 2/4] bootstrap: auto generate platform list

2015-11-19 Thread Nicolas Morey-Chaisemartin
This list all the available platforms looking for platform/*/m4/configure.m4 and generate the appropriate m4 file for the configure.ac to include Signed-off-by: Nicolas Morey-Chaisemartin --- bootstrap| 22 ++ configure.ac | 9 + 2 files changed, 23 insertions(

[lng-odp] [PATCH 4/4] configure.ac: move linux-generic conditionals to platform side

2015-11-19 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin --- configure.ac | 12 platform/linux-generic/m4/conditionals.m4 | 2 ++ 2 files changed, 2 insertions(+), 12 deletions(-) create mode 100644 platform/linux-generic/m4/conditionals.m4 diff --git a/configure.ac

[lng-odp] [PATCH 3/4] bootstrap: auto-include platform specific conditional definitions

2015-11-19 Thread Nicolas Morey-Chaisemartin
Conditionals must always be defined if reference in a Makefile.am even if the Makefile does not apply to the platform being built. As long as all the conditional are defined with default values in platform//m4/conditionals.m4, bootstrap will automatically include them so there won't be any issue w

[lng-odp] [PATCH 1/4] configure.ac: do not set linux-generic as default platform

2015-11-19 Thread Nicolas Morey-Chaisemartin
This prepares for the second patch and add support for repo which do not have a linux-generic platform Signed-off-by: Nicolas Morey-Chaisemartin --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index be21eaf..0d7860b 100644 --- a/c

[lng-odp] [PATCH 0/4] "Dynamic" platform handling

2015-11-19 Thread Nicolas Morey-Chaisemartin
The goal of this patch series is to auto generate the platform list and avoid platform maintaners to have to change the configure.ac to add/remove platforms depending on the setup. It will also work and make sense if odp-api is split from linux generic. As autoconf requires all m4 to be included

Re: [lng-odp] [PATCH 2/2] bootstrap: auto generate platform list

2015-11-19 Thread Nicolas Morey-Chaisemartin
On 11/19/2015 11:04 AM, Anders Roxell wrote: > On 2015-11-19 08:43, Nicolas Morey-Chaisemartin wrote: >> This list all the available platforms looking for >> platform/*/m4/configure.m4 and generate the appropriate m4 >> file for the configure.ac to include >> >> Signed-off-by: Nicolas Morey-Cha

[lng-odp] [PATCH] validation: remove strict dependency on CUnit 2.1-3

2015-11-19 Thread Stuart Haslam
This makes it a bit easier to get a new system set up for running the unit tests. There has always been a dependency on using a version of CUnit >= 2.1-3, using an older version would result in an error: error: initialization discards ‘const’ qualifier from pointer target type [-Werror] caused b

Re: [lng-odp] [PATCH] platform: move list of API files to Makefile.inc so it is common to all platforms

2015-11-19 Thread Maxim Uvarov
Merged, Maxim. On 11/18/2015 22:46, Bill Fischofer wrote: Based on our discussions earlier today, this patch should be applied as it helps whether or not we split the odp.git repo. +1 On Wed, Nov 18, 2015 at 12:14 PM, Mike Holmes > wrote: On 12 November 20

Re: [lng-odp] [PATCH 2/2] bootstrap: auto generate platform list

2015-11-19 Thread Anders Roxell
On 2015-11-19 08:43, Nicolas Morey-Chaisemartin wrote: > This list all the available platforms looking for > platform/*/m4/configure.m4 and generate the appropriate m4 > file for the configure.ac to include > > Signed-off-by: Nicolas Morey-Chaisemartin > --- > bootstrap| 23 +++

Re: [lng-odp] [PATCH] validation: scheduler: add missing ticketlock unlock

2015-11-19 Thread Maxim Uvarov
Merged, Maxim. On 11/18/2015 18:51, Bill Fischofer wrote: On Wed, Nov 18, 2015 at 9:47 AM, Stuart Haslam mailto:stuart.has...@linaro.org>> wrote: The scheduler validation test occasionally deadlocks due to a missing ticketlock unlock. Signed-off-by: Stuart Haslam mailto:stuart.

Re: [lng-odp] [PATCH] pkg/debian/control: add more packages to build-depends

2015-11-19 Thread Anders Roxell
On 19 November 2015 at 10:15, Maxim Uvarov wrote: > On 11/19/2015 11:48, Anders Roxell wrote: >> >> Reported-by: Riku Voipio >> Signed-off-by: Anders Roxell >> Reviewed-by: Riku Voipio >> --- >> pkg/debian/control | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/pk

[lng-odp] [PATCHv2] pkg/debian/control: add more packages to build-depends

2015-11-19 Thread Anders Roxell
Reported-by: Riku Voipio Signed-off-by: Anders Roxell Reviewed-by: Riku Voipio --- pkg/debian/control | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/debian/control b/pkg/debian/control index b4661cc..941e8a8 100644 --- a/pkg/debian/control +++ b/pkg/debian/control @@

[lng-odp] [PATCHv14 9/9] linux-generic: internal ipc_pktio test

2015-11-19 Thread Maxim Uvarov
2 example ipc pktio applications create ipc pktio to each other and do packet transfer, validation magic numbers and packets sequence counters inside it. Signed-off-by: Maxim Uvarov --- platform/linux-generic/Makefile.am | 1 + platform/linux-generic/m4/configure.m4

[lng-odp] [PATCHv14 8/9] linux-generic: add ipc pktio support

2015-11-19 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov --- platform/linux-generic/Makefile.am | 2 + .../linux-generic/include/odp_buffer_internal.h| 3 + .../linux-generic/include/odp_packet_io_internal.h | 38 ++ .../include/odp_packet_io_ipc_internal.h | 47 ++ platform/linux-generic/

[lng-odp] [PATCHv14 7/9] helper: move odp pause to installed helper

2015-11-19 Thread Maxim Uvarov
ipc pktio depends on ring which depends on pause. Move pause to installed helpers to support out of tree builds. Signed-off-by: Maxim Uvarov --- helper/Makefile.am | 4 ++-- helper/{odph_pause.h => include/odp/helper/pause.h} | 0 helper/ring.c

[lng-odp] [PATCHv14 6/9] linix-generic: return error for unsupported pktio calls

2015-11-19 Thread Maxim Uvarov
For some pktios like loop and ipc functions like mtu, promisc, and mac addr are not applicable. Instead of crash on deference null pointer just return error if functions are not defined. Signed-off-by: Maxim Uvarov --- platform/linux-generic/odp_packet_io.c | 24 1 file c

[lng-odp] [PATCHv14 5/9] helper: move ring test to helper

2015-11-19 Thread Maxim Uvarov
Move ring test to helper and kill not needed api_test directory. Unfortunately odp_ring_test.c had some old dirty code so I had to clean up it to to use cunit and latest helper apis. Signed-off-by: Maxim Uvarov --- configure.ac | 1 - helper/ring.c

[lng-odp] [PATCHv14 4/9] helpers: remove odp_ prefix for tests source files

2015-11-19 Thread Maxim Uvarov
Prefixed were removed for validation test suite and to be consistent we need to do the same for helpers. Signed-off-by: Maxim Uvarov --- helper/test/.gitignore | 11 ++- helper/test/Makefile.am | 25 - helper/test/{odp_chksum.c =

[lng-odp] [PATCHv14 3/9] helper: flag to not link ring to linked list

2015-11-19 Thread Maxim Uvarov
Add flag ODPH_RING_NO_LIST to ring to not link it to linked list. Signed-off-by: Maxim Uvarov --- helper/include/odp/helper/ring.h | 7 --- helper/ring.c| 3 ++- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/helper/include/odp/helper/ring.h b/helper/inclu

[lng-odp] [PATCHv14 2/9] linux-generic: create internal pool create function with shm flags

2015-11-19 Thread Maxim Uvarov
On init odp creates odp_sched_pool. We can not modify API to add new parameter to odp_pool_param_t and this pool should not be shared between different processes. To do that implemented internal linux-generic function with parameters to created shm. Note: create shm before and then provide it to th

[lng-odp] [PATCHv14 0/9] IPC (pktio)

2015-11-19 Thread Maxim Uvarov
v14: - rebase on the latest master - move odp pause to installed helper to make out of tree builds happen. v13: - added ipc.README with description and limitations notes. v12: - size size for ring names arrays; - add patch "linix-generic: return error for unsupported pktio calls"

[lng-odp] [PATCHv14 1/9] helper: ring: update ring with shm proc argument

2015-11-19 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov --- helper/include/odp/helper/ring.h | 2 ++ helper/ring.c| 9 - 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/helper/include/odp/helper/ring.h b/helper/include/odp/helper/ring.h index 65c32ad..5e640a7 100644 --- a/helper/in

Re: [lng-odp] [PATCH] pkg/debian/control: add more packages to build-depends

2015-11-19 Thread Maxim Uvarov
On 11/19/2015 11:48, Anders Roxell wrote: Reported-by: Riku Voipio Signed-off-by: Anders Roxell Reviewed-by: Riku Voipio --- pkg/debian/control | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/debian/control b/pkg/debian/control index b4661cc..0bea61a 100644 --- a/pkg

[lng-odp] [PATCH] pkg/debian/control: add more packages to build-depends

2015-11-19 Thread Anders Roxell
Reported-by: Riku Voipio Signed-off-by: Anders Roxell Reviewed-by: Riku Voipio --- pkg/debian/control | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/debian/control b/pkg/debian/control index b4661cc..0bea61a 100644 --- a/pkg/debian/control +++ b/pkg/debian/control @@ -1