Re: [lng-odp] Query on classification's rules ordering

2015-05-07 Thread Sunil Kumar Kori
Hello Bala, Thanks for the clarification of below query. In the continuation, ODP classification document http://www.opendataplane.org/wp-content/uploads/2014/01/ODP_Classifier.docx.pdf describes that precedence is undetermined in case of multiple pmr matched with received frame. See

Re: [lng-odp] [API-NEXT PATCH v3 0/5] Packet IO API additions

2015-05-07 Thread Savolainen, Petri (Nokia - FI/Espoo)
Ping. api: packet_io: added parse mode can be left out for now, others should be OK to merge. -Petri -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of ext Petri Savolainen Sent: Tuesday, May 05, 2015 4:12 PM To: lng-odp@lists.linaro.org

Re: [lng-odp] Setting up odp test in vm guest in Lava lab

2015-05-07 Thread Hongbo Zhang
On 5 May 2015 at 18:49, Ciprian Barbu ciprian.ba...@linaro.org wrote: On Tue, May 5, 2015 at 1:45 PM, Ciprian Barbu ciprian.ba...@linaro.org wrote: Maybe Anders can better reply to this, but I can throw in my two cents. In general it's useful to look at the lng ci bundle stream:

[lng-odp] Invitation: LNG ODP Design Hacking Sprint 2015 - @Cavium @ Mon 2015-06-15 - Fri 2015-06-19 (LNG Global Team Calendar)

2015-05-07 Thread marshall.guill...@linaro.org
BEGIN:VCALENDAR PRODID:-//Google Inc//Google Calendar 70.9054//EN VERSION:2.0 CALSCALE:GREGORIAN METHOD:REQUEST BEGIN:VEVENT DTSTART;VALUE=DATE:20150615 DTEND;VALUE=DATE:20150620 DTSTAMP:20150507T193351Z ORGANIZER;CN=LNG Global Team Calendar:mailto:linaro.org_9fv4qa2pilfc8imvjn1

Re: [lng-odp] squash or not to squash....

2015-05-07 Thread Bill Fischofer
On Thu, May 7, 2015 at 11:50 AM, Maxim Uvarov maxim.uva...@linaro.org wrote: Here I have patches from api-next needed to be merged to master: https://git.linaro.org/people/maxim.uvarov/odp.git/shortlog/refs/heads/api-next There are 4 commits: 5f9b8df example: classifier: remove

[lng-odp] [PATCHv4] linux-generic: packet: Add lazy parsing support

2015-05-07 Thread Bill Fischofer
Lazy parsing defers parsing until the results are actually needed. This allows applications that do their own parsing and never reference ODP parse results to avoid the overhead of SW parsing. Signed-off-by: Bill Fischofer bill.fischo...@linaro.org --- Changes in v4: - Rebase to avoid merge

[lng-odp] [PATCHv5] linux-generic: packet: Add lazy parsing support

2015-05-07 Thread Bill Fischofer
Lazy parsing defers parsing until the results are actually needed. This allows applications that do their own parsing and never reference ODP parse results to avoid the overhead of SW parsing. Signed-off-by: Bill Fischofer bill.fischo...@linaro.org --- Changes in v5: - Streamline implementation

Re: [lng-odp] validation: running the tests from the platform side (again :-) )

2015-05-07 Thread Christophe Milard
On 6 May 2015 at 22:48, Mike Holmes mike.hol...@linaro.org wrote: On 6 May 2015 at 11:26, Christophe Milard christophe.mil...@linaro.org wrote: Hi, Yet another try to reach a acceptable solution regarding the test environment, following tuesday meeting. Here comes the n+2th

Re: [lng-odp] [PATCH 1/3] example: classifier: remove extra local init

2015-05-07 Thread Savolainen, Petri (Nokia - FI/Espoo)
It's only this patch, there won't be a series. -Petri -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of ext Petri Savolainen Sent: Thursday, May 07, 2015 2:34 PM To: lng-odp@lists.linaro.org Subject: [lng-odp] [PATCH 1/3] example: classifier:

Re: [lng-odp] Setting up odp test in vm guest in Lava lab

2015-05-07 Thread Ciprian Barbu
On Thu, May 7, 2015 at 10:43 AM, Hongbo Zhang hongbo.zh...@linaro.org wrote: On 5 May 2015 at 18:49, Ciprian Barbu ciprian.ba...@linaro.org wrote: On Tue, May 5, 2015 at 1:45 PM, Ciprian Barbu ciprian.ba...@linaro.org wrote: Maybe Anders can better reply to this, but I can throw in my two

[lng-odp] [PATCH v2] linux-generic/odp_impl: add implementation version details

2015-05-07 Thread Mike Holmes
Signed-off-by: Mike Holmes mike.hol...@linaro.org --- V2 Reduce the number fo files touched. platform/Makefile.inc | 4 platform/linux-generic/odp_impl.c | 14 -- scripts/git_hash.sh | 9 + 3 files changed, 25 insertions(+), 2 deletions(-)

[lng-odp] [PATCH 1/3] example: classifier: remove extra local init

2015-05-07 Thread Petri Savolainen
Worker threads are created with odph_linux_pthread_create() which calls odp_local_init() before entering the function. Signed-off-by: Petri Savolainen petri.savolai...@nokia.com --- example/classifier/odp_classifier.c | 7 --- 1 file changed, 7 deletions(-) diff --git

Re: [lng-odp] Query regarding sequence number update in IPSEC application

2015-05-07 Thread Jacob, Jerin
I agree with Ola here on preserving the ingress order. However, I have experienced same performance issue as Nikhil pointed out (atomic queues have too much overhead for short critical section) I am not sure about any other HW but Cavium has support for introducing the critical section while

Re: [lng-odp] [PATCH 1/1] validation: pktio: fix polling tests

2015-05-07 Thread Maxim Uvarov
Merged, Maxim. On 04/30/2015 17:04, Stuart Haslam wrote: On Thu, Apr 30, 2015 at 11:40:51AM +0300, alexandru.badici...@linaro.org wrote: From: Alexandru Badicioiu alexandru.badici...@linaro.org Some tests poll input queues for packets but the queues are setup as scheduled. Signed-off-by:

Re: [lng-odp] [PATCH] test: pktio: add missing atomic init

2015-05-07 Thread Maxim Uvarov
Merged, Maxim. On 05/06/2015 15:29, Bill Fischofer wrote: On Wed, May 6, 2015 at 4:10 AM, Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu wrote: Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu Reviewed-by: Bill Fischofer

Re: [lng-odp] [PATCH v3] linux-generic: use inttypes defines to print (u)int32_t values

2015-05-07 Thread Maxim Uvarov
Merged, Maxim. On 05/07/2015 16:50, Bill Fischofer wrote: On Wed, May 6, 2015 at 3:04 AM, Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu wrote: Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu Reviewed-by: Bill Fischofer

Re: [lng-odp] [PATCH 1/3] example: classifier: remove extra local init

2015-05-07 Thread Savolainen, Petri (Nokia - FI/Espoo)
I noticed the same and will add that documentation. -Petri From: ext Bala Manoharan [mailto:bala.manoha...@linaro.org] Sent: Thursday, May 07, 2015 3:49 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [PATCH 1/3] example: classifier: remove extra local

Re: [lng-odp] [PATCH v3] linux-generic: use inttypes defines to print (u)int32_t values

2015-05-07 Thread Bill Fischofer
On Wed, May 6, 2015 at 3:04 AM, Nicolas Morey-Chaisemartin nmo...@kalray.eu wrote: Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu Reviewed-by: Bill Fischofer bill.fischo...@linaro.org --- Submitted this patch a couple of weeks ago with a bad title ( [PATCHv2] Use inttypes

Re: [lng-odp] [PATCH] test: use CU_ASSERT_FATAL in case odp_buffer_alloc failed to avoid segfaults

2015-05-07 Thread Maxim Uvarov
Merged, Maxim. On 05/05/2015 14:54, Bill Fischofer wrote: On Tue, May 5, 2015 at 2:24 AM, Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu wrote: Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu mailto:nmo...@kalray.eu Reviewed-by: Bill Fischofer

Re: [lng-odp] squash or not to squash....

2015-05-07 Thread Anders Roxell
On 2015-05-07 16:26, Bill Fischofer wrote: On Thu, May 7, 2015 at 11:50 AM, Maxim Uvarov maxim.uva...@linaro.org wrote: Here I have patches from api-next needed to be merged to master: https://git.linaro.org/people/maxim.uvarov/odp.git/shortlog/refs/heads/api-next There are 4

[lng-odp] NUMA aware memory allocation?

2015-05-07 Thread Gábor Sándor Enyedi
Hi! I just started to test ODP, trying to write my first application, but found a problem: if I want to write NUMA aware code, how should I allocate memory close to a given thread? I mean, I know there is libnuma, but should I use it? I guess not, but I cannot find memory allocation

Re: [lng-odp] Query regarding sequence number update in IPSEC application

2015-05-07 Thread Ola Liljedahl
Using atomic queues will preserve the ingress order when allocating and assigning the sequence number. Also you don't need to use an expensive atomic operation for updating the sequence number as the atomic queue and scheduling will provide mutual exclusion. If the packets that require a sequence

[lng-odp] [Bug 1394] CID 56895: Buffer not null terminated: odp_packet_netmap.c

2015-05-07 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1394 --- Comment #2 from Ciprian Barbu ciprian.ba...@linaro.org --- Patch applied: https://git.linaro.org/lng/odp-netmap.git/commit/cf5a145a1bb40a9ffe3367af8a8a9710062e4a36 -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [Bug 1394] CID 56895: Buffer not null terminated: odp_packet_netmap.c

2015-05-07 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1394 Mike Holmes mike.hol...@linaro.org changed: What|Removed |Added Resolution|--- |FIXED

[lng-odp] [Bug 1449] odp_timer_test core dump

2015-05-07 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1449 --- Comment #1 from Ola Liljedahl ola.liljed...@linaro.org --- Does this error occur all of the time for the specific configuration (e.g. 5 CPU's?) or is this problem intermittent? I was thinking that if the test (it is just the timer example?) is

[lng-odp] [Bug 1375] Packet IO tests pass ODP_PKTIO_INVALID to packet IO API

2015-05-07 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1375 Mike Holmes mike.hol...@linaro.org changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] Reverts in the repo ...

2015-05-07 Thread Ola Liljedahl
On 4 May 2015 at 19:19, Maxim Uvarov maxim.uva...@linaro.org wrote: On 4 May 2015 at 13:40, Savolainen, Petri (Nokia - FI/Espoo) petri.savolai...@nokia.com wrote: -Original Message- From: ext Maxim Uvarov [mailto:maxim.uva...@linaro.org] Sent: Thursday, April 30, 2015 4:24

Re: [lng-odp] NUMA aware memory allocation?

2015-05-07 Thread Zoltan Kiss
Hi, I'm not aware of any such interface, but others with more knowledge can comment about it. The ODP-DPDK implementation creates buffer pools on the NUMA node where the pool create function were actually called. Regards, Zoli On 07/05/15 16:32, Gábor Sándor Enyedi wrote: Hi! I just

[lng-odp] [Bug 1440] Documentation: odp_pktio_open to mention the pool is only used by the default CoS

2015-05-07 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=1440 --- Comment #3 from Bill Fischofer bill.fischo...@linaro.org --- Merged into api-next commit 6df86e37 -- You are receiving this mail because: You are on the CC list for the bug. ___ lng-odp mailing list

Re: [lng-odp] Query regarding sequence number update in IPSEC application

2015-05-07 Thread Ola Liljedahl
Another reason why the IPsec example uses atomic queues is that linux-generic and other ODP implementations thatt use SW scheduling do not support ordered queues... So atomic queues are used instead. I think the example still uses an atomic operation for the sequence number allocation which isn't

[lng-odp] squash or not to squash....

2015-05-07 Thread Maxim Uvarov
Here I have patches from api-next needed to be merged to master: https://git.linaro.org/people/maxim.uvarov/odp.git/shortlog/refs/heads/api-next There are 4 commits: 5f9b8df example: classifier: remove odp_pmr_create_range() support dab82ac validation: remove test case for

Re: [lng-odp] [PATCH 1/3] example: classifier: remove extra local init

2015-05-07 Thread Bala Manoharan
Reviewed-by: Balasubramanian Manoharan bala.manoha...@linaro.org IMO, we can add additional information in odph_linux_pthread_create() header file documentation that this function is expected to call odp_init_local() for the thread it creates. Current documentation only says the following /** *