[lng-odp] [PATCH 4/6] test:api_test:update the thread create prototype

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/api_test/odp_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/api_test/odp_common.c b/test/api_test/odp_common.c index 681d915..cebaa12 100644 --- a/test/api_test/odp_comm

[lng-odp] [PATCH 3/6] helper:test:update the prototype for thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- helper/test/odp_thread.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/helper/test/odp_thread.c b/helper/test/odp_thread.c index 592f856..65b5778 100644 --- a/helper/test/odp_thread.c +

[lng-odp] [PATCH 1/6] linux:add thread type in odph_linux_pthread_create

2015-12-14 Thread Hemant Agrawal
The exisiting helper routine only create the worker threads. However there is a need to use the same for creating the control thread as well. e.g. CLI thread. Signed-off-by: Hemant Agrawal --- helper/include/odp/helper/linux.h | 4 +++- helper/linux.c| 31 ++

[lng-odp] [PATCH 6/6] test:validation:common: update the thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/validation/common/odp_cunit_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/common/odp_cunit_common.c b/test/validation/common/odp_cunit_common.c index bc786e

[lng-odp] [PATCH 5/6] test:performance:update the thread create proto

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/performance/odp_atomic.c | 2 +- test/performance/odp_l2fwd.c | 3 ++- test/performance/odp_pktio_perf.c | 4 ++-- test/performance/odp_scheduling.c | 2 +- 4 files changed, 6 insertions(+), 5 de

[lng-odp] [PATCH 2/6] example: update the prototype for thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- example/classifier/odp_classifier.c | 2 +- example/generator/odp_generator.c | 9 ++--- example/ipsec/odp_ipsec.c | 2 +- example/packet/odp_pktio.c | 3 ++- example/timer/odp_timer_t

Re: [lng-odp] [PATCH] api: pktio link status

2015-12-14 Thread Maxim Uvarov
_link_up() might be confused with setting command. Not with getting status command. How about: odp_pktio_status_link_up()? There might be some other statuses for pktio which we can extend later and use the same prefix odp_pktio_status_ Maxim. On 12/11/2015 15:53, Alexandru Badicioiu wro

Re: [lng-odp] [PATCHv5 0/4] TAP pktio

2015-12-14 Thread Maxim Uvarov
Merged, Maxim. On 12/14/2015 10:08, Ilya Maximets wrote: On 11.12.2015 19:51, Stuart Haslam wrote: On Thu, Dec 10, 2015 at 06:32:00PM +0300, Ilya Maximets wrote: Creates a new pktio type that allows for creating and sending/receiving packets through TAP interface. Detailed description in commi

Re: [lng-odp] [PATCH] api: pktio link status

2015-12-14 Thread Alexandru Badicioiu
_link_up() looks consistent with ODP naming convention - setting the link up would be named _link_up_set(). Alex On 14 December 2015 at 10:42, Maxim Uvarov wrote: > > > _link_up() might be confused with setting command. Not with getting status > command. > > How about: > odp_pktio_status_link_u

Re: [lng-odp] [API-NEXT PATCHv2] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Maxim Uvarov
On 12/11/2015 16:22, Bill Fischofer wrote: The linux-generic implementation of odp_time_t makes use of POSIX APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection mechanism so that these dependencies do not "bleed through" the ODP API. This means that ODP applications can be in

Re: [lng-odp] [PATCH] api: pktio link status

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
A specific "is_up()" would not help if the call can fail (return <0). User would still need to check against 1 vs. 0 vs. <0. I think the current specification is logical in the way that "1" means (up and) OK, "0" (down and) try again later, "<0" (failure) no point to try again before something e

Re: [lng-odp] [PATCHv4] doc: userguide: add application programming section

2015-12-14 Thread Maxim Uvarov
Merged, Maxim. On 12/11/2015 21:51, Mike Holmes wrote: On 11 December 2015 at 12:23, Bill Fischofer mailto:bill.fischo...@linaro.org>> wrote: Complete the reformatting to standard asciidoc style, expand the ODP Application Programming section, and include a reorganized and expand

Re: [lng-odp] [API-NEXT/PATCHv1 1/4] api: classification: add class of serivce create api

2015-12-14 Thread Bala Manoharan
Ping. On 8 December 2015 at 15:51, Balasubramanian Manoharan wrote: > class of service create function now takes pool, queue, drop policy and > name as input parameters. > Adds class of service parameter structure odp_cls_cos_param_t and > initialization function odp_cls_cos_param_init() > > Sign

Re: [lng-odp] Does ODP support DPDK flow director feature APIs?

2015-12-14 Thread Krishna Garapati
including odp team. /Krishna On 13 December 2015 at 20:40, Dapeng Liu wrote: > Hi Wenxian, > > I checked the API document but failed to find a exact matching function. > Just share my understanding with you. > > ODP has an event distribution model working as below: > 1) packet classified to COS

[lng-odp] [PATCH 2/2] api: removing the platform_init defination

2015-12-14 Thread Hemant Agrawal
odp_plaform_init defination in standard API will conflict with platform specific defination. better to keep it in the linux-generic or other platform init header file. Signed-off-by: Hemant Agrawal --- include/odp/api/init.h | 12 1 file changed, 12 deletions(-) diff --git a/incl

[lng-odp] [PATCH 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Hemant Agrawal
odp_platform_init structure is platform specific data, it is up to the platform specific implementation to define it. Signed-off-by: Hemant Agrawal --- platform/linux-generic/include/odp/init.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/platform/linux-generic/include/odp/in

Re: [lng-odp] [PATCH 2/2] api: removing the platform_init defination

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
API modifying patches must be targeted to "api-next" branch, and labeled with "API-NEXT PATCH" prefix: [API-NEXT PATCH 2/2] api: init: moved platform_init struct definition Also, you need to leave the doxygen documentation of the type to the public API header file. The typedef code can be mov

Re: [lng-odp] [PATCH 2/2] api: removing the platform_init defination

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
Copy-paste error, @typedef line should have been... /** * @typedef odp_platform_init_t * ODP platform initialization data > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Savolainen, Petri (Nokia - FI/Espoo) > Sent: Monday, December 14

[lng-odp] [PATCHv7] helper : Fix UDP checksum computation

2015-12-14 Thread ion.grigore
From: Grigore Ion This patch fixes the following problems: - checksum computation for LE platforms - checksum is computed in the CPU endianness. The returned result must be converted to the BE ordering when it is used to update the UDP checksum in a packet. - checksum computation for packets havi

Re: [lng-odp] [API-NEXT PATCHv2] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 3:24 AM, Maxim Uvarov wrote: > On 12/11/2015 16:22, Bill Fischofer wrote: > >> The linux-generic implementation of odp_time_t makes use of POSIX >> APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection >> mechanism so that these dependencies do not "blee

[lng-odp] [PATCH] validation: classification: check return value of pktio stop

2015-12-14 Thread Balasubramanian Manoharan
Fixes https://bugs.linaro.org/show_bug.cgi?id=1937 Signed-off-by: Balasubramanian Manoharan --- test/validation/classification/odp_classification_common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/validation/classification/odp_classification_common.c b/test/vali

[lng-odp] [Bug 1937] CID 155811: Error handling issues odp_classification_common.c

2015-12-14 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1937 --- Comment #1 from Bala Manoharan --- V1 submitted: https://patches.linaro.org/58448/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.linar

Re: [lng-odp] [PATCHv6] helper : Fix UDP checksum computation

2015-12-14 Thread Ion Grigore
Hi Ilya, I checked the changes you suggested. It's Ok. I sent a new patch(v7). Tkanks, Grig -Original Message- From: Grigore Ion-B17953 Sent: Friday, December 11, 2015 5:57 PM To: 'Ilya Maximets' ; Maxim Uvarov ; lng-odp@lists.linaro.org Subject: RE: [PATCHv6] helper : Fix UDP checksu

Re: [lng-odp] [PATCH] validation: classification: check return value of pktio stop

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 6:00 AM, Balasubramanian Manoharan < bala.manoha...@linaro.org> wrote: > Fixes https://bugs.linaro.org/show_bug.cgi?id=1937 > > Signed-off-by: Balasubramanian Manoharan > Reviewed-by: Bill Fischofer > --- > test/validation/classification/odp_classification_common.c |

[lng-odp] [API-NEXT PATCHv2 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Hemant Agrawal
odp_platform_init structure is platform specific data, it is up to the platform specific implementation to define it. Signed-off-by: Hemant Agrawal --- platform/linux-generic/include/odp/init.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/platform/linux-generic/include/odp/in

[lng-odp] [API-NEXT PATCHv2 2/2] api:removing-the-platform_init-defination

2015-12-14 Thread Hemant Agrawal
odp_plaform_init defination in standard API will conflict with platform specific defination. better to keep it in the linux-generic or other platform init header file Signed-off-by: Hemant Agrawal --- include/odp/api/init.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/odp/api/in

Re: [lng-odp] [PATCH 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 4:06 AM, Hemant Agrawal wrote: > odp_platform_init structure is platform specific data, it is up > to the platform specific implementation to define it. > > Signed-off-by: Hemant Agrawal > --- > platform/linux-generic/include/odp/init.h | 11 +++ > 1 file changed

Re: [lng-odp] [PATCH 2/2] api: removing the platform_init defination

2015-12-14 Thread Agrawal Hemant
Thanks! Submitted the updated patch Regards, Hemant -Original Message- From: Savolainen, Petri (Nokia - FI/Espoo) [mailto:petri.savolai...@nokia.com] Sent: Monday, December 14, 2015 4:21 PM To: Agrawal Hemant-B10814 ; lng-odp@lists.linaro.org Subject: RE: [lng-odp] [PATCH 2/2] api: rem

Re: [lng-odp] [API-NEXT PATCHv2] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Maxim Uvarov
On 12/14/2015 14:39, Bill Fischofer wrote: On Mon, Dec 14, 2015 at 3:24 AM, Maxim Uvarov > wrote: On 12/11/2015 16:22, Bill Fischofer wrote: The linux-generic implementation of odp_time_t makes use of POSIX APIs that are sensitive to the _P

Re: [lng-odp] [API-NEXT PATCHv2 2/2] api:removing-the-platform_init-defination

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
The mail subject line becomes the git log entry. It should not contain dashes and should define the API (init) modified. So, something like this: [API-NEXT PATCHv2 2/2] api: init: removed platform_init struct definition The @typedef doxygen tag needs to be added. See under. Also try 'make dox

Re: [lng-odp] [API-NEXT PATCHv2 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Hemant Agrawal > Sent: Monday, December 14, 2015 2:34 PM > To: lng-odp@lists.linaro.org > Cc: Hemant Agrawal > Subject: [lng-odp] [API-NEXT PATCHv2 1/2] linux-generic:adding the > platform_in

Re: [lng-odp] [API-NEXT PATCHv2] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 6:40 AM, Maxim Uvarov wrote: > On 12/14/2015 14:39, Bill Fischofer wrote: > >> >> >> On Mon, Dec 14, 2015 at 3:24 AM, Maxim Uvarov > > wrote: >> >> On 12/11/2015 16:22, Bill Fischofer wrote: >> >> The linux-generic implementation

Re: [lng-odp] [PATCH] validation: classification: check return value of pktio stop

2015-12-14 Thread Maxim Uvarov
Merged, Maxim. On 12/14/2015 15:29, Bill Fischofer wrote: On Mon, Dec 14, 2015 at 6:00 AM, Balasubramanian Manoharan mailto:bala.manoha...@linaro.org>> wrote: Fixes https://bugs.linaro.org/show_bug.cgi?id=1937 Signed-off-by: Balasubramanian Manoharan mailto:bala.manoha...@linar

[lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: add the platform_init structure

2015-12-14 Thread Hemant Agrawal
odp_platform_init structure is platform specific data, it is up to the platform specific implementation to define it. Signed-off-by: Hemant Agrawal --- platform/linux-generic/include/odp/init.h | 2 ++ .../linux-generic/include/odp/plat/init_types.h| 35 ++ 2 fi

[lng-odp] [API-NEXT PATCHv3 2/2] api: init: removed platform_init struct definition

2015-12-14 Thread Hemant Agrawal
odp_platform_init defination in standard API will conflict with platform specific defination. better to keep it in the linux-generic or other platform init header file Signed-off-by: Hemant Agrawal --- include/odp/api/init.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/i

Re: [lng-odp] [API-NEXT/PATCHv1 1/4] api: classification: add class of serivce create api

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: EXT Balasubramanian Manoharan [mailto:bala.manoha...@linaro.org] > Sent: Tuesday, December 08, 2015 12:22 PM > To: lng-odp@lists.linaro.org > Cc: Savolainen, Petri (Nokia - FI/Espoo); Balasubramanian Manoharan > Subject: [API-NEXT/PATCHv1 1/4] api: classificat

Re: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: add the platform_init structure

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Hemant Agrawal > Sent: Monday, December 14, 2015 3:06 PM > To: lng-odp@lists.linaro.org > Cc: Hemant Agrawal > Subject: [lng-odp] [API-NEXT PATCHv3 1/2] linux-generic: add the > platform_init

Re: [lng-odp] [API-NEXT PATCHv3 2/2] api: init: removed platform_init struct definition

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
This patch is OK. The other has issues. Reviewed-by: Petri Savolainen > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Hemant Agrawal > Sent: Monday, December 14, 2015 3:06 PM > To: lng-odp@lists.linaro.org > Cc: Hemant Agrawal > Subject:

[lng-odp] [PATCH 1/2] validation: synchro tests split into 3 groups

2015-12-14 Thread Christophe Milard
No functionnal changes: just code reordering to match the ODP modules. Signed-off-by: Christophe Milard --- configure.ac|3 + platform/linux-generic/test/Makefile.am |3 + test/validation/Makefile.am |5 +- test/validation/atomic/.gitignore

[lng-odp] [PATCH 2/2] validation: removing synchronizers tests

2015-12-14 Thread Christophe Milard
Now redundant with atomic, barrier and lock tests. Signed-off-by: Christophe Milard --- configure.ac |1 - platform/linux-generic/test/Makefile.am|1 - test/validation/Makefile.am|1 - test/validation/synchroni

Re: [lng-odp] [PATCH 1/6] linux:add thread type in odph_linux_pthread_create

2015-12-14 Thread Zoltan Kiss
On 14/12/15 07:55, Hemant Agrawal wrote: The exisiting helper routine only create the worker threads. However there is a need to use the same for creating the control thread as well. e.g. CLI thread. Signed-off-by: Hemant Agrawal --- helper/include/odp/helper/linux.h | 4 +++- helper/lin

[lng-odp] [PATCH] linux-generic: fix tap compilation

2015-12-14 Thread Maxim Uvarov
pktio/tap.c:267:21: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] } else if (retval != pkt_len) { Cc: Ilya Maximets Signed-off-by: Maxim Uvarov --- platform/linux-generic/pktio/tap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [lng-odp] [PATCH] helper: hashtable: include only needed odp headers

2015-12-14 Thread Ivan Khoronzhuk
ping On 03.12.15 13:27, Ivan Khoronzhuk wrote: No need to include all headers in hashtable and lineartable. rwlock and shared_memory is enough. It allows to see which odp parts are used. Signed-off-by: Ivan Khoronzhuk --- helper/hashtable.c | 3 ++- helper/lineartable.c | 3 ++- 2 files

Re: [lng-odp] [PATCH] linux-generic: fix tap compilation

2015-12-14 Thread Ilya Maximets
Thanks. Reviewed-by: Ilya Maximets On 14.12.2015 18:26, Maxim Uvarov wrote: > pktio/tap.c:267:21: error: comparison between signed and > unsigned integer expressions [-Werror=sign-compare] >} else if (retval != pkt_len) { > > Cc: Ilya Maximets > Signed-off-by: Maxim Uvarov > --- > platfo

Re: [lng-odp] [PATCH] example: timer: free resources while termination

2015-12-14 Thread Ivan Khoronzhuk
ping On 19.11.15 12:54, Ivan Khoronzhuk wrote: Example should free resources in right order when terminates. Also it should have correct error path. Signed-off-by: Ivan Khoronzhuk --- example/timer/odp_timer_test.c | 46 ++ 1 file changed, 38 insertio

Re: [lng-odp] [PATCH] linux-generic: init: return -1 when system_info is failed

2015-12-14 Thread Ivan Khoronzhuk
ping On 03.12.15 14:58, Ivan Khoronzhuk wrote: Signed-off-by: Ivan Khoronzhuk --- platform/linux-generic/odp_init.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/platform/linux-generic/odp_init.c b/platform/linux-generic/odp_init.c index 5e19d86..087bfdf 100644 ---

Re: [lng-odp] [PATCH] linux-generic: fix tap compilation

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 9:26 AM, Maxim Uvarov wrote: > pktio/tap.c:267:21: error: comparison between signed and > unsigned integer expressions [-Werror=sign-compare] >} else if (retval != pkt_len) { > > Cc: Ilya Maximets > Signed-off-by: Maxim Uvarov > Reviewed-by: Bill Fischofer > ---

Re: [lng-odp] [PATCH] linux-generic: fix tap compilation

2015-12-14 Thread Maxim Uvarov
Merged, Maxim. On 12/14/2015 18:31, Bill Fischofer wrote: On Mon, Dec 14, 2015 at 9:26 AM, Maxim Uvarov > wrote: pktio/tap.c:267:21: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] } else if (retval != pk

Re: [lng-odp] Does ODP support DPDK flow director feature APIs?

2015-12-14 Thread Mike Holmes
CCing ODP-DPDK list On 14 December 2015 at 05:00, Krishna Garapati < balakrishna.garap...@linaro.org> wrote: > including odp team. > > /Krishna > > On 13 December 2015 at 20:40, Dapeng Liu wrote: > >> Hi Wenxian, >> >> I checked the API document but failed to find a exact matching function. >>

Re: [lng-odp] [API-NEXT PATCH v5 2/7] api: pktio: added multiple pktio input queues

2015-12-14 Thread Zoltan Kiss
Hi, I wanted to raise some questions about this, unfortunately it fell off the radar for a while. On 26/11/15 08:35, Petri Savolainen wrote: Added input queue configuration parameters and functions to setup multiple input queue and hashing. Added also functions to query the number of queues a

[lng-odp] [API-NEXT PATCH 5/6] validation: time: add test for odp_time_local_rate() and use resolution

2015-12-14 Thread Ivan Khoronzhuk
This function is used to get resolution, that test uses to check other APIs. Signed-off-by: Ivan Khoronzhuk --- test/validation/time/time.c | 46 ++--- test/validation/time/time.h | 1 + 2 files changed, 32 insertions(+), 15 deletions(-) diff --git a/tes

[lng-odp] [API-NEXT PATCH 6/6] validation: time: add test for odp_time_delay()

2015-12-14 Thread Ivan Khoronzhuk
This patch adds validation test for odp_time_delay(). Also it visually demonstrates 5 second count. Signed-off-by: Ivan Khoronzhuk --- test/validation/time/time.c | 26 ++ test/validation/time/time.h | 1 + 2 files changed, 27 insertions(+) diff --git a/test/validation/

[lng-odp] [API-NEXT PATCH 1/6] api: time: add rate and delay API calls

2015-12-14 Thread Ivan Khoronzhuk
This patch adds odp_time_local_rate() and odp_time_delay() APIs. Signed-off-by: Ivan Khoronzhuk --- include/odp/api/time.h| 12 platform/linux-generic/odp_time.c | 21 + 2 files changed, 33 insertions(+) diff --git a/include/odp/api/time.h b/include/

[lng-odp] [API-NEXT PATCH 2/6] performance: pktio_perf: use odp_time_delay() function instead of looping

2015-12-14 Thread Ivan Khoronzhuk
Now odp_time_delay() can be used instead of own implementation. Signed-off-by: Ivan Khoronzhuk --- test/performance/odp_pktio_perf.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/test/performance/odp_pktio_perf.c b/test/performance/odp_pktio_perf.c index

[lng-odp] [API-NEXT PATCH 4/6] validation: time: add test convertsion on 0

2015-12-14 Thread Ivan Khoronzhuk
Signed-off-by: Ivan Khoronzhuk --- test/validation/time/time.c | 4 1 file changed, 4 insertions(+) diff --git a/test/validation/time/time.c b/test/validation/time/time.c index 4040b9f..cef 100644 --- a/test/validation/time/time.c +++ b/test/validation/time/time.c @@ -50,6 +50,10 @@ voi

[lng-odp] [API-NEXT PATCH 3/6] validation: time: test time constants in ns

2015-12-14 Thread Ivan Khoronzhuk
Signed-off-by: Ivan Khoronzhuk --- test/validation/time/time.c | 10 ++ test/validation/time/time.h | 1 + 2 files changed, 11 insertions(+) diff --git a/test/validation/time/time.c b/test/validation/time/time.c index 50d9bdc..4040b9f 100644 --- a/test/validation/time/time.c +++ b/test/

[lng-odp] [API-NEXT PATCH 0/6] api: time: add rate and delay APIs

2015-12-14 Thread Ivan Khoronzhuk
This series adds odp_time_local_rate() and odp_time_delay() calls to time API and it's validation tests and usage. Ivan Khoronzhuk (6): api: time: add rate and delay API calls performance: pktio_perf: use odp_time_delay() function instead of looping validation: time: test time constants

Re: [lng-odp] [PATCH] helper: hashtable: include only needed odp headers

2015-12-14 Thread Mike Holmes
On 3 December 2015 at 06:27, Ivan Khoronzhuk wrote: > No need to include all headers in hashtable and lineartable. > rwlock and shared_memory is enough. It allows to see which odp > parts are used. > > To make all of the helpers lib clean of the general odp.h we also need to fix include/odp/help

Re: [lng-odp] [PATCH] helper: hashtable: include only needed odp headers

2015-12-14 Thread Ivan Khoronzhuk
On 14.12.15 21:15, Mike Holmes wrote: On 3 December 2015 at 06:27, Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>> wrote: No need to include all headers in hashtable and lineartable. rwlock and shared_memory is enough. It allows to see which odp parts are used. To make all

Re: [lng-odp] [API-NEXT PATCH v2] helper: add cuckoo hash implementation

2015-12-14 Thread Mike Holmes
Added to Tuesday agenda On 12 December 2015 at 22:18, HePeng wrote: > Ping. > > So what is your decision on this. > > 在 2015年12月10日,下午1:06,HePeng 写道: > > > 在 2015年12月9日,下午6:49,Ola Liljedahl 写道: > > On 9 December 2015 at 06:31, HePeng wrote: > >> >> 在 2015年12月8日,下午9:34,Ola Liljedahl 写道: >> >>

[lng-odp] [API-NEXT PATCHv3] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Bill Fischofer
The linux-generic implementation of odp_time_t makes use of POSIX APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection mechanism so that these dependencies do not "bleed through" the ODP API. This means that ODP applications can be independent of _POSIX_C_SOURCE level. Signed-o

Re: [lng-odp] [API-NEXT PATCHv3] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Bill Fischofer
v3 posted to fix doxygen issues. On Mon, Dec 14, 2015 at 4:14 PM, Bill Fischofer wrote: > The linux-generic implementation of odp_time_t makes use of POSIX > APIs that are sensitive to the _POSIX_C_SOURCE level. Use an indirection > mechanism so that these dependencies do not "bleed through" the

Re: [lng-odp] [PATCH] doc/users-guide: add helpers section

2015-12-14 Thread Mike Holmes
On 12 December 2015 at 10:13, Bill Fischofer wrote: > > > On Fri, Dec 11, 2015 at 1:52 PM, Mike Holmes > wrote: > >> Signed-off-by: Mike Holmes >> --- >> doc/users-guide/users-guide.adoc | 161 >> +++ >> 1 file changed, 161 insertions(+) >> >> diff --git a/d

Re: [lng-odp] [PATCH] doc/users-guide: add helpers section

2015-12-14 Thread Bill Fischofer
On Mon, Dec 14, 2015 at 4:28 PM, Mike Holmes wrote: > > > On 12 December 2015 at 10:13, Bill Fischofer > wrote: > >> >> >> On Fri, Dec 11, 2015 at 1:52 PM, Mike Holmes >> wrote: >> >>> Signed-off-by: Mike Holmes >>> --- >>> doc/users-guide/users-guide.adoc | 161 >>> ++

Re: [lng-odp] [PATCH] doc/users-guide: add helpers section

2015-12-14 Thread Mike Holmes
On 14 December 2015 at 17:36, Bill Fischofer wrote: > > On Mon, Dec 14, 2015 at 4:28 PM, Mike Holmes > wrote: > >> >> >> On 12 December 2015 at 10:13, Bill Fischofer >> wrote: >> >>> >>> >>> On Fri, Dec 11, 2015 at 1:52 PM, Mike Holmes >>> wrote: >>> Signed-off-by: Mike Holmes --- >

Re: [lng-odp] [API-NEXT PATCHv3] linux-generic: time: remove posix bleed through on odp_time_t

2015-12-14 Thread Maxim Uvarov
On 12/15/2015 01:15, Bill Fischofer wrote: v3 posted to fix doxygen issues. On Mon, Dec 14, 2015 at 4:14 PM, Bill Fischofer mailto:bill.fischo...@linaro.org>> wrote: The linux-generic implementation of odp_time_t makes use of POSIX APIs that are sensitive to the _POSIX_C_SOURCE level.

[lng-odp] [API-NEXT PATCHv4 2/2] api: init: removed platform_init struct definition

2015-12-14 Thread Hemant Agrawal
odp_platform_init defination in standard API will conflict with platform specific defination. better to keep it in the linux-generic or other platform init header file Signed-off-by: Hemant Agrawal --- include/odp/api/init.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/i

[lng-odp] [API-NEXT PATCHv4 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Hemant Agrawal
odp_platform_init structure is platform specific data, it is up to the platform specific implementation to define it. Signed-off-by: Hemant Agrawal --- platform/linux-generic/Makefile.am | 1 + platform/linux-generic/include/odp/init.h | 2 ++ .../linux-generic/include

[lng-odp] [PATCH v2 1/6] linux:add thread type in odph_linux_pthread_create

2015-12-14 Thread Hemant Agrawal
The exisiting helper routine only create the worker threads. However there is a need to use the same for creating the control thread as well. e.g. CLI thread. Signed-off-by: Hemant Agrawal --- helper/include/odp/helper/linux.h | 5 - helper/linux.c| 6 -- 2 files cha

[lng-odp] [PATCH v2 2/6] example: update the prototype for thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- example/classifier/odp_classifier.c | 2 +- example/generator/odp_generator.c | 9 ++--- example/ipsec/odp_ipsec.c | 2 +- example/packet/odp_pktio.c | 3 ++- example/timer/odp_timer_t

[lng-odp] [PATCH v2 4/6] test:api_test:update the thread create prototype

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/api_test/odp_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/api_test/odp_common.c b/test/api_test/odp_common.c index 681d915..cebaa12 100644 --- a/test/api_test/odp_comm

[lng-odp] [PATCH v2 3/6] helper:test:update the prototype for thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- helper/test/odp_thread.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/helper/test/odp_thread.c b/helper/test/odp_thread.c index 592f856..65b5778 100644 --- a/helper/test/odp_thread.c +

[lng-odp] [PATCH v2 6/6] test:validation:common: update the thread create

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/validation/common/odp_cunit_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/common/odp_cunit_common.c b/test/validation/common/odp_cunit_common.c index bc786e

[lng-odp] [PATCH v2 5/6] test:performance:update the thread create proto

2015-12-14 Thread Hemant Agrawal
Updating the prototype for odph_linux_pthread_create Signed-off-by: Hemant Agrawal --- test/performance/odp_atomic.c | 2 +- test/performance/odp_l2fwd.c | 3 ++- test/performance/odp_pktio_perf.c | 4 ++-- test/performance/odp_scheduling.c | 2 +- 4 files changed, 6 insertions(+), 5 de

Re: [lng-odp] [API-NEXT PATCHv4 1/2] linux-generic:adding the platform_init structure

2015-12-14 Thread Savolainen, Petri (Nokia - FI/Espoo)
For both patches: Reviewed-by: Petri Savolainen > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT > Hemant Agrawal > Sent: Tuesday, December 15, 2015 7:40 AM > To: lng-odp@lists.linaro.org > Cc: Hemant Agrawal > Subject: [lng-odp] [API-NEXT