[lng-odp] DDF design considerations

2017-06-16 Thread Bogdan Pricope
Hi, This doc contains some thoughts on DDF design. Please take a look and share your comments. https://docs.google.com/a/linaro.org/document/d/1OJg1Ouj0Fktg_Ng93DqJgNwuWA_d8xyLBQ2izd3GuD8/edit?usp=sharing /B

Re: [lng-odp] [API-NEXT PATCH] linux-gen: queue: clean up after modular interface

2017-06-16 Thread Peltonen, Janne (Nokia - FI/Espoo)
Honnappa Nagarahalli wrote: > On 12 June 2017 at 06:11, Petri Savolainen > wrote: > > Clean up function and parameter naming after modular interface > > patch. Queue_t type is referred as "queue internal": queue_int or > > q_int. Term "handle" is reserved for API level handles (e.g. > > odp_queu

Re: [lng-odp] [PATCH API-NEXT 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
On 15.06.2017 14:21, Savolainen, Petri (Nokia - FI/Espoo) wrote: > >> index d71f4464af48..3cd4a73cbefb 100644 >> --- a/platform/linux-generic/odp_event.c >> +++ b/platform/linux-generic/odp_event.c >> @@ -19,6 +19,11 @@ odp_event_type_t odp_event_type(odp_event_t event) >> return _odp_buffer_

[lng-odp] [PATCH API-NEXT v2 0/2] event subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Applies on top of [API-NEXT PATCH v2 0/3] IPSEC packet event These patches provide event subtype implementation and tests. Ideally they should be merged together with event subtypes API definitions. Changes since v1: - Implement odp_event_types() - Update event subtype properly in odp_crypto.c

[lng-odp] [PATCH API-NEXT v2 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov --- platform/linux-generic/include/odp_buffer_inlines.h | 2 ++ platform/linux-generic/include/odp_buffer_internal.h | 3 +++ platform/linux-generic/odp_crypto.c | 14 -- platform/linux-generic/odp_event.c |

[lng-odp] [PATCH API-NEXT v2 2/2] validation: test correctness of events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov --- test/common_plat/validation/api/buffer/buffer.c | 37 +++-- test/common_plat/validation/api/packet/packet.c | 5 test/common_plat/validation/api/timer/timer.c | 14 -- 3 files changed, 39 insertions(+), 17 deletions(-)

Re: [lng-odp] [PATCH API-NEXT 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Savolainen, Petri (Nokia - FI/Espoo)
> >>switch (odp_event_type(event)) { > >> diff --git a/platform/linux-generic/odp_packet.c b/platform/linux- > >> generic/odp_packet.c > >> index eb66af2d3b9c..3789feca45f9 100644 > >> --- a/platform/linux-generic/odp_packet.c > >> +++ b/platform/linux-generic/odp_packet.c > >> @@ -268,6 +268,7

Re: [lng-odp] [PATCH API-NEXT v2 0/2] event subtype implementation

2017-06-16 Thread Savolainen, Petri (Nokia - FI/Espoo)
Reviewed-by: Petri Savolainen Tests for odp_event_types() and subtype for crypto_compl are missing. Those can be added on top of this or as v3. > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Dmitry Eremin-Solenikov > Sent: Friday, June 1

Re: [lng-odp] [PATCH API-NEXT v1 1/1] api: schedule: add thread removal correction

2017-06-16 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Github ODP bot > Sent: Thursday, June 15, 2017 11:00 PM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCH API-NEXT v1 1/1] api: schedule: add thread > removal correction > > From: Honna

Re: [lng-odp] [PATCH API-NEXT 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
On 16.06.2017 12:37, Savolainen, Petri (Nokia - FI/Espoo) wrote: switch (odp_event_type(event)) { diff --git a/platform/linux-generic/odp_packet.c b/platform/linux- generic/odp_packet.c index eb66af2d3b9c..3789feca45f9 100644 --- a/platform/linux-generic/odp_packet.c >>>

Re: [lng-odp] [API-NEXT PATCH] linux-gen: queue: clean up after modular interface

2017-06-16 Thread Elo, Matias (Nokia - FI/Espoo)
>> void *buf_hdr[], int num, int >> *ret); >> typedef int (*schedule_init_global_fn_t)(void); >> typedef int (*schedule_term_global_fn_t)(void); >> diff --git a/platform/linux-generic/odp_queue.c >> b/platform/linux-generic/odp_queue.c >> index 3e18f578.

Re: [lng-odp] Suspected SPAM - Re: [API-NEXT PATCH] linux-gen: queue: clean up after modular interface

2017-06-16 Thread Elo, Matias (Nokia - FI/Espoo)
> On 16 Jun 2017, at 11:03, Peltonen, Janne (Nokia - FI/Espoo) > wrote: > > > Honnappa Nagarahalli wrote: >> On 12 June 2017 at 06:11, Petri Savolainen >> wrote: >>> Clean up function and parameter naming after modular interface >>> patch. Queue_t type is referred as "queue internal": queue_

[lng-odp] [API-NEXT PATCH v3 3/3] api: ipsec: disable event is the last event

2017-06-16 Thread Petri Savolainen
Disable event is guaranteed to be the last event for the SA, so that application can use it for SA destroy synchronization. Signed-off-by: Petri Savolainen --- include/odp/api/spec/ipsec.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/odp/api/spec/ipsec.h b/incl

[lng-odp] [API-NEXT PATCH v3 0/3] IPSEC packet event

2017-06-16 Thread Petri Savolainen
Applies on top of: "[API-NEXT PATCH v2 0/2] IPsec API update" Input and output of IPSEC operations are packets. Parameter and result structures are cleaner when packet arrays are direct parameters to functions. Also API is more flexible for application or API pipelining when output is packets with

[lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread Petri Savolainen
Event subtype gives more detailed information about the event. Two subtypes (basic and IPSEC packet) are introduced initially. Later on, other packet producing APIs (crypto, comp, etc) may also produce packet events with additional subtypes. Signed-off-by: Petri Savolainen --- include/odp/api/sp

[lng-odp] [API-NEXT PATCH v3 2/3] api: ipsec: change IPSEC result to packet

2017-06-16 Thread Petri Savolainen
Input and output of IPSEC operations are packets. Parameter and result structures are cleaner when packet arrays are direct parameters to functions. Also API is more flexible for application and API pipelining when output is packets with additional metadata. Application or API pipeline stages which

[lng-odp] [RFC/API-NEXT] api: schedule: add ordered context unlock-lock

2017-06-16 Thread Balasubramanian Manoharan
Adds api to release an existing ordered lock and acquire a new lock. Signed-off-by: Balasubramanian Manoharan --- include/odp/api/spec/schedule.h | 20 1 file changed, 20 insertions(+) diff --git a/include/odp/api/spec/schedule.h b/include/odp/api/spec/schedule.h index 82

[lng-odp] [PATCH API-NEXT v3 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov --- platform/linux-generic/include/odp_buffer_inlines.h | 2 ++ platform/linux-generic/include/odp_buffer_internal.h | 3 +++ platform/linux-generic/odp_crypto.c | 15 +-- platform/linux-generic/odp_event.c |

[lng-odp] [PATCH API-NEXT v3 0/2] event subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Applies on top of [API-NEXT PATCH v2 0/3] IPSEC packet event These patches provide event subtype implementation and tests. Ideally they should be merged together with event subtypes API definitions. Changes since v2: - Validate odp_event_types() implementation - Validate CRYPTO_COMPL event type

[lng-odp] [PATCH API-NEXT v3 2/2] validation: test correctness of events subtype implementation

2017-06-16 Thread Dmitry Eremin-Solenikov
Signed-off-by: Dmitry Eremin-Solenikov --- test/common_plat/validation/api/buffer/buffer.c| 50 -- .../validation/api/crypto/odp_crypto_test_inp.c| 14 ++ test/common_plat/validation/api/packet/packet.c| 21 - test/common_plat/validation/api/timer/timer

Re: [lng-odp] [PATCH API-NEXT v1 1/1] api: schedule: add thread removal correction

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 5:16 AM, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >> Github ODP bot >> Sent: Thursday, June 15, 2017 11:00 PM >> To: lng-odp@lists.linaro.org >> Subject: [lng-odp] [

Re: [lng-odp] [PATCH API-NEXT 1/2] linux-generic: events subtype implementation

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 5:21 AM, Dmitry Eremin-Solenikov wrote: > On 16.06.2017 12:37, Savolainen, Petri (Nokia - FI/Espoo) wrote: >switch (odp_event_type(event)) { > diff --git a/platform/linux-generic/odp_packet.c b/platform/linux- > generic/odp_packet.c > index eb66af2d3b9c.

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 5:49 AM, Petri Savolainen wrote: > Event subtype gives more detailed information about the event. > Two subtypes (basic and IPSEC packet) are introduced initially. > Later on, other packet producing APIs (crypto, comp, etc) may > also produce packet events with additional s

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread Savolainen, Petri (Nokia - FI/Espoo)
> > diff --git a/include/odp/api/spec/event.h b/include/odp/api/spec/event.h > > index f22efce5..2ad3ce84 100644 > > --- a/include/odp/api/spec/event.h > > +++ b/include/odp/api/spec/event.h > > @@ -37,21 +37,91 @@ extern "C" { > > > > /** > > * @typedef odp_event_type_t > > - * ODP event types:

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 8:04 AM, Savolainen, Petri (Nokia - FI/Espoo) wrote: >> > diff --git a/include/odp/api/spec/event.h b/include/odp/api/spec/event.h >> > index f22efce5..2ad3ce84 100644 >> > --- a/include/odp/api/spec/event.h >> > +++ b/include/odp/api/spec/event.h >> > @@ -37,21 +37,91 @@ e

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread shally verma
On Fri, Jun 16, 2017 at 6:42 PM, Bill Fischofer wrote: > On Fri, Jun 16, 2017 at 8:04 AM, Savolainen, Petri (Nokia - FI/Espoo) > wrote: >>> > diff --git a/include/odp/api/spec/event.h b/include/odp/api/spec/event.h >>> > index f22efce5..2ad3ce84 100644 >>> > --- a/include/odp/api/spec/event.h >>>

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: event: add subtype to expand event type

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 8:33 AM, shally verma wrote: > On Fri, Jun 16, 2017 at 6:42 PM, Bill Fischofer > wrote: >> On Fri, Jun 16, 2017 at 8:04 AM, Savolainen, Petri (Nokia - FI/Espoo) >> wrote: > diff --git a/include/odp/api/spec/event.h b/include/odp/api/spec/event.h > index f22efce5

[lng-odp] [Linaro/odp] b8de7d: api: classification: add additional doxygen docume...

2017-06-16 Thread GitHub
Branch: refs/heads/next Home: https://github.com/Linaro/odp Commit: b8de7dfbadb10da65a0a675db5f588cc3998508a https://github.com/Linaro/odp/commit/b8de7dfbadb10da65a0a675db5f588cc3998508a Author: Bill Fischofer Date: 2017-06-16 (Fri, 16 Jun 2017) Changed paths: M include

Re: [lng-odp] [RFC/API-NEXT] api: schedule: add ordered context unlock-lock

2017-06-16 Thread Bill Fischofer
On Fri, Jun 16, 2017 at 6:21 AM, Balasubramanian Manoharan wrote: > Adds api to release an existing ordered lock and acquire a new lock. > > Signed-off-by: Balasubramanian Manoharan > --- > include/odp/api/spec/schedule.h | 20 > 1 file changed, 20 insertions(+) > > diff -

[lng-odp] [Linaro/odp] b8de7d: api: classification: add additional doxygen docume...

2017-06-16 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: b8de7dfbadb10da65a0a675db5f588cc3998508a https://github.com/Linaro/odp/commit/b8de7dfbadb10da65a0a675db5f588cc3998508a Author: Bill Fischofer Date: 2017-06-16 (Fri, 16 Jun 2017) Changed paths: M inclu

[lng-odp] [Bug 2952] doxygen errors and travis does not catch them

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2952 --- Comment #30 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/b8de7dfbadb10da65a0a675db5f588cc3998508a refs/heads/master 2017-06-16T16:51:09+03:00 Bill Fischofer bill.fischo...@linaro.org api: classification: add additional doxygen documen

[lng-odp] [Bug 2952] doxygen errors and travis does not catch them

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2952 --- Comment #31 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/583dbc9adab460c926339353cc882095594b5b60 refs/heads/master 2017-06-16T16:51:10+03:00 Bill Fischofer bill.fischo...@linaro.org api: pool: add additional doxygen documentation Re

[lng-odp] [Bug 2952] doxygen errors and travis does not catch them

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2952 --- Comment #32 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/b9676fc5ba51d0f26a9c8c40f0b65a003113f1b5 refs/heads/master 2017-06-16T16:51:10+03:00 Bill Fischofer bill.fischo...@linaro.org api: tm: add additional doxygen documentation Reso

[lng-odp] [Bug 2952] doxygen errors and travis does not catch them

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2952 --- Comment #33 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/af05215e26c99693c000981fddfaf3cdd2225e3f refs/heads/master 2017-06-16T16:51:10+03:00 Bill Fischofer bill.fischo...@linaro.org linux-generic: types: add additional doxygen docume

[lng-odp] [Bug 2952] doxygen errors and travis does not catch them

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2952 --- Comment #34 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/629be2dd2d166ed1246b8c1982af3a55a7035657 refs/heads/master 2017-06-16T16:51:10+03:00 Bill Fischofer bill.fischo...@linaro.org helper: add additional doxygen documentation Resol

[lng-odp] [Bug 3003] AES-GCM returns 'valid' tag when checking invalid tag

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3003 --- Comment #5 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/697c3be9b2155379dee458d66335442375a9354d refs/heads/master 2017-06-16T16:51:11+03:00 Dmitry Eremin-Solenikov dmitry.ereminsoleni...@linaro.org linux: crypto: fix checking of GCM

[lng-odp] [Bug 3039] Socket pktio recv fails on large number of packet

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3039 --- Comment #3 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/79ed8c746ba13233c111a61d2d4f17e360dd8023 refs/heads/master 2017-06-16T17:01:11+03:00 Bill Fischofer bill.fischo...@linaro.org changelog: add bug 3039 to list of bug fixes for v1.

[lng-odp] [Bug 2938] make file deps failure

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2938 --- Comment #1 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/0b5e8c6870b959ad9db426aa067326f4baad5d58 refs/heads/master 2017-06-16T17:01:11+03:00 Maxim Uvarov maxim.uva...@linaro.org build: fix order of test execution l2fwd perf test requ

Re: [lng-odp] [RFC/API-NEXT] api: schedule: add ordered context unlock-lock

2017-06-16 Thread Honnappa Nagarahalli
Can you please explain the use case for this API? We already have odp_schedule_order_unlock and odp_schedule_order_lock. These APIs can be used to do the same operation. On 16 June 2017 at 09:12, Bill Fischofer wrote: > On Fri, Jun 16, 2017 at 6:21 AM, Balasubramanian Manoharan > wrote: >> Adds

Re: [lng-odp] [API-NEXT PATCH v4] timer: allow timer processing to run on worker cores

2017-06-16 Thread Honnappa Nagarahalli
Reviewed-by: Honnappa Nagarahalli On 14 June 2017 at 14:34, Brian Brooks wrote: > Run timer pool processing on worker cores if the application hints > that the scheduler will be used. This reduces the latency and jitter > of the point at which timer pool processing begins. See [1] for details. >

Re: [lng-odp] [API-NEXT PATCH] linux-gen: queue: clean up after modular interface

2017-06-16 Thread Honnappa Nagarahalli
On 16 June 2017 at 03:03, Peltonen, Janne (Nokia - FI/Espoo) wrote: > > Honnappa Nagarahalli wrote: >> On 12 June 2017 at 06:11, Petri Savolainen >> wrote: >> > Clean up function and parameter naming after modular interface >> > patch. Queue_t type is referred as "queue internal": queue_int or >

Re: [lng-odp] [API-NEXT PATCH] linux-gen: queue: clean up after modular interface

2017-06-16 Thread Honnappa Nagarahalli
On 16 June 2017 at 05:25, Elo, Matias (Nokia - FI/Espoo) wrote: > >>> void *buf_hdr[], int num, int >>> *ret); >>> typedef int (*schedule_init_global_fn_t)(void); >>> typedef int (*schedule_term_global_fn_t)(void); >>> diff --git a/platform/linux-generic/

Re: [lng-odp] [RFC/API-NEXT] api: schedule: add ordered context unlock-lock

2017-06-16 Thread Bala Manoharan
Regards, Bala On 16 June 2017 at 19:42, Bill Fischofer wrote: > On Fri, Jun 16, 2017 at 6:21 AM, Balasubramanian Manoharan > wrote: > > Adds api to release an existing ordered lock and acquire a new lock. > > > > Signed-off-by: Balasubramanian Manoharan > > --- > > include/odp/api/spec/schedu

Re: [lng-odp] [RFC/API-NEXT] api: schedule: add ordered context unlock-lock

2017-06-16 Thread Bala Manoharan
On 16 June 2017 at 21:14, Honnappa Nagarahalli < honnappa.nagaraha...@linaro.org> wrote: > Can you please explain the use case for this API? > > We already have odp_schedule_order_unlock and odp_schedule_order_lock. > These APIs can be used to do the same operation. > Yes. This API can be interna

[lng-odp] [Bug 3051] New: ./configure has to fail on not recognized option

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3051 Bug ID: 3051 Summary: ./configure has to fail on not recognized option Product: OpenDataPlane - linux- generic reference Version: v1.14.0.0 Hardware: Other OS: Linux Stat

Re: [lng-odp] [API-NEXT PATCH v7 0/5] A scalable software scheduler

2017-06-16 Thread Maxim Uvarov
On 06/14/17 04:21, Brian Brooks wrote: > Brian Brooks (5): > test: odp_pktio_ordered: add queue size > Add arch/ files > Add a bitset > Add a concurrent queue > Add scalable scheduler Please rename commits with existence commit style. Has to be: linux-gen: sched scalable: addr arch files

[lng-odp] [Bug 3052] New: api-next out of tree build broken

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3052 Bug ID: 3052 Summary: api-next out of tree build broken Product: OpenDataPlane - linux- generic reference Version: v1.14.0.0 Hardware: Other OS: Linux Status: UNCONFIRMED

[lng-odp] [Bug 3052] api-next out of tree build broken

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3052 Maxim Uvarov changed: What|Removed |Added Assignee|bala.manoha...@linaro.org |maxim.uva...@linaro.org Component|Clas

[lng-odp] [Bug 3052] api-next out of tree build broken

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3052 Maxim Uvarov changed: What|Removed |Added Assignee|maxim.uva...@linaro.org |petri.savolai...@linaro.org -- You are receivi

[lng-odp] [Bug 3052] api-next out of tree build broken

2017-06-16 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3052 --- Comment #1 from Maxim Uvarov --- Assign to Petri as feature creator of deprecated.h -- You are receiving this mail because: You are on the CC list for the bug.

Re: [lng-odp] [API-NEXT PATCH v3 0/3] IPSEC packet event

2017-06-16 Thread Bill Fischofer
This should still be reviewed by Bala and Nikhil with respect to their HW, but these changes resolve all of the concerns I had with the original version. For his series: Reviewed-by: Bill Fischofer On Fri, Jun 16, 2017 at 5:49 AM, Petri Savolainen wrote: > Applies on top of: "[API-NEXT PATCH v2