[lng-odp] [PATCH v1/1] linux-generic : crypto

2014-10-31 Thread alexandru.badicioiu
From: Alexandru Badicioiu alexandru.badici...@linaro.org Add missing API functions to crypto implementation. Required by crypto unit testing. Signed-off-by: Alexandru Badicioiu alexandru.badici...@linaro.org --- platform/linux-generic/odp_crypto.c | 26 ++ 1 files

Re: [lng-odp] Move Typedefs to a new Header File

2014-10-31 Thread Bill Fischofer
Adding the ODP mailing list since this should be a topic of general interest. Now that we have separate repositories for each implementation what we'd ideally like is the following. - The API definitions are in .h files in odp.git. These are the function prototypes for all of the public

[lng-odp] [PATCH v1/1] linux-generic : crypto

2014-10-31 Thread alexandru.badicioiu
From: Alexandru Badicioiu alexandru.badici...@linaro.org Add missing API functions to crypto implementation. Required by crypto unit testing. Signed-off-by: Alexandru Badicioiu alexandru.badici...@linaro.org --- platform/linux-generic/odp_crypto.c | 26 ++ 1 files

[lng-odp] [PATCH v2 1/1] cunit : add tests for crypto APIs

2014-10-31 Thread alexandru.badicioiu
From: Alexandru Badicioiu alexandru.badici...@linaro.org This patch adds a suite for async inplace mode of crypto APIs. Correctness of crypto operations output is verified with known test vectors as well as various options and functionalities: use session IV or operation IV for ciphering, use

[lng-odp] [PATCH] Scheduler atomic and ordered definitions

2014-10-31 Thread Petri Savolainen
Improved atomic and ordered synchronisation definitions. Added order skip function prototype. Signed-off-by: Petri Savolainen petri.savolai...@linaro.org --- This is the ordered queue definition (in patch format) promised in the call yesterday. --- platform/linux-generic/include/api/odp_queue.h

Re: [lng-odp] [PATCH v1/1] linux-generic : crypto

2014-10-31 Thread Mike Holmes
On 31 October 2014 08:07, alexandru.badici...@linaro.org wrote: From: Alexandru Badicioiu alexandru.badici...@linaro.org Add missing API functions to crypto implementation. Required by crypto unit testing. Signed-off-by: Alexandru Badicioiu alexandru.badici...@linaro.org ---

Re: [lng-odp] [PATCH] Scheduler atomic and ordered definitions

2014-10-31 Thread Bill Fischofer
I can well imagine the step-by-step order preservation to be simpler to implement (in SW) but it would also seem to limit performance since the only way to ensure end-to-end order preservation would be if each intermediate queue from ingress to egress were an ordered queue. If there is a parallel

Re: [lng-odp] [PATCH] Scheduler atomic and ordered definitions

2014-10-31 Thread Savolainen, Petri (NSN - FI/Espoo)
Yes, it’s step-by-step and I think it’s the level of ordering we need for v1.0. Most SoCs can implement it, even when the HW scheduler would not have order support but only atomic/parallel. This way defined atomic scheduling can be used to implement functionality correct ordered queues, the

Re: [lng-odp] [PATCH] Scheduler atomic and ordered definitions

2014-10-31 Thread Bill Fischofer
This may well be a reasonable restriction for ODP v1.0 but I believe it's something we need to put on the list for production grade improvements for 2015. Bill On Fri, Oct 31, 2014 at 7:57 AM, Savolainen, Petri (NSN - FI/Espoo) petri.savolai...@nsn.com wrote: Yes, it’s step-by-step and I

Re: [lng-odp] [PATCH] Scheduler atomic and ordered definitions

2014-10-31 Thread Bill Fischofer
It would be documentation in v1.0 but worth keeping since if the step-by-step restriction is relaxed next year it would mean we wouldn't have to change the API. On Fri, Oct 31, 2014 at 8:10 AM, Alexandru Badicioiu alexandru.badici...@linaro.org wrote: In this case the dest argument in the

[lng-odp] [PATCH] Removed odp_atomic_int_t

2014-10-31 Thread Petri Savolainen
Integer version is not needed. Unsigned 32 and 64 bit atomics are used instead. If signed 32/64 bits can be added later on need basis. Signed-off-by: Petri Savolainen petri.savolai...@linaro.org --- platform/linux-generic/include/api/odp_atomic.h| 115 -

Re: [lng-odp] [PATCH v2 1/1] cunit : add tests for crypto APIs

2014-10-31 Thread Jerin Jacob
On Fri, Oct 31, 2014 at 02:08:47PM +0200, alexandru.badici...@linaro.org wrote: From: Alexandru Badicioiu alexandru.badici...@linaro.org This patch adds a suite for async inplace mode of crypto APIs. Correctness of crypto operations output is verified with known test vectors as well as

Re: [lng-odp] [ODP/PATCH v1] ODP Buffer Segment Support API

2014-10-31 Thread Bill Fischofer
This patch does not apply to the current repository. Is there a revised version available? Thanks. Bill On Wed, Oct 22, 2014 at 8:47 AM, Bill Fischofer bill.fischo...@linaro.org wrote: I had previously detailed some of the problems that arise if we remove segmentation support from buffers

Re: [lng-odp] [PATCH v2 1/1] cunit : add tests for crypto APIs

2014-10-31 Thread Jerin Jacob
On Fri, Oct 31, 2014 at 04:35:21PM +0200, Alexandru Badicioiu wrote: This suite is only for async inplace mode.The tests can be reused, with some modifications, for other modes too (sync/inplace/outplace/). Command line argument to select the the suite(s) or separate test programs could work

[lng-odp] [PATCH] odp_example.c: Fix doxygen warnings

2014-10-31 Thread Mike Holmes
Signed-off-by: Mike Holmes mike.hol...@linaro.org --- example/odp_example/odp_example.c | 5 + 1 file changed, 5 insertions(+) diff --git a/example/odp_example/odp_example.c b/example/odp_example/odp_example.c index 5f25f89..4459cb0 100644 --- a/example/odp_example/odp_example.c +++

Re: [lng-odp] [PATCH] odp_example.c: Fix doxygen warnings

2014-10-31 Thread Bill Fischofer
s/Bareier/Barrier/ On Fri, Oct 31, 2014 at 10:20 AM, Mike Holmes mike.hol...@linaro.org wrote: Signed-off-by: Mike Holmes mike.hol...@linaro.org --- example/odp_example/odp_example.c | 5 + 1 file changed, 5 insertions(+) diff --git a/example/odp_example/odp_example.c

[lng-odp] [PATCH v2] odp_example.c: Fix doxygen warnings

2014-10-31 Thread Mike Holmes
Signed-off-by: Mike Holmes mike.hol...@linaro.org --- Sent version prior to spell check, fix spelling. example/odp_example/odp_example.c | 5 + 1 file changed, 5 insertions(+) diff --git a/example/odp_example/odp_example.c b/example/odp_example/odp_example.c index 5f25f89..4f74916 100644

Re: [lng-odp] [PATCH] odp_example.c: Fix doxygen warnings

2014-10-31 Thread Mike Holmes
Thanks, already saw that :) On 31 October 2014 11:33, Bill Fischofer bill.fischo...@linaro.org wrote: s/Bareier/Barrier/ On Fri, Oct 31, 2014 at 10:20 AM, Mike Holmes mike.hol...@linaro.org wrote: Signed-off-by: Mike Holmes mike.hol...@linaro.org --- example/odp_example/odp_example.c |

Re: [lng-odp] [PATCH 1/4] odp_debug_internal.h: move to include dir from API dir

2014-10-31 Thread Mike Holmes
On 30 October 2014 16:47, Anders Roxell anders.rox...@linaro.org wrote: Signed-off-by: Anders Roxell anders.rox...@linaro.org Reviewed-and-Tested-by:MIke Holmes mike.hol...@linaro.org --- platform/linux-generic/include/{api = }/odp_debug_internal.h | 0 1 file changed, 0 insertions(+), 0

Re: [lng-odp] [PATCH 2/4] configure.ac: accurately name compile option

2014-10-31 Thread Mike Holmes
On 30 October 2014 16:47, Anders Roxell anders.rox...@linaro.org wrote: Signed-off-by: Anders Roxell anders.rox...@linaro.org Reviewed-and-Tested-by:Mike Holmes mike.hol...@linaro.org --- configure.ac | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

Re: [lng-odp] [PATCH 4/4] configure.ac: add configure option to enable ODP_DEBUG

2014-10-31 Thread Mike Holmes
On 30 October 2014 16:47, Anders Roxell anders.rox...@linaro.org wrote: Signed-off-by: Anders Roxell anders.rox...@linaro.org Reviewed-and-Tested-by:Mike Holmes mike.hol...@linaro.org --- configure.ac | 11 +++ 1 file changed, 11 insertions(+) diff --git a/configure.ac

[lng-odp] [Bug 638] linux-dpdk specific header files not installed

2014-10-31 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=638 --- Comment #5 from Ciprian Barbu ciprian.ba...@linaro.org --- The problem that needs to be solved is that currently linux-dpdk does not install odp_pktio_socket.h. I tested this manually with openvswitch_odp, because that's how I spotted this issue.

Re: [lng-odp] [PATCH v2 1/1] cunit : add tests for crypto APIs

2014-10-31 Thread Mike Holmes
I had thought that in this case it might be something like below using the cunit ability to organise tests into different suites. The code can then be organized and reused as needed, but it is all built together as a suite that can be run for async only, sync only or both, with the main entry

[lng-odp] [PATCH 2/2] linux-generic: clean unnecessary includes

2014-10-31 Thread Mike Holmes
With the system headers cleaned up linux-generic no longer needs so many system includes Signed-off-by: Mike Holmes mike.hol...@linaro.org --- platform/linux-generic/odp_buffer.c| 1 - platform/linux-generic/odp_buffer_pool.c | 3 --- platform/linux-generic/odp_coremask.c | 1 -

[lng-odp] [PATCH 1/2] fix including odp_std_types.h

2014-10-31 Thread Mike Holmes
The public ODP API should not directly call Linux system headers Signed-off-by: Mike Holmes mike.hol...@linaro.org --- platform/linux-generic/include/api/odp_byteorder.h | 1 - platform/linux-generic/include/api/odp_debug.h | 3 +-- platform/linux-generic/include/api/odp_std_types.h | 4

[lng-odp] [PATCH v2] add odp_buffer_pool_destroy

2014-10-31 Thread Mike Holmes
Signed-off-by: Mike Holmes mike.hol...@linaro.org --- platform/linux-generic/include/api/odp_buffer_pool.h | 18 ++ platform/linux-generic/odp_buffer_pool.c | 7 +++ 2 files changed, 25 insertions(+) diff --git

Re: [lng-odp] [PATCH OVS v2 3/4] Config option to enable ODP debug

2014-10-31 Thread Anders Roxell
On 2014-10-29 13:24, Zoltan Kiss wrote: Signed-off-by: Zoltan Kiss zoltan.k...@linaro.org --- acinclude.m4 | 31 ++- configure.ac | 1 + 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/acinclude.m4 b/acinclude.m4 index 1579286..5e9f3ee 100644

Re: [lng-odp] [PATCH OVS v2 4/4] netdev, ofpbuf: Adjust OVS implementation to the latest ODP

2014-10-31 Thread Anders Roxell
On 2014-10-29 13:24, Zoltan Kiss wrote: --- lib/netdev-odp.c | 32 +--- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/lib/netdev-odp.c b/lib/netdev-odp.c index 74fbac9..59de46d 100644 --- a/lib/netdev-odp.c +++ b/lib/netdev-odp.c @@ -102,7

Re: [lng-odp] [PATCH 0/4] cleanup debug

2014-10-31 Thread Maxim Uvarov
Please in next time set up proper tag: WARNING: 'Reviewed-and-tested-by:' is the preferred signature form #7: Reviewed-and-Tested-by: Mike Holmes mike.hol...@linaro.org Patch set is merged. Maxim. On 10/30/2014 11:47 PM, Anders Roxell wrote: Hi, This patch set cleans up debug and lets us

[lng-odp] [PATCH] gitignore: ignore test-driver

2014-10-31 Thread Anders Roxell
Signed-off-by: Anders Roxell anders.rox...@linaro.org --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index a721904..57b47ea 100644 --- a/.gitignore +++ b/.gitignore @@ -45,3 +45,4 @@ odp_l2fwd odp_ipsec odp_init doxygen-doc +test-driver -- 2.1.0

[lng-odp] [PATCH v2] add tests for queue

2014-10-31 Thread yan.songming
From: yan.songming yan.songm...@linaro.org Fix spelling problem. Add term_local and term_global. Move cunit queue test to test_odp_queue_base for none syn queue. Signed-off-by: Yan Songming yan.songm...@linaro.org --- test/cunit/Makefile.am | 4 +- test/cunit/odp_queue_test.c | 168

[lng-odp] [PATCH v2] add tests for queue

2014-10-31 Thread yan.songm...@linaro.org
Fix spelling problem. Add term_local and term_global. Move cunit queue test to test_odp_queue_base for none syn queue. Signed-off-by: Yan Songming yan.songm...@linaro.org --- test/cunit/Makefile.am | 4 +- test/cunit/odp_queue_test.c | 168 2

[lng-odp] [PATCH v2] add tests for queue

2014-10-31 Thread yan.songm...@linaro.org
Fix spelling problem. Add term_local and term_global. Move cunit queue test to test_odp_queue_base for none syn queue. Signed-off-by: Yan Songming yan.songm...@linaro.org --- test/cunit/Makefile.am | 4 +- test/cunit/odp_queue_test.c | 168 2