Re: [lng-odp] [RFC] api: crypto: Move completion event allocation to implementation

2014-12-08 Thread Alexandru Badicioiu
Some comments inline. Alex On 5 December 2014 at 17:45, Taras Kondratiuk taras.kondrat...@linaro.org wrote: On 12/05/2014 04:16 PM, Alexandru Badicioiu wrote: Hi , what this patch wants to accomplish can be done with the actual API this way : if the implementation really cannot reuse the

Re: [lng-odp] linux-d01's rebase progress

2014-12-08 Thread Weilong Chen
When I try to create a the repo https://git.linaro.org/lng/odp-d01.git. I use this config in git: [remote lng-d01] url = ssh://g...@git.linaro.org/lng/odp-d01.git fetch = +refs/heads/*:refs/remotes/lava/* When I do git push, It says: FATAL: W any lng/odp-d01 weilong.chen DENIED by fallthru

Re: [lng-odp] linux-d01's rebase progress

2014-12-08 Thread Ciprian Barbu
On Mon, Dec 8, 2014 at 11:22 AM, Weilong Chen weilong.c...@linaro.org wrote: When I try to create a the repo https://git.linaro.org/lng/odp-d01.git. I use this config in git: [remote lng-d01] url = ssh://g...@git.linaro.org/lng/odp-d01.git fetch = +refs/heads/*:refs/remotes/lava/*

Re: [lng-odp] [RFC] api: crypto: Move completion event allocation to implementation

2014-12-08 Thread Taras Kondratiuk
On 12/08/2014 10:58 AM, Alexandru Badicioiu wrote: Some comments inline. Alex On 5 December 2014 at 17:45, Taras Kondratiuk taras.kondrat...@linaro.org mailto:taras.kondrat...@linaro.org wrote: On 12/05/2014 04:16 PM, Alexandru Badicioiu wrote: Hi , what this patch wants

Re: [lng-odp] [PATCH] api: odp_align: move internal macros

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
Hi, My original todo list slides said: “Move internal macros (marked with @internal) into odp_align_internal.h” That line just didn’t get copied correctly to the delta doc… -Petri From: ext Mike Holmes [mailto:mike.hol...@linaro.org] Sent: Friday, December 05, 2014 4:25 PM To: Taras

Re: [lng-odp] [PATCHv6 3/6] API: pktio: mac addr get function

2014-12-08 Thread Ola Liljedahl
On 5 December 2014 at 17:20, Maxim Uvarov maxim.uva...@linaro.org wrote: Define API to get MAC address for specific packet i/o and implement linux-generic version. Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org --- platform/linux-generic/include/api/odp_packet_io.h | 12 +++

Re: [lng-odp] [PATCHv4 02/11] API: promisc mode manipulation functions

2014-12-08 Thread Ola Liljedahl
On 5 December 2014 at 17:11, Maxim Uvarov maxim.uva...@linaro.org wrote: On 12/01/2014 09:17 PM, Zoltan Kiss wrote: On 01/12/14 16:15, Maxim Uvarov wrote: Define API and implement promisc functions for linux-generic. Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org ---

Re: [lng-odp] [PATCHv6 4/6] linux-generic: odp_pktio_open loop support

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
-Original Message- From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp- boun...@lists.linaro.org] On Behalf Of ext Ola Liljedahl Sent: Monday, December 08, 2014 1:08 PM To: Maxim Uvarov Cc: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCHv6 4/6] linux-generic:

Re: [lng-odp] [RFC] api: crypto: Move completion event allocation to implementation

2014-12-08 Thread Bala
On Friday 05 December 2014 10:16 PM, Taras Kondratiuk wrote: On 12/05/2014 04:38 PM, Bala Manoharan wrote: Hi, In case of synchronous crypto operation the completion event is need only in case of an error hence the application can check the return code for failure and get info from completion

Re: [lng-odp] [PATCHv6 2/6] API: promisc mode manipulation functions

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
After typo/style corrections: Reviewed-by: Petri Savolainen petri.savolai...@linaro.org -Original Message- From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp- boun...@lists.linaro.org] On Behalf Of ext Maxim Uvarov Sent: Friday, December 05, 2014 6:20 PM To:

[lng-odp] [PATCH v6 2/3] linux-generic:pktio:move static inline functions to header

2014-12-08 Thread Balasubramanian Manoharan
Moves static inline access functions in pktio module to header file. This changes is required as these functions are accessed by classification module Signed-off-by: Balasubramanian Manoharan bala.manoha...@linaro.org --- .../linux-generic/include/odp_packet_io_internal.h | 17 ++

[lng-odp] [PATCH v6 1/3] helper: odph_tcp header description

2014-12-08 Thread Balasubramanian Manoharan
Adds TCP header description struct odph_tcphdr_t in helper directory. This structure is used for accessing TCP header information from the packet Signed-off-by: Balasubramanian Manoharan bala.manoha...@linaro.org --- V6: Incoporates review comments from Bill helper/include/odph_tcp.h

[lng-odp] [PATCH v6 3/3] linux-generic: classification initial implementation

2014-12-08 Thread Balasubramanian Manoharan
The following features are implemented in this classification implementation: * Attaches PMR, PMR_SET to a Pktio entry * Adds classifier object to pktio entry * Attaches CoS values for L2 and L3 QoS to a Pktio entry * Selects ClassOfService for a packet based on PMR, L2 QoS and L3 QoS values *

Re: [lng-odp] [RFC] api: crypto: Move completion event allocation to implementation

2014-12-08 Thread Taras Kondratiuk
On 12/08/2014 01:30 PM, Bala wrote: On Friday 05 December 2014 10:16 PM, Taras Kondratiuk wrote: On 12/05/2014 04:38 PM, Bala Manoharan wrote: Hi, In case of synchronous crypto operation the completion event is need only in case of an error hence the application can check the return code for

[lng-odp] [PATCH] linux-generic: odp_time: include odp_debug_internal.h

2014-12-08 Thread Anders Roxell
commit f158b205eb531646 broke cross compiling. Signed-off-by: Anders Roxell anders.rox...@linaro.org --- platform/linux-generic/odp_time.c | 1 + 1 file changed, 1 insertion(+) diff --git a/platform/linux-generic/odp_time.c b/platform/linux-generic/odp_time.c index 1695f04..9eaacc0 100644 ---

Re: [lng-odp] [PATCH] linux-generic: odp_time: include odp_debug_internal.h

2014-12-08 Thread Anders Roxell
On 8 December 2014 at 13:11, Anders Roxell anders.rox...@linaro.org wrote: commit f158b205eb531646 broke cross compiling. Signed-off-by: Anders Roxell anders.rox...@linaro.org --- platform/linux-generic/odp_time.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [lng-odp] odp_ticketlock_trylock declared but not implemented

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
-Original Message- From: ext Victor Kamensky [mailto:victor.kamen...@linaro.org] Sent: Friday, December 05, 2014 7:25 PM To: Ola Liljedahl Cc: Savolainen, Petri (NSN - FI/Espoo); lng-odp@lists.linaro.org Subject: Re: [lng-odp] odp_ticketlock_trylock declared but not implemented

[lng-odp] [PATCHv7 2/6] API: promisc mode manipulation functions

2014-12-08 Thread Maxim Uvarov
Define API and implement promisc functions for linux-generic. Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org Reviewed-by: Petri Savolainen petri.savolai...@linaro.org --- platform/linux-generic/include/api/odp_packet_io.h | 22 ++ platform/linux-generic/odp_packet_io.c | 84

[lng-odp] [PATCHv6 0/6] pktio changes: API linux-generic cunit

2014-12-08 Thread Maxim Uvarov
v7: - fix typos and loop dev selection; v6: - mac addr fun change from char* to void*; - doxygen correct to promisc mode retval; - check from -1 to = 0. v5: - add lock entries; - fix checks and loop0 - loop rename from previous version; - git

[lng-odp] [PATCHv7 6/6] cunit: pktio: mtu/promisc/mac

2014-12-08 Thread Maxim Uvarov
Add cunit tests for new pktio api: - MAC address get; - Promisc mode set / test; - MTU value set / get; Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org --- test/validation/.gitignore | 3 +- test/validation/Makefile.am | 4 +- test/validation/odp_pktio.c | 155

Re: [lng-odp] [PATCH] odp_version.h: Update version number from 0.3.0 to 0.4.0

2014-12-08 Thread Mike Holmes
We need to add the following to odp/doc/doxygen.cfg PROJECT_NUMBER = 0.4.0 In odp/configure.ac we need the following update AC_INIT([OpenDataPlane], [0.4.0], [lng-odp@lists.linaro.org]) On 8 December 2014 at 10:10, Maxim Uvarov maxim.uva...@linaro.org wrote: Signed-off-by: Maxim

Re: [lng-odp] odp_ticketlock_trylock declared but not implemented

2014-12-08 Thread Victor Kamensky
On 8 December 2014 at 04:41, Savolainen, Petri (NSN - FI/Espoo) petri.savolai...@nsn.com wrote: -Original Message- From: ext Victor Kamensky [mailto:victor.kamen...@linaro.org] Sent: Friday, December 05, 2014 7:25 PM To: Ola Liljedahl Cc: Savolainen, Petri (NSN - FI/Espoo);

Re: [lng-odp] [RFC PATCH] RFC: Implement v0.5 buffer pool APIs

2014-12-08 Thread Savolainen, Petri (NSN - FI/Espoo)
Didn't review the implementation. -Original Message- From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp- boun...@lists.linaro.org] On Behalf Of ext Bill Fischofer Sent: Monday, December 08, 2014 1:24 AM To: lng-odp@lists.linaro.org Subject: [lng-odp] [RFC PATCH] RFC: Implement

Re: [lng-odp] [PATCHv2] Update version number from 0.3.0 to 0.4.0

2014-12-08 Thread Maxim Uvarov
Merged! Maxim. On 12/08/2014 07:27 PM, Mike Holmes wrote: On 8 December 2014 at 11:25, Maxim Uvarov maxim.uva...@linaro.org mailto:maxim.uva...@linaro.org wrote: Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org mailto:maxim.uva...@linaro.org Reviewed-by: Mike Holmes

Re: [lng-odp] [RFC PATCH] RFC: Implement v0.5 buffer pool APIs

2014-12-08 Thread Bill Fischofer
Bala and I had a hangout earlier to discuss his comments. See inline responses for Petri's. On Mon, Dec 8, 2014 at 10:38 AM, Savolainen, Petri (NSN - FI/Espoo) petri.savolai...@nsn.com wrote: Didn't review the implementation. -Original Message- From:

[lng-odp] [PATCH 3/3] test: odp_timer.h: cunit test

2014-12-08 Thread Ola Liljedahl
Signed-off-by: Ola Liljedahl ola.liljed...@linaro.org --- (This document/code contribution attached is provided under the terms of agreement LES-LTM-21309) A new cunit test program test/validation/odp_timer.c for the updated timer API. test/validation/.gitignore | 1 +

[lng-odp] [PATCH 0/3] Timer updates - updated API, new lock-less implementation, new cunit test

2014-12-08 Thread Ola Liljedahl
(This document/code contribution attached is provided under the terms of agreement LES-LTM-21309) The timer API is updated according to https://docs.google.com/a/linaro.org/document/d/1bfY_J8ecLJPsFTmYftb0NVmGnB9qkEc_NpcJ87yfaD8 A major change is that timers are allocated and freed separately

[lng-odp] [PATCHv2] RFC: Implement v0.5 buffer pool APIs

2014-12-08 Thread Bill Fischofer
Signed-off-by: Bill Fischofer bill.fischo...@linaro.org --- v2 of this patch incorporates comments from Bala and Petri. In particular: - buf_size and buf_align in odp_buffer_pool_param_t are now uint32_t - odp_buffer_pool_info_t has been changed to add the shm - odp_buffer_pool_info() has been

Re: [lng-odp] [PATCH 0/3] Timer updates - updated API, new lock-less implementation, new cunit test

2014-12-08 Thread Bill Fischofer
This patch will need rework to be compatible with the buffer pool API changes. Since the plan is for that to merge ahead of timers you can rebase either before or after that happens. Latest buffer pool patch is https://patches.linaro.org/42027/ On Mon, Dec 8, 2014 at 4:49 PM, Ola Liljedahl