Re: [lng-odp] [PATCH] doc: userguide: add portability and usage info for odp time apis

On Thu, Feb 16, 2017 at 3:20 PM, Brian Brooks wrote: > On Mon, Feb 13, 2017 at 4:47 PM, Bill Fischofer > wrote: >> Clarify and expand on portability and performance considerations >> regarding the use of the ODP time APIs in fulfillment of JIRA

Re: [lng-odp] [PATCH 5/5] travis: check odp license agreement

On 15 February 2017 at 14:19, Maxim Uvarov wrote: > On 02/15/17 20:49, Mike Holmes wrote: > > > > > > On 14 February 2017 at 15:32, Maxim Uvarov > > wrote: > > > > Signed-off-by: Maxim Uvarov

Re: [lng-odp] [PATCH 5/5] travis: check odp license agreement

On 16 February 2017 at 15:49, Mike Holmes wrote: > > > On 15 February 2017 at 14:19, Maxim Uvarov > wrote: > >> On 02/15/17 20:49, Mike Holmes wrote: >> > >> > >> > On 14 February 2017 at 15:32, Maxim Uvarov > >

[lng-odp] [PATCH v2 1/3] linux-gen: dpdk: improve pmd driver linking

Previously each dpdk pmd driver had to be individually referred in the odp code to ensure proper gcc constructor linking. Using the -—whole-archive option when linking the drivers removes this need. After this patch new dpdk pmd drivers are automatically linked. Signed-off-by: Matias Elo

[lng-odp] [Bug 2494] Extend odp_pktio_capability_t to include minimal pool size required by pktio implementation.

https://bugs.linaro.org/show_bug.cgi?id=2494 Bala Manoharan changed: What|Removed |Added Status|IN_PROGRESS |RESOLVED

Re: [lng-odp] [EXT] Re: ODP crypto support with HW offload

Hello, As part of action items from previous meeting I am going to suggest crypto APIs with burst support. But before I would like to confirm my understanding of existing odp_crypto_operation() API definition int odp_crypto_operation(odp_crypto_op_params_t *param,

Re: [lng-odp] [PATCH 3/5] travis: catch doxygen errors

On 15 February 2017 at 14:29, Maxim Uvarov wrote: > On 02/15/17 20:56, Mike Holmes wrote: > > > > > > On 14 February 2017 at 15:32, Maxim Uvarov > > wrote: > > > > make doxygen-doc does not return negative

[lng-odp] [PATCH v2 2/3] linux-gen: dpdk: disable pci devices when running make check

Disable pci devices during DPDK pktio validation test. If the system has pci devices mapped to DPDK pmd drivers they may overlap with the test vdev device indices, which will cause the validation test to fail. Signed-off-by: Matias Elo ---

[lng-odp] [Bug 2889] New: Unused helper extension code does not build, and is not tested in CI

https://bugs.linaro.org/show_bug.cgi?id=2889 Bug ID: 2889 Summary: Unused helper extension code does not build, and is not tested in CI Product: OpenDataPlane - linux- generic reference Version: v1.13.0.0 Hardware: Other

Re: [lng-odp] [PATCH 3/5] travis: catch doxygen errors

On 02/16/17 16:59, Mike Holmes wrote: > > > On 15 February 2017 at 14:29, Maxim Uvarov > wrote: > > On 02/15/17 20:56, Mike Holmes wrote: > > > > > > On 14 February 2017 at 15:32, Maxim Uvarov

Re: [lng-odp] [API-NEXT PATCH v2 3/4] Revert "linux-generic: packet: implement reference apis"

I'm going to explicitly NAK this part just in case there was any confusion. If we have a better replacement for the current packet reference implementation then it can be posted the same way we've revised the implementation of other APIs. Until then this is an unnecessary step backwards. Parts 1

[lng-odp] [Bug 2254] check-odp: valgrind generates "No rule to make target"

https://bugs.linaro.org/show_bug.cgi?id=2254 Mike Holmes changed: What|Removed |Added Version|v1.10 |v1.13.0.0 -- You are

[lng-odp] [PATCH v2 3/3] linux-gen: dpdk: bump target dpdk version to 17.02

Signed-off-by: Matias Elo --- DEPENDENCIES | 2 +- scripts/build-pktio-dpdk | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/DEPENDENCIES b/DEPENDENCIES index 574859c..b97a7dd 100644 --- a/DEPENDENCIES +++ b/DEPENDENCIES @@ -165,7 +165,7 @@

Re: [lng-odp] [EXT] Re: ODP crypto support with HW offload

On 16 February 2017 at 13:31, Dmitri Epshtein wrote: > Hello, > > As part of action items from previous meeting I am going to suggest crypto > APIs with burst support. > > But before I would like to confirm my understanding of existing > odp_crypto_operation() API definition >

[lng-odp] [PATCH] linux-gen: abi: add symlink to arch specific dir

Added symbolic link to the installed include directory. The link points to the arch specific abi dir, so that application does not need to add the arch specific dir into its makefile. Signed-off-by: Petri Savolainen --- configure.ac | 2 ++

[lng-odp] [Bug 2254] check-odp: valgrind generates "No rule to make target"

https://bugs.linaro.org/show_bug.cgi?id=2254 --- Comment #2 from Mike Holmes --- New issues arise in 1.13.0.0 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.

[lng-odp] [PATCH v3] validation: A wrapper script to run test isolated.

The wrapper script; odp_run_app_isolated.sh can be used to run ODP testcases in a isolated environment. Background noise can also be generated. Signed-off-by: Ravineet Singh v2: moved task-isolation dir to odp/scripts, requested my Maxim Uvarov v3: fixed checkpatch.pl

Re: [lng-odp] Why do not odp use a multi-process model?

Hi, I'd like to complement Christophe's answers. Today, Linaro produces an ***API*** and a reference implementation so that silicon vendors can produce high performance, production grade ODP on their platform. So I would encourage you to use the version that is compatible with your hardware

Re: [lng-odp] [PATCH 1/2] helper: linux: renamed threads_extn to linux helpers

> -Original Message- > From: Savolainen, Petri (Nokia - FI/Espoo) > Sent: Monday, February 13, 2017 5:10 PM > To: 'Mike Holmes' > Cc: lng-odp > Subject: RE: [lng-odp] [PATCH 1/2] helper: linux: renamed threads_extn to > linux helpers >

Re: [lng-odp] [PATCH v3] validation: A wrapper script to run test isolated.

On Thu, Feb 16, 2017 at 09:52:37AM +0100, Ravineet Singh wrote: > The wrapper script; odp_run_app_isolated.sh can be used to run ODP > testcases in a isolated environment. Background noise can also be > generated. > > Signed-off-by: Ravineet Singh > > v2: moved

[lng-odp] [Bug 2779] CID 173342: Error handling issues (CHECKED_RETURN)

https://bugs.linaro.org/show_bug.cgi?id=2779 Mike Holmes changed: What|Removed |Added CC||mike.hol...@linaro.org

Re: [lng-odp] [API-NEXT PATCH v2 3/4] Revert "linux-generic: packet: implement reference apis"

> -Original Message- > From: Bill Fischofer [mailto:bill.fischo...@linaro.org] > Sent: Thursday, February 16, 2017 3:00 PM > To: Savolainen, Petri (Nokia - FI/Espoo) labs.com> > Cc: Maxim Uvarov ; lng-odp@lists.linaro.org >

Re: [lng-odp] [PATCH 1/5] travis: switch to ubuntu trusty

On 14 February 2017 at 15:31, Maxim Uvarov wrote: > ubuntu trusty has updated doxygen 1.8.4 which is the same > stable version 1.8 which is described in DEPENDANCIES file. > Now doxygen do not generate any warnings. Compiler version > stays the same 4.8 for gcc. > >

Re: [lng-odp] [PATCH 4/5] travis: do not post codecov tests for not official repo

On 14 February 2017 at 15:32, Maxim Uvarov wrote: > Github may allow administrators in Linaro github group post > codecov from their local repositories. That happens if > same token is used for personal git and Linaro group. There is > one small side affect that history

Re: [lng-odp] Data corruption during TCP download

PS: sorry for the longitude of the email, I tried to be as detailed as possible given the weirdness of the problem... -- Oriol Arcas Software Engineer Starflow Networks On Thu, Feb 16, 2017 at 4:37 PM, Oriol Arcas wrote: > Hi, > > We have been using ODP for a while,

[lng-odp] [Bug 2670] ./configure --disable-abi-compat fails make distcheck

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

Re: [lng-odp] [PATCH] travis: mirror cunit on github

On 15 February 2017 at 12:40, Maxim Uvarov wrote: > because of cunit fails to download from sourceforge.net > it's reasonable place source to github. > > Signed-off-by: Maxim Uvarov > Reviewed-by: Mike Holmes > --- >

Re: [lng-odp] [PATCH 2/5] travis: clang 3.8

On 14 February 2017 at 15:31, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > Reviewed-by: Mike Holmes > --- > .travis.yml | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/.travis.yml

[lng-odp] [Bug 2875] performance/odp_crypto segfaults

https://bugs.linaro.org/show_bug.cgi?id=2875 Christophe Milard changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[lng-odp] [Bug 2797] scheduler_main runs for long time

https://bugs.linaro.org/show_bug.cgi?id=2797 Mike Holmes changed: What|Removed |Added CC||mike.hol...@linaro.org

[lng-odp] Data corruption during TCP download

Hi, We have been using ODP for a while, and we found this weird bug which is difficult to explain, involving data corruption in TCP transfers, I hope somebody may reproduce and shed some light on this. To reproduce this bug, we set up the following environment: 1- Two Debian Jessie VMs, running

Re: [lng-odp] [API-NEXT PATCH v2 3/4] Revert "linux-generic: packet: implement reference apis"

The packet reference code is already compact. The reason there are many lines is that we have many packet APIs and each of them must be prepared to accept references as well as non-references as input. If you want to be able to do zero-copy references (which we do) most of these changes are needed

[lng-odp] [API-NEXT PATCH] linux-generic: packet: improve packet init efficiency

Atomic operations are relatively expensive because they involve implicit memory synchronizations and pipeline flushes. During packet_init() processing we know that there are no other references to this packet, so avoid this overhead via a controlled breach of the atomic type to set the initial

[lng-odp] [API-NEXT PATCHv2] linux-generic: packet: minimize use of atomic ops on free paths

While reference counts are implemented as atomics, try to avoid unnecessary atomic decrement operations in free paths. This streamlines free processing in the expected case where packets do not contain shared segments. Signed-off-by: Bill Fischofer --- Changes for v2 -

[lng-odp] [PATCHv2] configure: libatomic check

upcoming patch ip fragmentation example fails to with gcc 4.8.4 on linking __atomic_compare_exchange_16 functions on x86. For some version of gcc both -latomic and -mcx16 needs to be provided in that case. For clang only -mcx16. That options are set internally for platform but do not set for

Re: [lng-odp] [PATCH] linux-gen: abi: add symlink to arch specific dir

that also fixes compilation for apps against installed odp, right? Maxim. On 02/16/17 14:48, Petri Savolainen wrote: > Added symbolic link to the installed include directory. The link > points to the arch specific abi dir, so that application does > not need to add the arch specific dir into its

Re: [lng-odp] [API-NEXT PATCH v2 3/4] Revert "linux-generic: packet: implement reference apis"

Continuing my efficiency review, I've posted patch http://patches.opendataplane.org/patch/8145/ that should help on the packet free side. If my theory is correct that adding atomic operations severely distorts these microbenchmarks because of its effect on pipelining, this second patch avoids

[lng-odp] [API-NEXT PATCH] linux-generic: packet: minimize use of atomic ops on free paths

While reference counts are implemented as atomics, try to avoid unnecessary atomic decrement operations in free paths. This streamlines free processing in the expected case where packets do not contain shared segments. Signed-off-by: Bill Fischofer ---

Re: [lng-odp] [API-NEXT PATCHv4 0/4] a new API to support timer capability

Petri, please review API patch. Maxim. On 02/08/17 05:37, Kevin Wang wrote: > *** BLURB HERE *** > > Kevin Wang (4): > api: timer: add odp_timer_capability() api > linux-generic: timer: implement odp_timer_capability() > validation/example: call odp_timer_capability() before creating a >

Re: [lng-odp] [API-NEXT PATCHv4 2/4] linux-generic: timer: implement odp_timer_capability()

why not clock_getres() ? Maxim. On 02/08/17 05:37, Kevin Wang wrote: > Implement a new internal function timer_res_init() to detect the max > timer resolution without overrun at the ODP init stage. It will check > timer resolution from 1ms to 100us, 10us...1ns until the timer is > overrun. > >

[lng-odp] [Linaro/odp] 622103: travis: mirror cunit on github

Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: 622103adf9b6d8e73e86c751469ee2523f4dd952 https://github.com/Linaro/odp/commit/622103adf9b6d8e73e86c751469ee2523f4dd952 Author: Maxim Uvarov Date: 2017-02-16 (Thu, 16 Feb 2017)

Re: [lng-odp] [PATCH 4/5] travis: do not post codecov tests for not official repo

On 02/16/17 19:11, Mike Holmes wrote: > > > On 14 February 2017 at 15:32, Maxim Uvarov > wrote: > > Github may allow administrators in Linaro github group post > codecov from their local repositories. That happens if > same

Re: [lng-odp] [PATCH 4/5] travis: do not post codecov tests for not official repo

On 16 February 2017 at 14:11, Maxim Uvarov wrote: > On 02/16/17 19:11, Mike Holmes wrote: > > > > > > On 14 February 2017 at 15:32, Maxim Uvarov > > wrote: > > > > Github may allow administrators in Linaro

[lng-odp] [Linaro/odp] af5205: helper: linux: renamed threads_extn to linux helpe...

Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: af5205809e3921ea0c4db5771ea1da5d2f0eaaf5 https://github.com/Linaro/odp/commit/af5205809e3921ea0c4db5771ea1da5d2f0eaaf5 Author: Petri Savolainen Date: 2017-02-16 (Thu, 16 Feb

Re: [lng-odp] [PATCH 1/2] helper: linux: renamed threads_extn to linux helpers

On 02/16/17 11:51, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > >> -Original Message- >> From: Savolainen, Petri (Nokia - FI/Espoo) >> Sent: Monday, February 13, 2017 5:10 PM >> To: 'Mike Holmes' >> Cc: lng-odp >> Subject: RE:

Re: [lng-odp] [PATCH] doc: userguide: add portability and usage info for odp time apis

On Mon, Feb 13, 2017 at 4:47 PM, Bill Fischofer wrote: > Clarify and expand on portability and performance considerations > regarding the use of the ODP time APIs in fulfillment of JIRA > issue https://projects.linaro.org/browse/ODP-575 > > Signed-off-by: Bill Fischofer