Re: [lng-odp] [PATCHv2] apply-and-build.sh: cosmetic fix

On 2015-05-29 12:58, Christophe Milard wrote: > function apply_patch() now returns a status rather than directely breaking the > loop of its caller. (for loop in main, continue statment in apply_patch()) > > Signed-off-by: Christophe Milard > --- > apply-and-build.sh | 7 ++- > 1 file change

Re: [lng-odp] [PATCHv2] apply-and-build.sh: fixing cover-letter issue

On 2015-05-29 12:58, Christophe Milard wrote: > When an empty patch (such as a cover letter) was given to > apply-and-build, the latter would fail applying the patch and all > following patches (saying OK when it could be wrong) > > Signed-off-by: Christophe Milard > --- > apply-and-build.sh | 1

[lng-odp] [PATCHv7 0/5] IPC (pktio)

v7: - 1. do pool_create things more accurate. Pass shm flags internal where is needed. Do not allocate shm before because Bill already posted patch to remove shm from odp_pool_create. 2. Rework example application: remove time stamps. Add sequence counter and magic number

[lng-odp] [PATCHv7 1/5] linux-generic: zero params for pool create

Adding more params to pool create breaks original work. Make sure that not set params are zero. Signed-off-by: Maxim Uvarov --- platform/linux-generic/odp_schedule.c | 1 + test/validation/odp_queue.c | 1 + 2 files changed, 2 insertions(+) diff --git a/platform/linux-generic/odp_sche

[lng-odp] [PATCHv7 3/5] linux-generic: create internal pool create function with shm flags

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] [PATCHv7 2/5] api ipc: update ring with shm proc argument

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

[lng-odp] [PATCHv7 5/5] ipc: example app

Simple example app creates one packet i/o to external interface and one ipc pktio to other process. Then transfer packet from external interface to other process and back thought ipc queue. Signed-off-by: Maxim Uvarov --- configure.ac| 1 + example/Makefile.am | 2 +- example

[lng-odp] [PATCHv7 4/5] linux-generic: add ipc pktio support

Signed-off-by: Maxim Uvarov --- helper/include/odp/helper/ring.h | 7 +- helper/ring.c | 3 +- platform/linux-generic/Makefile.am | 3 + .../linux-generic/include/odp_buffer_internal.h| 3 + .../linux-generic/inclu

Re: [lng-odp] [PATCH] linux-generic: timer: use plain buffers for timeouts

If I understand your patch correctly, a timeout event is really a buffer with the timeout-specific header stored just after the buffer header (as before). You don't change the definition of odp_timeout_hdr_t and the implementation of odp_timeout_hdr(odp_buffer_t buf) has just moved from odp_timer_i

[lng-odp] Weekly Public Call Update - Important

Please note that beginning tomorrow, June 2nd, we are moving the ODP weekly call to Blue Jeans to address some of the audio issues we were having with Uberconference. The time of the call is unchanged: 15:00 UTC every Tuesday. A redirector has been set up, so to attend just point your web browser

[lng-odp] [PATCHv2 0/2] checkpatch: turn on spelling checks

v2: 1) do not add codespell depandancy to configure.ac 2) simplify description that spelling.txt was taken from linux kernel. Maxim Uvarov (2): checkpatch: add spelling.txt checkpatch: use codespell for odp .checkpatch.conf |1 + scripts/spelling.txt | 1043 +

[lng-odp] [PATCHv2 1/2] checkpatch: add spelling.txt

Add linux kernel spelling.txt file to detect common typos. Signed-off-by: Maxim Uvarov --- scripts/spelling.txt | 1043 ++ 1 file changed, 1043 insertions(+) create mode 100644 scripts/spelling.txt diff --git a/scripts/spelling.txt b/scripts/spel

[lng-odp] [PATCHv2 2/2] checkpatch: use codespell for odp

Turn on spelling checks with codespell tool. Tool can be installed with apt-get install codespell. Signed-off-by: Maxim Uvarov --- .checkpatch.conf | 1 + 1 file changed, 1 insertion(+) diff --git a/.checkpatch.conf b/.checkpatch.conf index 9076410..17fcea6 100644 --- a/.checkpatch.conf +++ b/.

Re: [lng-odp] [PATCHv2 0/2] checkpatch: turn on spelling checks

On 1 June 2015 at 14:58, Maxim Uvarov wrote: > v2: 1) do not add codespell depandancy to configure.ac You need to add "depandancy" to spelling.txt. > > 2) simplify description that spelling.txt was taken from linux kernel. > > > Maxim Uvarov (2): > checkpatch: add spelling.txt > check

Re: [lng-odp] [PATCHv2 0/2] checkpatch: turn on spelling checks

On 06/01/15 17:14, Ola Liljedahl wrote: On 1 June 2015 at 14:58, Maxim Uvarov > wrote: v2: 1) do not add codespell depandancy to configure.ac You need to add "depandancy" to spelling.txt. yes :) Maxim. 2) simplify descr

Re: [lng-odp] [API-NEXT PATCH v2 0/4] Init API and thread type

On 05/29/15 14:06, Savolainen, Petri (Nokia - FI/Espoo) wrote: Ping. Petri can you please reply on Bills comment? I think some more info is needed why do we need that changes. Thank you, Maxim. -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf O

Re: [lng-odp] [PATCH 0/6] validation: pktio test move to platform side

Hello Christophe, I just added few tests cases which we tested with Ciprian for current version. I think you don't need to remove from PATH original directories. Comments are: 1. I don't like that name. _main is not needed: ./test/validation/pktio/pktio_main 2. Then I run ./scripts/builddeb

[lng-odp] [Bug 1603] New: regression: scripts/git_hash.sh: No such file or directory

https://bugs.linaro.org/show_bug.cgi?id=1603 Bug ID: 1603 Summary: regression: scripts/git_hash.sh: No such file or directory Product: OpenDataPlane Version: unspecified Hardware: Other OS: Linux

Re: [lng-odp] [PATCH] linux-generic: pool: group and document pool statistics

Merged, thanks, Maxim. On 05/28/15 14:41, Bill Fischofer wrote: Address bug https://bugs.linaro.org/show_bug.cgi?id=1480 Signed-off-by: Bill Fischofer --- platform/linux-generic/include/odp_pool_internal.h | 44 +++ platform/linux-generic/odp_pool.c | 50 ++

[lng-odp] [PATCH] test: pktio_perf: add support for batch reception

Signed-off-by: Nicolas Morey-Chaisemartin --- test/performance/odp_pktio_perf.c | 72 +-- 1 file changed, 54 insertions(+), 18 deletions(-) diff --git a/test/performance/odp_pktio_perf.c b/test/performance/odp_pktio_perf.c index fbe27a0..bb557b0 100644 --- a/

Re: [lng-odp] [API-NEXT PATCH v2 0/4] Init API and thread type

Hi, I have stated the reasoning in couple of weekly calls and this cover letter: Implementations may be HW limited on how many worker threads can be supported (efficiently). Number of control threads may be only SW limited. When user identifies worker threads (from control threads), an implemen