[lng-odp] [API-NEXT 1/2] helper: table: add impl of ip lookup table

This is an implementation of the 16,8,8 ip lookup (longest prefix matching) algorithm. The key of the table is 32-bit IPv4 address. Signed-off-by: Ru Jia --- helper/Makefile.am | 2 + helper/iplookuptable.c | 974

[lng-odp] [API-NEXT 2/2] helper: test: add validation test of ip lookup table

Signed-off-by: Ru Jia --- helper/test/Makefile.am | 4 +- helper/test/iplookuptable.c | 150 2 files changed, 153 insertions(+), 1 deletion(-) create mode 100644 helper/test/iplookuptable.c diff --git a/helper/test/Makefile.am

[lng-odp] [PATCH] doc: users-guide: add packet marking documentation

Updates packet marking api documentation to traffic manager user guide Signed-off-by: Balasubramanian Manoharan --- doc/users-guide/users-guide-tm.adoc | 73 + 1 file changed, 73 insertions(+) diff --git

Re: [lng-odp] [PATCH] doc: user-guide: Improve Crypto section.

I have those changes. Please check if i have missed anything. On 12 May 2016 at 23:32, Bala Manoharan wrote: > Hi Nikhil, > > I had some comments on V1 regarding session type. > If you agree can you please incorporate on next version. > > Regards, > Bala > > On 12 May

Re: [lng-odp] [PATCH] doc: process-guide: convert CONTRIBUTING to asciidoc

Applying this patch shows whitespace errors: bill@Ubuntu15:~/linaro/mikedoc2$ git am ~/Mail/Incoming/Mike/1 Applying: doc: process-guide: convert CONTRIBUTING to asciidoc .git/rebase-apply/patch:129: indent with spaces. `odp_pktio_t` .git/rebase-apply/patch:164: new blank line at EOF. +

Re: [lng-odp] [PATCH] linux-generic: pktio: ipc: remove not needed assignment

On Thu, May 12, 2016 at 1:56 PM, Maxim Uvarov wrote: > Not used assigned, should just removed. > https://bugs.linaro.org/show_bug.cgi?id=2221 > > Signed-off-by: Maxim Uvarov > Reviewed-by: Bill Fischofer > --- >

[lng-odp] [PATCHv4 1/3] doc: images: add packet flow overview image

From: Mike Holmes Signed-off-by: Mike Holmes Signed-off-by: Bill Fischofer --- doc/images/packet_flow.svg | 532 + 1 file changed, 532 insertions(+) create mode 100644

[lng-odp] [PATCHv4 3/3] doc: users-guide: add packet flow overview

From: Mike Holmes Signed-off-by: Mike Holmes Signed-off-by: Bill Fischofer --- doc/users-guide/Makefile.am | 1 + doc/users-guide/users-guide.adoc | 89 +--- 2 files changed,

[lng-odp] [PATCHv4 2/3] doc: users-guide: depend on the images

From: Mike Holmes Signed-off-by: Mike Holmes Signed-off-by: Bill Fischofer --- doc/users-guide/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/users-guide/Makefile.am

Re: [lng-odp] [API-NEXT, RFC, 1/1] api: cpu: performance profiling start/stop

On Thu, May 12, 2016 at 3:40 AM, Yi He wrote: > Hi, Bill and Petri > > After yesterday's discussion/email I realized some strict rules in ARCH: > >1. odp apis do not want to handle thread-to-core arrangement except >only providing availability info, right? (may be

Re: [lng-odp] [PATCHv3 2/2] linux-generic: drop odp_ prefix for internal cpuinfo

Compiling this on a 32-bit system I get: CC odp_system_info.lo odp_system_info.c: In function ‘default_huge_page_size’: odp_system_info.c:86:19: error: format ‘%lu’ expects argument of type ‘long unsigned int *’, but argument 3 has type ‘uint64_t * {aka long long unsigned int *}’

Re: [lng-odp] [PATCH] validation: timer: fix handle unused tmo

On Thu, May 12, 2016 at 2:35 PM, Maxim Uvarov wrote: > Current validation test can produce bug: > timer.c:250:handle_tmo():Wrong tick: expected 18446744073709551615 actual > 149 > FAILED > 1. timer.c:246 - CU_FAIL("Wrong status (stale) for fresh timeout") > 2.

Re: [lng-odp] [PATCH v3 2/2] linux-generic: timer: use strong typing

On Thu, May 12, 2016 at 5:13 AM, Matias Elo wrote: > Use strong typing for odp_timer_t and odp_timeout_t. Fix > couple exposed usage errors. > > Signed-off-by: Matias Elo > Reviewed-and-tested-by: Bill Fischofer > --- >

Re: [lng-odp] [PATCH 4/4] linux-generic: packet: initialize only selected odp_packet_hdr_t members

On Thu, May 12, 2016 at 7:36 AM, Matias Elo wrote: > Using memset to initialize struct odp_packet_hdr_t contents > to 0 has a significant overhead. Instead, initialize only > the required members of the struct. > > Signed-off-by: Matias Elo > --- >

Re: [lng-odp] [PATCH v2] doc: user-guide: Improve crypto section.

On Thu, May 12, 2016 at 10:46 AM, Nikhil Agarwal wrote: > Signed-off-by: Nikhil Agarwal > --- > doc/users-guide/users-guide.adoc | 83 > +++- > 1 file changed, 65 insertions(+), 18 deletions(-) > > diff

[lng-odp] [PATCH] validation: timer: fix handle unused tmo

Current validation test can produce bug: timer.c:250:handle_tmo():Wrong tick: expected 18446744073709551615 actual 149 FAILED 1. timer.c:246 - CU_FAIL("Wrong status (stale) for fresh timeout") 2. timer.c:251 - CU_FAIL("odp_timeout_tick() wrong tick") Which caused with wrong destroy

[lng-odp] [PATCH] doc: process-guide: convert CONTRIBUTING to asciidoc

Converting to asciidoc allows a tidy page to be added to the online documentation without cutting and pasting into wordpress. Being Asccidoc a tiny amount of clutter is added to show code snippets attractively when rendered that make it slightly hard to read as a raw document. Signed-off-by: Mike

Re: [lng-odp] [PATCH] validation: time: remove print and add verbose faults

On Thu, May 12, 2016 at 6:22 AM, Maxim Uvarov wrote: > Having system call inside loop couse unpredictable delay > as the result wrong time diff calculation. Just removing > print make hole test execution shorter on 0.1 seconds > according to cunit stats. Also make

[lng-odp] [PATCH] linux-generic: pktio: ipc: remove not needed assignment

Not used assigned, should just removed. https://bugs.linaro.org/show_bug.cgi?id=2221 Signed-off-by: Maxim Uvarov --- platform/linux-generic/pktio/ipc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/platform/linux-generic/pktio/ipc.c

[lng-odp] [Bug 2222] Coverity: uninitialized variables

https://bugs.linaro.org/show_bug.cgi?id= --- Comment #1 from Maxim Uvarov --- http://patches.opendataplane.org/patch/6004/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing

Re: [lng-odp] [PATCH] validation: pktio: initialize pkt_seq

On Thu, May 12, 2016 at 10:50 AM, Maxim Uvarov wrote: > recv_packets_tmo() has if for not initialized > pkt_seq array, while init is done in create_packets(), > so we need just reorder that lines. > https://bugs.linaro.org/show_bug.cgi?id= > > Signed-off-by: Maxim

Re: [lng-odp] [PATCH] linux-generic:pktio mmap: fix nb_rx overflow

On Thu, May 12, 2016 at 10:24 AM, Maxim Uvarov wrote: > bug: > https://bugs.linaro.org/show_bug.cgi?id=2224 > > Signed-off-by: Maxim Uvarov > Reviewed-by: Bill Fischofer > --- >

Re: [lng-odp] [PATCH] validation: pktio: fix assert for odp_pktin_queue ret code

On Thu, May 12, 2016 at 10:20 AM, Maxim Uvarov wrote: > Fix wrong return code check due to assignment to unsigned variable. > https://bugs.linaro.org/show_bug.cgi?id=2224 > > Signed-off-by: Maxim Uvarov > Reviewed-by: Bill Fischofer

Re: [lng-odp] [PATCH] doc: user-guide: Improve Crypto section.

Good idea Bill, I will add to ARCH call On 12 May 2016 at 14:02, Bala Manoharan wrote: > Hi Nikhil, > > I had some comments on V1 regarding session type. > If you agree can you please incorporate on next version. > > Regards, > Bala > > On 12 May 2016 at 22:22, Nikhil

Re: [lng-odp] [PATCH] doc: user-guide: Improve Crypto section.

Hi Nikhil, I had some comments on V1 regarding session type. If you agree can you please incorporate on next version. Regards, Bala On 12 May 2016 at 22:22, Nikhil Agarwal wrote: > Yes I do agree with mentioning only the API names here. I did that in v2 > of this

Re: [lng-odp] [PATCH 1/2] linux-generic: includes: typo correction

Reviewed-by: Zoltan Kiss On 12/05/16 17:29, Bill Fischofer wrote: Correct typo in spelling of _LARGEFILE64_SOURCE. This also removes an extraneous non-ascii character from the source file. Suggested-by: Zoltan Kiss Signed-off-by: Bill Fischofer

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

On Thu, May 12, 2016 at 8:24 AM, Christophe Milard < christophe.mil...@linaro.org> wrote: > Hi Bill, > Just sent you a v4 containing my proposal regarding the FSMs and the text > around it. > I still have a few comments: > > First, your FSM mixed events and actions on the transition arrow. I

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

This looks good. I've sent a v5 that slightly reformats the images to make them consistent, but I'm happy with the two diagrams if that looks better to you. And I've now just read your other comments. Let me look and see if a v6 is needed to address them :) On Thu, May 12, 2016 at 8:24 AM,

[lng-odp] [PATCHv5 2/2] doc: userguide: add timer and timeout event section to user guide

Signed-off-by: Ivan Khoronzhuk Signed-off-by: Bill Fischofer Signed-off-by: Christophe Milard --- doc/users-guide/Makefile.am| 1 + doc/users-guide/users-guide-timer.adoc | 107

[lng-odp] [PATCHv5 1/2] doc: userguide: add timer/timeout state diagrams

Signed-off-by: Bill Fischofer Signed-off-by: Christophe Milard --- doc/images/.gitignore | 2 ++ doc/images/timeout_fsm.gv | 29 + doc/images/timer_fsm.gv | 24

Re: [lng-odp] [PATCH] doc: user-guide: Improve Crypto section.

Yes I do agree with mentioning only the API names here. I did that in v2 of this patch. On 12 May 2016 at 22:14, Bill Fischofer wrote: > > On Thu, May 12, 2016 at 11:25 AM, Mike Holmes > wrote: > >> >> >> On 11 May 2016 at 19:15, Bill

Re: [lng-odp] [PATCH] doc: user-guide: Improve Crypto section.

On Thu, May 12, 2016 at 11:25 AM, Mike Holmes wrote: > > > On 11 May 2016 at 19:15, Bill Fischofer wrote: > >> A number of grammatical corrections and suggestions for improvement. >> >> On Wed, May 11, 2016 at 9:45 AM, Nikhil Agarwal < >>

[lng-odp] [PATCH 2/2] doc: guides: remove extraneous non-ascii characters from doc files

Suggested-by: Zoltan Kiss Signed-off-by: Bill Fischofer --- doc/application-api-guide/api_guide_lines.dox | 14 ++-- doc/process-guide/bylaws-guide.adoc | 16 +++--- doc/users-guide/users-guide-tm.adoc | 32

[lng-odp] [PATCH 1/2] linux-generic: includes: typo correction

Correct typo in spelling of _LARGEFILE64_SOURCE. This also removes an extraneous non-ascii character from the source file. Suggested-by: Zoltan Kiss Signed-off-by: Bill Fischofer --- platform/linux-generic/include/odp_posix_extensions.h | 2 +-

Re: [lng-odp] [PATCHv6 00/38] running things in process mode

On 4 May 2016 at 20:38, Brian Brooks wrote: > Reviewed-by: Brian Brooks > > for the series. > > On 05/11 18:41:44, Christophe Milard wrote: > > Pull can be performed from the URL at the end of this cover letter > > All patches sent to the list

Re: [lng-odp] Non-ASCII characters in the repo

Good catch. The only legit ones are the tree diagrams. I'll post a patch to fix the rest. A lot of it seems to have come from cut and paste resulting in "smart quotes" being carried over. On Thu, May 12, 2016 at 10:05 AM, Zoltan Kiss wrote: > Hi, > > I've came accross a

Re: [lng-odp] examples howto

On 05/12/16 18:47, Mike Holmes wrote: On 11 May 2016 at 14:08, Sudeepto Roy > wrote: HI, I have been able to successfully install odp on my machine. I wanted to know if you have a any kind of documentation available

[lng-odp] [PATCH] validation: pktio: initialize pkt_seq

recv_packets_tmo() has if for not initialized pkt_seq array, while init is done in create_packets(), so we need just reorder that lines. https://bugs.linaro.org/show_bug.cgi?id= Signed-off-by: Maxim Uvarov --- test/validation/pktio/pktio.c | 10 +- 1 file

Re: [lng-odp] examples howto

On 11 May 2016 at 14:08, Sudeepto Roy wrote: > HI, > > I have been able to successfully install odp on my machine. > > I wanted to know if you have a any kind of documentation available for the > examples directory so that i can understand the use case. > The examples

Re: [lng-odp] OpenDataPlane is now on Wikipedia

I already did On 12 May 2016 at 11:36, Grant Likely wrote: > It's Wikipedia! Go fix it! :-) > > g. > > On Thu, May 12, 2016 at 4:12 PM, Zoltan Kiss > wrote: > > Hi, > > > > The link to Wind River is broken, it should point here: > > > >

Re: [lng-odp] OpenDataPlane is now on Wikipedia

It's Wikipedia! Go fix it! :-) g. On Thu, May 12, 2016 at 4:12 PM, Zoltan Kiss wrote: > Hi, > > The link to Wind River is broken, it should point here: > > https://en.wikipedia.org/wiki/Wind_River_Systems > > On 12/05/16 13:27, Raj Murali wrote: >> >> Dear All, >> >> I

[lng-odp] [Bug 2227] classification_main: Conditional jump or move depends on uninitialised value

https://bugs.linaro.org/show_bug.cgi?id=2227 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS Ever

[lng-odp] [Bug 2224] Coverity: Negative return

https://bugs.linaro.org/show_bug.cgi?id=2224 --- Comment #2 from Maxim Uvarov --- http://patches.opendataplane.org/patch/6002/ http://patches.opendataplane.org/patch/6003/ -- You are receiving this mail because: You are on the CC list for the

[lng-odp] [PATCH] linux-generic:pktio mmap: fix nb_rx overflow

bug: https://bugs.linaro.org/show_bug.cgi?id=2224 Signed-off-by: Maxim Uvarov --- platform/linux-generic/pktio/socket_mmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/pktio/socket_mmap.c

[lng-odp] [PATCH] validation: pktio: fix assert for odp_pktin_queue ret code

Fix wrong return code check due to assignment to unsigned variable. https://bugs.linaro.org/show_bug.cgi?id=2224 Signed-off-by: Maxim Uvarov --- test/validation/pktio/pktio.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [lng-odp] OpenDataPlane is now on Wikipedia

Hi, The link to Wind River is broken, it should point here: https://en.wikipedia.org/wiki/Wind_River_Systems On 12/05/16 13:27, Raj Murali wrote: Dear All, I am happy to inform you all that, we are now on Wikipedia - at last. https://en.wikipedia.org/wiki/Open_Data_Plane_(ODP) Thanks to all

Re: [lng-odp] [PATCHv6 12/38] helper: parsing the complete set of options

On 12 May 2016 at 12:54, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > Those that add new definitions into helper/linux.h and depend on each > other, so that the entire proposed spec is easy to see today and later on > from git history. Since those are new calls, it

[lng-odp] Non-ASCII characters in the repo

Hi, I've came accross a lot in the doc folder, I don't know if we want to do anything about them, but here is an easy way to find them: git grep --color='auto' -P -n "[\x80-\xFF]" There is only one place outside docs: diff --git a/platform/linux-generic/include/odp_posix_extensions.h

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

On 05/12 13:57:52, Christophe Milard wrote: > On 12 May 2016 at 13:33, Savolainen, Petri (Nokia - FI/Espoo) < > petri.savolai...@nokia.com> wrote: > > > > > *From:* lng-odp [mailto:lng-odp-boun...@lists.linaro.org] *On Behalf Of > > *Christophe > > Milard > > *Sent:* Thursday, May 12, 2016 2:14

Re: [lng-odp] Fwd: No huge pages, fall back to normal pages

Hi Maxim, Yes, it is working now. Sorry, my suggestion was wrong. On Thu, May 12, 2016 at 6:32 PM, Maxim Uvarov wrote: > Hi nousi, > > can you please confirm that this patch solves issue: > > http://patches.opendataplane.org/patch/5947/ > > Thank you, > Maxim. > > On

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

Hi Bill, Just sent you a v4 containing my proposal regarding the FSMs and the text around it. I still have a few comments: First, your FSM mixed events and actions on the transition arrow. I could not get graphviz to have 2 colors on the same transition (one for events, one for actions), so I

Re: [lng-odp] Fwd: No huge pages, fall back to normal pages

Hi nousi, can you please confirm that this patch solves issue: http://patches.opendataplane.org/patch/5947/ Thank you, Maxim. On 05/11/16 15:24, nousi wrote: Hi Maxim, plese make correction in patch, check sscanf with "!= 1" instead "== 1'. if (pos) { *(pos - 1) = '\0';

Re: [lng-odp] [PATCH v3 2/2] linux-generic: timer: use strong typing

Bill, this patch was a little bit changes since v2, please review it. Maxim. On 05/12/16 13:13, Matias Elo wrote: Use strong typing for odp_timer_t and odp_timeout_t. Fix couple exposed usage errors. Signed-off-by: Matias Elo --- V2: + Split too long string lines

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

https://bugs.linaro.org/show_bug.cgi?id=2031 Mike Holmes changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS -- You are

[lng-odp] [Bug 2082] Helper documentation is missing

https://bugs.linaro.org/show_bug.cgi?id=2082 Mike Holmes changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] [API-NEXT PATCH v3 1/2] helper: table: add impl of cuckoo hash table

Ping, looks like it wasn't merged for some reason? Any blockers? Maxim. On 04/28/16 10:38, HePeng wrote: Hi, We appreciate more comments on the patch. Thanks. 在 2016年4月27日，下午7:35，Raj Murali > 写道： Agree with Bill. The original

[lng-odp] [Bug 2103] netmap is not tested under CI

https://bugs.linaro.org/show_bug.cgi?id=2103 Mike Holmes changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS

[lng-odp] [Bug 2211] clang fails with 32bit build for odp_timer.c

https://bugs.linaro.org/show_bug.cgi?id=2211 Mike Holmes changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS -- You are

[lng-odp] [Bug 2102] make distcheck does not inherit original options

https://bugs.linaro.org/show_bug.cgi?id=2102 Anders Roxell changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS -- You

[lng-odp] [Bug 2220] Coverity: Unused values in packet test

https://bugs.linaro.org/show_bug.cgi?id=2220 Mike Holmes changed: What|Removed |Added Status|CONFIRMED |IN_PROGRESS -- You are

[lng-odp] [Bug 2235] odp_timer.c: Contains arch specific code

https://bugs.linaro.org/show_bug.cgi?id=2235 Mike Holmes changed: What|Removed |Added Assignee|lng-odp@lists.linaro.org

[lng-odp] [PATCH 3/4] linux-generic: buffer: ifdef ipc_addr_offset member from odp_buffer_hdr_t

Define ipc_addr_offset member of struct odp_buffer_hdr_t only if ipc pktio is enabled to reduce struct size. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_buffer_internal.h | 8 +--- platform/linux-generic/pktio/ipc.c | 14

[lng-odp] [PATCH 4/4] linux-generic: packet: initialize only selected odp_packet_hdr_t members

Using memset to initialize struct odp_packet_hdr_t contents to 0 has a significant overhead. Instead, initialize only the required members of the struct. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_internal.h | 8 +---

[lng-odp] [PATCH 2/4] linux-generic: packet: remove vlan_s_tag and vlan_c_tag members from odp_packet_hdr_t

There is no way to read vlan tags in the ODP API, so don't save them. Signed-off-by: Matias Elo --- platform/linux-generic/include/odp_packet_internal.h | 4 platform/linux-generic/odp_packet.c | 4 2 files changed, 8 deletions(-) diff --git

[lng-odp] OpenDataPlane is now on Wikipedia

Dear All, I am happy to inform you all that, we are now on Wikipedia - at last. https://en.wikipedia.org/wiki/Open_Data_Plane_(ODP) Thanks to all for sharing links and contributing content used in the page. Raj Murali Director - Linaro Networking Group Skype: rajmurali_s │ M: +91 98450 70135

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

On 12 May 2016 at 13:33, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > > > > > *From:* lng-odp [mailto:lng-odp-boun...@lists.linaro.org] *On Behalf Of > *Christophe > Milard > *Sent:* Thursday, May 12, 2016 2:14 PM > *To:* Savolainen, Petri (Nokia - FI/Espoo)

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Christophe Milard Sent: Thursday, May 12, 2016 2:14 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating

[lng-odp] [PATCH] validation: time: remove print and add verbose faults

Having system call inside loop couse unpredictable delay as the result wrong time diff calculation. Just removing print make hole test execution shorter on 0.1 seconds according to cunit stats. Also make verbose errors in places where we out of limit in cunit. This should be very helpful to

Re: [lng-odp] [PATCHv6 12/38] helper: parsing the complete set of options

Those that add new definitions into helper/linux.h and depend on each other, so that the entire proposed spec is easy to see today and later on from git history. Since those are new calls, it should be easy to introduce those in their final form. Patches (e.g. 1 and 12) should not expose

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

> +int odph_odpthreads_create(odph_odpthread_t *thread_tbl, > +const odp_cpumask_t *mask, > +const odph_odpthread_params_t *thr_params) > +{ > + int i; > + int num; > + int cpu_count; > + int cpu; > + > + num =

[lng-odp] [PATCH v2] doc: user-guide: Improve crypto section.

Signed-off-by: Nikhil Agarwal --- doc/users-guide/users-guide.adoc | 83 +++- 1 file changed, 65 insertions(+), 18 deletions(-) diff --git a/doc/users-guide/users-guide.adoc b/doc/users-guide/users-guide.adoc index 0221634..e8d690e

[lng-odp] [PATCH v3 2/2] linux-generic: timer: use strong typing

Use strong typing for odp_timer_t and odp_timeout_t. Fix couple exposed usage errors. Signed-off-by: Matias Elo --- V2: + Split too long string lines (Maxim) example/timer/odp_timer_test.c| 10 ++

[lng-odp] [PATCH v3 1/2] linux-generic: timer: add missing odp_time*_to_u64() functions

Add missing odp_timer_pool_to_u64(), odp_timer_to_u64(), and odp_timeout_to_u64() functions. Use the functions in timer validation tests. Signed-off-by: Matias Elo --- V2: + Reversed patch order (Maxim) platform/linux-generic/odp_timer.c | 15 +++

Re: [lng-odp] [PATCHv6 12/38] helper: parsing the complete set of options

Not sure which patches you want to squash here: "helper: adding a function to merge getopt parameters" and "parsing the complete set of options" ? Aren't these 2 different steps? They were at least tested separately... Or did I misunderstood? Christophe. On 12 May 2016 at 10:21, Savolainen,

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

On 12 May 2016 at 10:17, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > > > > -Original Message- > > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > > Christophe Milard > > Sent: Wednesday, May 11, 2016 7:42 PM > > To:

Re: [lng-odp] [API-NEXT, RFC, 1/1] api: cpu: performance profiling start/stop

Hi, Bill and Petri After yesterday's discussion/email I realized some strict rules in ARCH: 1. odp apis do not want to handle thread-to-core arrangement except only providing availability info, right? (may be repeated several time until I got it :)). 2. then application needs

Re: [lng-odp] [PATCHv6 12/38] helper: parsing the complete set of options

It would be really good to squash these patches introducing (and then modifying the previously introduced) new helper function prototypes into a single patch. This way it's hard to see the complete picture what functions were actually added into helper/linux.h -Petri > -Original

Re: [lng-odp] [PATCH v2 2/2] linux-generic: timer: add missing odp_time*_to_u64() functions

Thanks, I'll fix this and the patch 1 right away. -Matias > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim > Uvarov > Sent: Wednesday, May 11, 2016 7:12 PM > To: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH v2 2/2]

Re: [lng-odp] [PATCHv6 05/38] helpers: linux: creating functions to handle odpthreads

> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Christophe Milard > Sent: Wednesday, May 11, 2016 7:42 PM > To: brian.bro...@linaro.org; mike.hol...@linaro.org; lng- > o...@lists.linaro.org > Subject: [lng-odp] [PATCHv6 05/38] helpers: linux:

Re: [lng-odp] [PATCHv2 1/2] doc: userguide: add timer/timeout state diagram

Hi, I am still confused with this shared FSM for the 2 distinct objects: The red seems to be actions associated with the transitions (and the timer expiration event is no longer shown on the transition showing the actions). I am not sure what the distinction between the green and blue events are.