[lng-odp] [API-NEXT PATCHv4 5/5] validation: tm: add cunit test for traffic mngr

From: Barry Spinney Signed-off-by: Barry Spinney Signed-off-by: Bill Fischofer --- configure.ac |1 + test/validation/Makefile.am |1 +

[lng-odp] [API-NEXT PATCHv4 4/5] linux-generic: tm: fix numerous bugs in timer wheel and main tm implementation

From: Barry Spinney Signed-off-by: Barry Spinney Signed-off-by: Bill Fischofer --- platform/linux-generic/odp_timer_wheel.c | 230 --- platform/linux-generic/odp_traffic_mngr.c | 1024 +++--

[lng-odp] [API-NEXT PATCHv4 3/5] linux-generic: tm: correct some old comments and stop using cpu cycles

From: Barry Spinney This patch fixes some incorrect comments regarding the format of the fixed point integer types used to represent the shaper's token bucket count values. Also since the TM and the TM tests were changed to use odp_time_local() instead of the cpu cycles

[lng-odp] [API-NEXT PATCHv4 2/5] linux-generic: tm: Add missing name table initialization call.

From: Barry Spinney This patch adds a call from odp_init.c to _odp_int_name_tbl_init(). Signed-off-by: Barry Spinney Signed-off-by: Bill Fischofer --- platform/linux-generic/include/odp_internal.h | 9 +++--

[lng-odp] [API-NEXT PATCHv4 1/5] example: tm: remove reference to internal name table init api

From: Barry Spinney Signed-off-by: Barry Spinney Signed-off-by: Bill Fischofer --- example/traffic_mgmt/odp_traffic_mgmt.c | 4 1 file changed, 4 deletions(-) diff --git a/example/traffic_mgmt/odp_traffic_mgmt.c

[lng-odp] pktio stats counters

Hi, On ODP-DPDK this assert fails every now and then: https://git.linaro.org/lng/odp.git/blob/refs/heads/api-next:/test/validation/pktio/pktio.c#l1193 The reason is that when you set the interfaces up, there are other system services which can hook into the interface creation, and start

Re: [lng-odp] why odp driver can not use odp api?

On 18 February 2016 at 07:15, Maxim Uvarov wrote: > On 02/18/16 14:58, Mike Holmes wrote: > >> >> >> On 18 February 2016 at 06:33, Maxim Uvarov > > wrote: >> >> Hello, >> >> Does someone have clear view

Re: [lng-odp] scheduler_test_wait_time() tolerance

On 18 February 2016 at 12:52, Zoltan Kiss wrote: > > > On 17/02/16 16:59, Mike Holmes wrote: > >> >> >> On 17 February 2016 at 11:53, Ivan Khoronzhuk >> > wrote: >> >> >> >> On 17.02.16 18:44, Mike Holmes

Re: [lng-odp] scheduler_test_wait_time() tolerance

On 17/02/16 16:59, Mike Holmes wrote: On 17 February 2016 at 11:53, Ivan Khoronzhuk > wrote: On 17.02.16 18:44, Mike Holmes wrote: Does this tuning need to be documented in the implementer's guide ? I thinks,

Re: [lng-odp] [PATCH] validation: scheduler: correct pause/resume sequence

ping On 16.02.16 16:02, Ivan Khoronzhuk wrote: When test for single thread is finished the following test for many threads can be started, and for some implementations can happen that future one event can arrive to main thread, as it was requested in previous test. As result one event can be

Re: [lng-odp] [API-NEXT PATCHv3 5/6] test/validation/traffic_mngr: Add the .gitignore and Makefile.am files.

this patch also should be merged with patch 6 and merged with corresponding code. Maxim. On 02/18/16 05:03, Bill Fischofer wrote: From: Barry Spinney This patch now adds the Makefile for the traffic_mngr tests. Signed-off-by: Barry Spinney

Re: [lng-odp] [API-NEXT PATCHv3 6/6] odp build system: Changed configure.ac and test/validation/Makefile.am so that traffic_mngr tests are built.

that patch is better to merge to corresponding patches. They strange see only Makefile changes in separate file. Maxim. On 02/18/16 05:03, Bill Fischofer wrote: From: Barry Spinney These two changes now cause the traffic_mngr validation tests to be automatically built as

[lng-odp] [Bug 1449] odp_timer_test core dump

https://bugs.linaro.org/show_bug.cgi?id=1449 Maxim Uvarov changed: What|Removed |Added Status|CONFIRMED |RESOLVED

[lng-odp] [Bug 1548] Untested API parse_ipv6

https://bugs.linaro.org/show_bug.cgi?id=1548 Bill Fischofer changed: What|Removed |Added Status|CONFIRMED |RESOLVED

[lng-odp] [Bug 1549] Untested API parse_tcp

https://bugs.linaro.org/show_bug.cgi?id=1549 Bill Fischofer changed: What|Removed |Added Status|CONFIRMED |RESOLVED

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

https://bugs.linaro.org/show_bug.cgi?id=1615 Maxim Uvarov changed: What|Removed |Added Status|CONFIRMED |RESOLVED

[lng-odp] [Bug 2023] linux-generic: validation: fix dependency for tests-validation.env

https://bugs.linaro.org/show_bug.cgi?id=2023 Bill Fischofer changed: What|Removed |Added CC|

[lng-odp] [Bug 2025] odp/test/api_tests dir is not used and the code is stale

https://bugs.linaro.org/show_bug.cgi?id=2025 Maxim Uvarov changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] [PATCH V2 0/2] linux-generic: fix BUG 2027 - ODP cpumask creation issues

ping. Please review. Maxim. On 02/16/16 17:54, Gary S. Robertson wrote: This patch revision identifies linux-generic as the component being modified, per Maxim U.'s suggested correction. Skipped repeat of the problem description and chosen path to solution in this revision - see the cover

[lng-odp] [Bug 2031] CID 157958: Integer handling issues: timer.c

https://bugs.linaro.org/show_bug.cgi?id=2031 Maxim Uvarov changed: What|Removed |Added CC|

Re: [lng-odp] [API-NEXT PATCHv3 1/6] linux-generic:tm:Add missing name table initialization call.

Ok, I'll do a v4 based on all of your review comments. Please advise when they're complete. On Thu, Feb 18, 2016 at 7:14 AM, Maxim Uvarov wrote: > On 02/18/16 15:51, Bill Fischofer wrote: > >> This was just a rebase. I agree we should add a stage. I'll do that in >>

Re: [lng-odp] [API-NEXT PATCHv3 1/6] linux-generic:tm:Add missing name table initialization call.

On 02/18/16 15:51, Bill Fischofer wrote: This was just a rebase. I agree we should add a stage. I'll do that in a separate patch if this one is acceptable. Or would you prefer to see that added directly here? If possible that changes should be in that serries to not postpone clean up

Re: [lng-odp] [API-NEXT PATCHv3 4/6] test/validation/traffic_mngr: Add the new traffic_mngr tests.

On 02/18/16 05:03, Bill Fischofer wrote: +static void flush_leftover_pkts(odp_tm_t odp_tm, odp_pktio_t pktio) +{ + odp_packet_t rcv_pkt; + odp_time_t start_time, current_time, duration; + uint64_t min_timeout_ns, max_timeout_ns, duration_ns; + int rc; + +

Re: [lng-odp] [API-NEXT PATCHv3 4/6] test/validation/traffic_mngr: Add the new traffic_mngr tests.

Agreed. Again, this was just a rebase of Barry's earlier patch that predated that routine. I can post a follow-on patch to address this. On Thu, Feb 18, 2016 at 6:52 AM, Maxim Uvarov wrote: > On 02/18/16 05:03, Bill Fischofer wrote: > >> +static void

Re: [lng-odp] [API-NEXT PATCHv3 4/6] test/validation/traffic_mngr: Add the new traffic_mngr tests.

On 02/18/16 05:03, Bill Fischofer wrote: + odp_pool_param_init(_param); + odp_pktio_param_init(_param); + + pktio_param.in_mode = ODP_PKTIN_MODE_DIRECT; + pool_param.pkt.num = 10 * MAX_PKTS; + pool_param.type = ODP_POOL_PACKET; Just my preference to have init

Re: [lng-odp] [API-NEXT PATCHv3 4/6] test/validation/traffic_mngr: Add the new traffic_mngr tests.

On 02/18/16 05:03, Bill Fischofer wrote: + +static odp_bool_t approx_eq32(uint32_t val, uint32_t correct) +{ + uint64_t low_bound, val_times_100, high_bound; + + if (val == correct) + return true; + + low_bound = 98 * (uint64_t)correct; + val_times_100

[lng-odp] [Bug 2016] validation: add test case for odp_cos_drop()

https://bugs.linaro.org/show_bug.cgi?id=2016 --- Comment #1 from Bala Manoharan --- v1 submitted: https://patches.linaro.org/patch/62171/ -- You are receiving this mail because: You are on the CC list for the bug.___

Re: [lng-odp] [API-NEXT PATCHv3 4/6] test/validation/traffic_mngr: Add the new traffic_mngr tests.

On 02/18/16 05:03, Bill Fischofer wrote: +static void busy_wait(uint64_t nanoseconds) +{ + odp_time_t start_time, end_time; + uint32_t cnt; + + start_time = odp_time_local(); + end_time = odp_time_sum(start_time, +

Re: [lng-odp] [API-NEXT PATCHv3 1/6] linux-generic:tm:Add missing name table initialization call.

This was just a rebase. I agree we should add a stage. I'll do that in a separate patch if this one is acceptable. Or would you prefer to see that added directly here? On Thu, Feb 18, 2016 at 6:41 AM, Maxim Uvarov wrote: > On 02/18/16 05:03, Bill Fischofer wrote: >

[lng-odp] [PATCH v1] validation: classification: add test case for odp_cos_drop() function

Fixes: https://bugs.linaro.org/show_bug.cgi?id=2016 Signed-off-by: Balasubramanian Manoharan --- test/validation/classification/odp_classification_basic.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/validation/classification/odp_classification_basic.c

Re: [lng-odp] [API-NEXT PATCHv3 1/6] linux-generic:tm:Add missing name table initialization call.

On 02/18/16 05:03, Bill Fischofer wrote: From: Barry Spinney This patch adds a call from odp_init.c to _odp_int_name_tbl_init(). Signed-off-by: Barry Spinney Signed-off-by: Bill Fischofer ---

Re: [lng-odp] [API-NEXT PATCH 1/4] api: crypto: add odp_crypto_capability() api

Thanks, all. v2 submitted to fix the typo Petri noted. On Thu, Feb 18, 2016 at 5:23 AM, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > Yes that’s the idea, but those can be added later on top of this initial > definition. > > > > We’d need to decide, if capability

[lng-odp] [API-NEXT PATCHv2 4/4] validation: crypto: add unit test for odp_crypto_capability()

Signed-off-by: Bill Fischofer --- test/validation/crypto/crypto.c | 8 1 file changed, 8 insertions(+) diff --git a/test/validation/crypto/crypto.c b/test/validation/crypto/crypto.c index b6d06b6..8f4ce87 100644 --- a/test/validation/crypto/crypto.c +++

[lng-odp] [API-NEXT PATCHv2 3/4] validation: crypto: use odp_pool_param_init() instead of memset()

Signed-off-by: Bill Fischofer --- test/validation/crypto/crypto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/crypto/crypto.c b/test/validation/crypto/crypto.c index a0411c6..b6d06b6 100644 --- a/test/validation/crypto/crypto.c

[lng-odp] [API-NEXT PATCHv2 1/4] api: crypto: add odp_crypto_capability() api

Signed-off-by: Bill Fischofer --- include/odp/api/spec/crypto.h | 18 ++ 1 file changed, 18 insertions(+) diff --git a/include/odp/api/spec/crypto.h b/include/odp/api/spec/crypto.h index 41beedb..81e4621 100644 --- a/include/odp/api/spec/crypto.h +++

Re: [lng-odp] [API-NEXT PATCH] api: time: fix typo for cmp function

Merged, Maxim. On 02/17/16 18:53, Zoltan Kiss wrote: Signed-off-by: Zoltan Kiss On 17/02/16 15:34, Ivan Khoronzhuk wrote: Signed-off-by: Ivan Khoronzhuk --- include/odp/api/time.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

Re: [lng-odp] why odp driver can not use odp api?

On 18 February 2016 at 06:33, Maxim Uvarov wrote: > Hello, > > Does someone have clear view why odp driver can not use odp api functions? > Can you please explain why code should be duplicated? > I thought we defined these at least Required change to an existing api

Re: [lng-odp] [API-NEXT PATCHv4] linux-generic: pktio: add option to enable and disable ipc pktio

Merged with change. Will see not what CI show. Thanks, Maxim. On 02/18/16 14:21, Elo, Matias (Nokia - FI/Espoo) wrote: Looks good to me assuming you rename PKTIO_IPC to _ODP_PKTIO_IPC on apply. Reviewed-by: Matias Elo -Original Message- From: lng-odp

[lng-odp] 答复: lng-odp Digest, Vol 23, Issue 207

Hi Uvarov, Problem has been fixed, thanks. As I re-write the makefile of ODP on aarch64 of SUSE11, the TEST_THRMASK macro was missed, which caused this compiling failure. Thank you very much. -邮件原件- 发件人: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] 代表

Re: [lng-odp] [API-NEXT PATCH 1/4] api: crypto: add odp_crypto_capability() api

Yes that’s the idea, but those can be added later on top of this initial definition. We’d need to decide, if capability struct lists algos supported OR algos supported on HW (and demand that everything is always implemented either in SW or HW). Maybe it makes sense to allow low end or domain

Re: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: pktio: add option to enable and disable ipc pktio

On 02/18/16 12:06, Savolainen, Petri (Nokia - FI/Espoo) wrote: -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim Uvarov Sent: Thursday, February 18, 2016 10:45 AM To: Elo, Matias (Nokia - FI/Espoo) ; lng-

Re: [lng-odp] [API-NEXT PATCH 1/4] api: crypto: add odp_crypto_capability() api

Shall supported algos be add in crypto capabilities? Regards Nikhil From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Bala Manoharan Sent: Thursday, February 18, 2016 10:40 AM To: Bill Fischofer Cc: LNG ODP Mailman List

Re: [lng-odp] [PATCH v3 API-NEXT 1/4] linux-generic: separate x86 ODP_CACHE_LINE_SIZE to its arch file

On 9 February 2016 at 18:41, Anders Roxell wrote: > On 2016-02-05 16:56, hongbo.zh...@linaro.org wrote: >> From: Hongbo Zhang >> >> Currently all ODP_CACHE_LINE_SIZE macros for different architectures are >> held in one header file, they should

Re: [lng-odp] crypto contexts

Othe option is instead of taking key and session as input, it is better to add an API odp_crypto_op_xxx which takes all input needed (Algos/Keys/buffer as input) and perform the single crypto operation without any session. Regards Nikhil From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org]

Re: [lng-odp] crypto contexts

Hi Nikhil, On 18 February 2016 at 15:24, Nikhil Agarwal wrote: > Hi, > > > > It seems none of your statement is true. Comments inline. > > It is platform implementation specific that how many crypto sessions it > supports. If it does not support required number of

Re: [lng-odp] crypto contexts

On 18 February 2016 at 13:58, Gábor Sándor Enyedi < gabor.sandor.eny...@ericsson.com> wrote: > OK, so back to the original question: I have up to ~100K users (but always > higher than 32 by orders of magnitude) all with its own crypto key. In > worst case, all of them are sending packets at the

Re: [lng-odp] [QUESTION] test: validation:thread: some functions cannot be found

On 02/18/16 12:09, Xu Zaibo wrote: Hi, In my compiling test validation, some functions, such as ‘thread_test_odp_thrmask_and’, ’ thread_test_odp_thrmask_or’…. cannot be found and failed. And I search these function in the home directory of ODP, but get nothing. So, is it the problem of

Re: [lng-odp] crypto contexts

Hi, It seems none of your statement is true. Comments inline. It is platform implementation specific that how many crypto sessions it supports. If it does not support required number of sessions, then only way out as of today is to create n destroy session. Bala, If this is a generic

[lng-odp] [API-NEXT PATCHv4] linux-generic: pktio: add option to enable and disable ipc pktio

With more options to enable/disable features it's more easy to isolate and debug problem if any. For now I have suspitious that in CI make check runs in parallel with other make checks which share the same pool memory. I.e. one process can corrupt memory for other process. Moving IPC to option it

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

I am willing to put the following definitions: ODP instantiation process:: The process calling +odp_init_global()+, which is probably the first process which is started when an ODP application is started. There is one single such process per ODP instantiation. ODP thread:: An

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

On 18 February 2016 at 09:50, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > > > > > *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, February 18, 2016 10:40 AM > *To:* Savolainen, Petri (Nokia - FI/Espoo)

[lng-odp] [QUESTION] test: validation:thread: some functions cannot be found

Hi, In my compiling test validation, some functions, such as 'thread_test_odp_thrmask_and', ' thread_test_odp_thrmask_or' cannot be found and failed. And I search these function in the home directory of ODP, but get nothing. So, is it the problem of 'validation/thread.c'? Zaibo Xu

Re: [lng-odp] [PATCHv2 2/4] doc: user-guide: shmem

On 18 February 2016 at 09:29, Maxim Uvarov wrote: > On 02/18/16 11:06, Christophe Milard wrote: > >> section regarding shared memo added. >> >> Signed-off-by: Christophe Milard >> Reviewed-and-tested-by: Bill Fischofer

Re: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: pktio: add option to enable and disable ipc pktio

> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Maxim Uvarov > Sent: Thursday, February 18, 2016 10:45 AM > To: Elo, Matias (Nokia - FI/Espoo) ; lng- > o...@lists.linaro.org > Subject: Re: [lng-odp] [API-NEXT PATCHv3

Re: [lng-odp] [PATCH API-NEXTv2] doc: user-guide documentation for classification

On 18 February 2016 at 08:54, Balasubramanian Manoharan < bala.manoha...@linaro.org> wrote: > Adds user-guide documentation for classification module > > Signed-off-by: Balasubramanian Manoharan > --- > v2: Incorporates review comments from Christophe > >

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

From: EXT Christophe Milard [mailto:christophe.mil...@linaro.org] Sent: Thursday, February 18, 2016 10:40 AM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: mike.hol...@linaro.org; bill.fischo...@linaro.org; lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCHv2 1/4]

Re: [lng-odp] [API-NEXT PATCH 1/4] api: crypto: add odp_crypto_capability() api

> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Bill Fischofer > Sent: Wednesday, February 17, 2016 9:37 PM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [API-NEXT PATCH 1/4] api: crypto: add > odp_crypto_capability() api > >

Re: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: pktio: add option to enable and disable ipc pktio

Hello Matias, thanks for comments will send v4. During update I thinge we can not use ODP_ prefix here: +#ifdef ODP_PKTIO_IPC _pktio_ops, +#endif as well as in: +#ifdef ODP_PKTIO_DPDK +#endif Because it might be confusing other people thinking that it's related to some

Re: [lng-odp] [PATCHv2 2/4] doc: user-guide: shmem

On 02/18/16 11:06, Christophe Milard wrote: section regarding shared memo added. Signed-off-by: Christophe Milard Reviewed-and-tested-by: Bill Fischofer --- doc/users-guide/users-guide.adoc | 129

Re: [lng-odp] crypto contexts

OK, so back to the original question: I have up to ~100K users (but always higher than 32 by orders of magnitude) all with its own crypto key. In worst case, all of them are sending packets at the same time, so I need to decrypt a lot of packets from other users, before I face a packet from

Re: [lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Christophe Milard > Sent: Thursday, February 18, 2016 10:07 AM > To: mike.hol...@linaro.org; bill.fischo...@linaro.org > Cc: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCHv2 1/4] doc:

Re: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: pktio: add option to enable and disable ipc pktio

Comments below. -Matias > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim > Uvarov > Sent: Wednesday, February 17, 2016 9:43 AM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: pktio: add option

[lng-odp] [PATCHv2 4/4] doc: removing invalid characters

linux "tree" command output contains non printable character 160 which made its way into the asciidoc. Removed and replaced by space here. Signed-off-by: Christophe Milard Reviewed-and-tested-by: Bill Fischofer ---

[lng-odp] [PATCHv2 3/4] doc: user-guide: fixing other typos

Signed-off-by: Christophe Milard Reviewed-and-tested-by: Bill Fischofer --- doc/users-guide/users-guide.adoc | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/users-guide/users-guide.adoc

[lng-odp] [PATCHv2 1/4] doc: glossary: defining ODP thread more precisely

Signed-off-by: Christophe Milard Reviewed-and-tested-by: Bill Fischofer --- doc/glossary.adoc | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/doc/glossary.adoc b/doc/glossary.adoc index 8e514ee..becaf16

[lng-odp] [PATCHv2 2/4] doc: user-guide: shmem

section regarding shared memo added. Signed-off-by: Christophe Milard Reviewed-and-tested-by: Bill Fischofer --- doc/users-guide/users-guide.adoc | 129 +++ 1 file changed, 129 insertions(+) diff

[lng-odp] [PATCHv2 0/4] shmem docs

since v1: fixed escape around '+' char. (Bill) Documentation updates documenting shmem and related items. Christophe Milard (4): doc: glossary: defining ODP thread more precisely doc: user-guide: shmem doc: user-guide: fixing other typos doc: removing invalid characters

Re: [lng-odp] [PATCH] helper: remove ring code

merged, Maxim. On 02/17/16 21:20, Mike Holmes wrote: On 17 February 2016 at 05:16, Maxim Uvarov > wrote: Remove ring and dead api-next code. In api-next pktio ipc patches moved ring code to linux-generic and updated that test