Re: [lng-odp] [PATCH v4 0/8] First ABI files

2017-02-02 Thread Savolainen, Petri (Nokia - FI/Espoo)
Ping. I have next patch set ready (for packet accessor function inline), just waiting for this to be merged first. -Petri > -Original Message- > From: Bill Fischofer [mailto:bill.fischo...@linaro.org] > Sent: Wednesday, February 01, 2017 3:11 AM > To: Petri Savolainen

[lng-odp] [PATCH] [RFC] API:update odp_schedle_multi to return events from multiple queues.

2017-02-02 Thread Nikhil Agarwal
Signed-off-by: Nikhil Agarwal --- include/odp/api/spec/schedule.h | 36 +++- 1 file changed, 31 insertions(+), 5 deletions(-) diff --git a/include/odp/api/spec/schedule.h b/include/odp/api/spec/schedule.h index f8fed17..6e8d759 100644

[lng-odp] [Bug 2829] CID 168965: Unused value in helper iptablelookup

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2829 Bill Fischofer changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS

[lng-odp] [Bug 2830] CID 168956: Storage leak in cuckoo.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2830 Bill Fischofer changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS

[lng-odp] [PATCH 3/3] helper: tables: avoid invalid odp_shm_addr() calls

2017-02-02 Thread Bill Fischofer
In the various table lookup routines, check that odp_shm_lookup() returns a valid handle before attempting to dereference it via odp_shm_addr(). Signed-off-by: Bill Fischofer --- helper/cuckootable.c | 7 +-- helper/hashtable.c | 7 +--

[lng-odp] [PATCH 2/3] helper: iplookuptable: removed unused variable

2017-02-02 Thread Bill Fischofer
Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2829 by avoiding unused variable. Signed-off-by: Bill Fischofer --- helper/iplookuptable.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/helper/iplookuptable.c b/helper/iplookuptable.c

[lng-odp] [PATCH 1/3] helper: cuckootable: avoid storage leaks on error paths

2017-02-02 Thread Bill Fischofer
Ensure that malloced storage areas are freed on error paths. This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=2830 Signed-off-by: Bill Fischofer --- helper/test/cuckootable.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

[lng-odp] [Bug 2816] Packet reference performance tuning needed

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2816 --- Comment #5 from Bill Fischofer --- See https://lists.linaro.org/pipermail/lng-odp/2017-January/027808.html for a discussion of the observed results of before/after applying this patch. An extract from that post: For

[lng-odp] [Bug 2852] ODP_STATIC_ASSERT() fails when used by C++

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2852 --- Comment #3 from Bill Fischofer --- Patch v4 posted at http://patches.opendataplane.org/patch/8037/ to address issue with older C++ compilers, as reported by Brian Brooks. -- You are receiving this mail because: You are

[lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

2017-02-02 Thread Bill Fischofer
The ODP_STATIC_ASSERT() macro expands to _Static_assert(), however when used in C++ programs this needs to expand to static_assert(). This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=2852 Reported-by: Moshe Tubul Signed-off-by: Bill Fischofer

[lng-odp] [PATCHv3] linux-generic: debug: enable helper use from c++ programs

2017-02-02 Thread Bill Fischofer
The ODP_STATIC_ASSERT() macro expands to _Static_assert(), however when used in C++ programs this needs to expand to static_assert(). This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=2852 Reported-by: Moshe Tubul Signed-off-by: Bill Fischofer

Re: [lng-odp] [PATCH 1/2] codecov: add code coverage intergated to github

2017-02-02 Thread Maxim Uvarov
On 02/02/17 18:40, Mike Holmes wrote: > On 2 February 2017 at 09:41, Maxim Uvarov wrote: >> >> Signed-off-by: Maxim Uvarov >> --- >> .travis.yml | 51 +++ >> 1 file changed, 43 insertions(+), 8

Re: [lng-odp] [PATCH 1/2] codecov: add code coverage intergated to github

2017-02-02 Thread Maxim Uvarov
On 02/02/17 18:48, Mike Holmes wrote: > On 2 February 2017 at 09:41, Maxim Uvarov wrote: >> Signed-off-by: Maxim Uvarov > > Adding a note in the read me on how to find your code coverage results > might help > ok, in general there will be: 1.

[lng-odp] [MONARCH PATCH] example: generator: fix udp checksum computation

2017-02-02 Thread Nicolas Morey-Chaisemartin
odph_ipv4_udp_chksum checks for IPv4 and UDP in packet flags which were not set as the packet parse is triggered when setting the l3 offset Signed-off-by: Nicolas Morey-Chaisemartin --- example/generator/odp_generator.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [lng-odp] [PATCHv2] travis: better comments to generate token in the travis file

2017-02-02 Thread Maxim Uvarov
Merged, Maxim. On 01/19/17 20:18, Christophe Milard wrote: > Signed-off-by: Christophe Milard > --- > .travis.yml | 12 > 1 file changed, 12 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index 03e61b1..cfa6c3d 100644 > --- a/.travis.yml >

Re: [lng-odp] [PATCH] linux-generic: Add ARM odp_cpu_pause()

2017-02-02 Thread Maxim Uvarov
I merged this patch because it does not introduce problem and the same static inlines in all others arches. Separate patch will be needed if we decide to un inline all of them. Maxim. On 02/01/17 21:41, Brian Brooks wrote: > On 02/01 08:38:08, Mike Holmes wrote: >> If this is in the API, then to

Re: [lng-odp] rstp support on odp

2017-02-02 Thread Maxim Uvarov
yes, the point it that classier or pktio should pass BPDUs and drop other packets without cpu load if that possible... On 2 February 2017 at 20:01, Brian Brooks wrote: > On Wed, Feb 1, 2017 at 9:41 AM, Bill Fischofer > wrote: > > On Wed, Feb

[lng-odp] [Bug 2867] New: code coverage odp_sorted_list.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2867 Bug ID: 2867 Summary: code coverage odp_sorted_list.c Product: OpenDataPlane - linux- generic reference Version: v1.13.0.0 Hardware: Other OS: Linux Status: UNCONFIRMED

[lng-odp] [Bug 2866] New: code coverage odp_name_table.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2866 Bug ID: 2866 Summary: code coverage odp_name_table.c Product: OpenDataPlane - linux- generic reference Version: v1.13.0.0 Hardware: Other OS: Linux Status: UNCONFIRMED

Re: [lng-odp] [PATCH v2] example: add IPv4 fragmentation/reassembly example

2017-02-02 Thread Ola Liljedahl
On 02/02/2017, 17:26, "Maxim Uvarov" wrote: >>> >>>My point is: >>> >>>original packet -> wrongly fragmented -> original packet >>>vs >>>original packet -> good fragmented -> original packet >>> >>>result is the same, middle is different. >>I don't understand. How do

Re: [lng-odp] rstp support on odp

2017-02-02 Thread Brian Brooks
On Wed, Feb 1, 2017 at 9:41 AM, Bill Fischofer wrote: > On Wed, Feb 1, 2017 at 9:29 AM, Maxim Uvarov wrote: >> I have some theoretical question to think... >> >> >> rstp on hardware switches supported with moving port into disabled mode >>

Re: [lng-odp] [PATCH v2] example: add IPv4 fragmentation/reassembly example

2017-02-02 Thread Maxim Uvarov
On 02/01/17 18:50, Ola Liljedahl wrote: > On 1 February 2017 at 16:19, Maxim Uvarov wrote: >> On 02/01/17 13:26, Joe Savage wrote: >>> Hey Maxim, >>> >>> I'm adding the mailing list to the CCs. >>> >> >> sorry, looks like I pressed replay instead of replay all. >> >> I

Re: [lng-odp] [PATCH 1/2] codecov: add code coverage intergated to github

2017-02-02 Thread Mike Holmes
On 2 February 2017 at 09:41, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov Adding a note in the read me on how to find your code coverage results might help > --- > .travis.yml | 51 +++ > 1

Re: [lng-odp] [PATCH 1/2] codecov: add code coverage intergated to github

2017-02-02 Thread Mike Holmes
On 2 February 2017 at 09:41, Maxim Uvarov wrote: > > Signed-off-by: Maxim Uvarov > --- > .travis.yml | 51 +++ > 1 file changed, 43 insertions(+), 8 deletions(-) > > diff --git a/.travis.yml

Re: [lng-odp] [PATCHv2] linux-generic: debug: enable helper use from c++ programs

2017-02-02 Thread Brian Brooks
On 02/01 14:25:32, Bill Fischofer wrote: > On Wed, Feb 1, 2017 at 1:14 PM, Brian Brooks wrote: > > On 02/01 08:30:22, Bill Fischofer wrote: > >> The ODP_STATIC_ASSERT() macro expands to _Static_assert(), however when > >> used in C++ programs this needs to expand to

[lng-odp] [Bug 2829] CID 168965: Unused value in helper iptablelookup

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2829 Mike Holmes changed: What|Removed |Added Assignee|lng-odp@lists.linaro.org

[lng-odp] [Bug 2830] CID 168956: Storage leak in cuckoo.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2830 Mike Holmes changed: What|Removed |Added Assignee|lng-odp@lists.linaro.org

[lng-odp] [PATCH] example: add IPv4 fragmentation/reassembly example

2017-02-02 Thread Maxim Uvarov
From: Joe Savage Add an example application implementing lock-free IPv4 fragmentation and reassembly functionality using ODP's packet "concat" and "split". Signed-off-by: Joe Savage Signed-off-by: Maxim Uvarov ---

[lng-odp] [Bug 2856] CID 175038: Memory - corruptions: odp_bench_packet.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2856 Mike Holmes changed: What|Removed |Added Resolution|FIXED |WONTFIX -- You are

Re: [lng-odp] [PATCH] example: add IPv4 fragmentation/reassembly example

2017-02-02 Thread Maxim Uvarov
skip that patch. Series has to be with 2 patches and this patch was in my my dir and git am took it. Maxim.

[lng-odp] [Bug 2856] CID 175038: Memory - corruptions: odp_bench_packet.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2856 Mike Holmes changed: What|Removed |Added Resolution|--- |FIXED

[lng-odp] [Bug 2857] Bus error on not enough hdd space (Ubuntu VM, aarch64)

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2857 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED

[lng-odp] [PATCH 2/2] codecov: add codecov settings

2017-02-02 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov --- .codecov.yml | 21 + 1 file changed, 21 insertions(+) create mode 100644 .codecov.yml diff --git a/.codecov.yml b/.codecov.yml new file mode 100644 index ..eac099cf --- /dev/null +++ b/.codecov.yml @@ -0,0

[lng-odp] [PATCH 1/2] codecov: add code coverage intergated to github

2017-02-02 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov --- .travis.yml | 51 +++ 1 file changed, 43 insertions(+), 8 deletions(-) diff --git a/.travis.yml b/.travis.yml index 03e61b18..9e474da2 100644 --- a/.travis.yml +++ b/.travis.yml @@ -7,21

[lng-odp] [Bug 2861] Remove redundant loop_support parameter in pktio capability

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2861 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED Ever

Re: [lng-odp] [PATCH 0/5] Add ABI compatibility to helper lib

2017-02-02 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: Mike Holmes [mailto:mike.hol...@linaro.org] Sent: Thursday, February 02, 2017 2:39 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCH 0/5] Add ABI compatibility to helper lib On 2 February 2017

Re: [lng-odp] [PATCH 0/5] Add ABI compatibility to helper lib

2017-02-02 Thread Mike Holmes
On 2 February 2017 at 02:59, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > > > -Original Message- > > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Mike > > Holmes > > Sent: Wednesday, February 01, 2017 3:14 PM > > To: lng-odp@lists.linaro.org > > Subject:

[lng-odp] [Bug 2842] Fall back to normal page failure

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2842 Christophe Milard changed: What|Removed |Added Resolution|--- |FIXED

[lng-odp] [Bug 2827] CID 174663: Unchecked return in _ishm.c

2017-02-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2827 Christophe Milard changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] [PATCH 0/5] Add ABI compatibility to helper lib

2017-02-02 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Mike > Holmes > Sent: Wednesday, February 01, 2017 3:14 PM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCH 0/5] Add ABI compatibility to helper lib > > Reusing the mechanism used for