Re: [lng-odp] [API-NEXT PATCHv7 5/5] linux-gen: drv_drivers: loading modules from config file

2016-12-30 Thread Anders Roxell
On 2016-12-30 10:09, Christophe Milard wrote: > The shared objects listed in the ODP configuration files are > loaded at init time. The odp configuration file list the > shared objects to be loaded as shown in the following example: > module = { > modules = ["enumerator1.so",

Re: [lng-odp] [API-NEXT PATCHv7 3/5] linux-gen: init: adding configuration file parsing

2016-12-30 Thread Anders Roxell
; + } > + } > + if (config_filename) { > + ODP_DBG("Reading configuration file: %s\n", config_filename); > + if (!config_read_file(cf, config_filename)) { > + ODP_ERR("%s:%d - %s\n", > + config_error_file(cf), > + config_error_line(cf), > + config_error_text(cf)); > + config_destroy(cf); > + return(-1); > + } > + } > + > + return 0; > +} > + > int odp_init_global(odp_instance_t *instance, > const odp_init_t *params, > const odp_platform_init_t *platform_params ODP_UNUSED) > @@ -86,6 +153,9 @@ int odp_init_global(odp_instance_t *instance, > odp_global_data.abort_fn = params->abort_fn; > } > > + if (read_configfile()) > + goto init_failed; > + > if (odp_cpumask_init_global(params)) { > ODP_ERR("ODP cpumask init failed.\n"); > goto init_failed; > -- > 2.7.4 > -- Anders Roxell anders.rox...@linaro.org M: +46 709 71 42 85 | IRC: roxell

Re: [lng-odp] [PATCH] enable shared library for non-abi compat mode

2016-12-02 Thread Anders Roxell
On 2 December 2016 at 15:30, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > configure.ac | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index 3e89b0a..57edd33 100644 > --- a/configure.ac > +++

Re: [lng-odp] [PATCHv10 3/3] configure.ac update version numbers

2016-12-01 Thread Anders Roxell
t;On 1 December 2016 at 09:17, Steve McIntyre <steve.mcint...@linaro.org > ><mailto:steve.mcint...@linaro.org>> wrote: > > > >On Thu, Dec 01, 2016 at 04:05:14PM +0300, Maxim Uvarov wrote: > >>On 12/01/16 01:09, Anders Roxell wrote: > >>> Since

Re: [lng-odp] [PATCHv10 3/3] configure.ac update version numbers

2016-11-30 Thread Anders Roxell
On 30 November 2016 at 22:02, Mike Holmes wrote: > I just CC'ed you in Steve. > > My head is spinning but I think we have this straight now, perhaps you have > time to sync with Maxim and possibly Anders if he has time to check this > from ytour Debian background? > > I

Re: [lng-odp] [PATCHv7 3/3] configure.ac update version numbers

2016-11-29 Thread Anders Roxell
On 2016-11-24 23:24, Maxim Uvarov wrote: > Default is abi compat mode, all interface functions changed, > so increase first number of .so > > Signed-off-by: Maxim Uvarov > --- > configure.ac | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git

Re: [lng-odp] [PATCHv6 1/3] configure.ac: disable shared library for non abi compat mode

2016-11-24 Thread Anders Roxell
On 24 November 2016 at 17:03, Maxim Uvarov wrote: > original configure.ac enables abi compat mode by default, > even without --enable-abi-compat provided. And has broken > logic to disable abi compat mode. Correct logic to build abi > compat mode and option to disable it.

Re: [lng-odp] [PATCHv2] changelog: summary of changes for odp v1.11.1.0

2016-11-23 Thread Anders Roxell
some other library name is non compat mode. (I.e. abi compat application > uses libodp-linux.so on any platforms, > and non abi application uses libodp-linux-plat.so only on platform where it > was compiled.). > > What do you think? > > Thank you, > Maxim. >

Re: [lng-odp] [PATCHv5 2/2] configure.ac update version numbers

2016-11-22 Thread Anders Roxell
On 2016-11-22 21:25, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > configure.ac | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/configure.ac b/configure.ac > index be5a292..287e478 100644 > --- a/configure.ac > +++

Re: [lng-odp] [PATCHv4 0/2] changelog: summary of changes for odp v1.12.0.0

2016-11-22 Thread Anders Roxell
On 2016-11-22 19:04, Maxim Uvarov wrote: > going to merge is very soon, any objection? Features already in monarch that should not be in the change log Hello world thread local Documentation examples added l2fwd Implementers guide - updated Release guide - updated Cheers, Anders > > Maxim. >

Re: [lng-odp] [PATCHv2] changelog: summary of changes for odp v1.11.1.0

2016-11-18 Thread Anders Roxell
On 18 November 2016 at 01:34, Bill Fischofer wrote: > Signed-off-by: Bill Fischofer > --- > Changes for v2: > - Changed release level to v1.11.1.0 per Mike > > CHANGELOG | 211 > ++

Re: [lng-odp] [PATCH] travis: linux-gen: add dpdk pktio

2016-11-18 Thread Anders Roxell
On 18 November 2016 at 14:05, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 11/18/16 12:06, Anders Roxell wrote: >> >> On 7 November 2016 at 16:49, Maxim Uvarov <maxim.uva...@linaro.org> wrote: >>> >>> Turn on linux-generic dpdk and pcap pktios wi

Re: [lng-odp] [PATCH] travis: linux-gen: add dpdk pktio

2016-11-18 Thread Anders Roxell
On 7 November 2016 at 16:49, Maxim Uvarov wrote: > Turn on linux-generic dpdk and pcap pktios with their validation > tests. > > Signed-off-by: Maxim Uvarov > --- > .travis.yml | 22 ++ > 1 file changed, 18 insertions(+), 4

Re: [lng-odp] [PATCH v2] linux-gen: build: de-couple abi compatibility from shared lib

2016-09-29 Thread Anders Roxell
On 30 September 2016 at 01:46, Petri Savolainen wrote: > Building ABI compatible or shared library are two different > targets. A shared library may be used also without ABI > compatibility. A new --enable-abi-compat configuration option > is introduced. By default

Re: [lng-odp] [PATCHv3] test: linux-gen: add pcap playback test

2016-09-14 Thread Anders Roxell
On 2016-09-12 19:54, Bill Fischofer wrote: > On Mon, Sep 12, 2016 at 10:24 AM, Mike Holmes > wrote: > > > On 12 September 2016 at 10:43, Maxim Uvarov > > wrote: > > > > > if no comments than it can be merged? > > > > > > Not without a review, I

Re: [lng-odp] [PATCH] linux-gen: build: de-couple abi compatibility from shared lib

2016-09-13 Thread Anders Roxell
On 8 September 2016 at 16:24, Petri Savolainen wrote: > Building ABI compatible or shared library are two different > targets. A shared library may be used also without ABI > compatibility. A new --enable-abi-compat configuration option > is introduced. By default

Re: [lng-odp] [PATCHv2] test: linux-gen: add pcap playback test

2016-08-26 Thread Anders Roxell
On 25 August 2016 at 10:14, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 08/25/16 00:34, Anders Roxell wrote: >>>>> >>>>> +#include >>>> >>>> > >why do we include an internal header file? >>>> > > >>

Re: [lng-odp] [PATCHv2] test: linux-gen: add pcap playback test

2016-08-24 Thread Anders Roxell
On 2016-08-09 15:46, Maxim Uvarov wrote: > On 08/06/16 22:30, Anders Roxell wrote: > >--- a/test/linux-generic/m4/configure.m4 > >>+++ b/test/linux-generic/m4/configure.m4 > >>@@ -1,5 +1,6 @@ > >> AC_CONFIG_FILES([test/linux-generic/Makefile > >>

Re: [lng-odp] [PATCHv2] test: linux-gen: add pcap playback test

2016-08-24 Thread Anders Roxell
On 2016-08-08 09:59, Maxim Uvarov wrote: > Hello Anders, > > thanks for review, there are 2 comments bellow. Will send v2. > > Maxim. > > On 08/06/16 22:30, Anders Roxell wrote: > >On 2016-08-02 19:08, Maxim Uvarov wrote: > >>add pcap play back test which tak

Re: [lng-odp] [MONARCH PATCHv3 2/2] configure.ac update version

2016-08-18 Thread Anders Roxell
On 2016-08-17 22:25, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org> Reviewed-by: Anders Roxell <anders.rox...@linaro.org> > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac

Re: [lng-odp] [MONARCH PATCHv3 1/2] api: version update from 1.10.1 to 1.11.0

2016-08-18 Thread Anders Roxell
On 2016-08-17 22:25, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org> Reviewed-by: Anders Roxell <anders.rox...@linaro.org> > --- > include/odp/api/spec/version.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > dif

Re: [lng-odp] [MONARCH_LTS PATCHv2 2/2] configure.ac update version

2016-08-17 Thread Anders Roxell
On 2016-08-10 18:25, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index 6551287..0285b0a 100644 > --- a/configure.ac > +++ b/configure.ac >

Re: [lng-odp] [MONARCH_LTS PATCHv2 1/2] scripts/git_hash.sh: to support monarch_lts tags

2016-08-17 Thread Anders Roxell
On 2016-08-10 18:25, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > scripts/git_hash.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh > index 336eb01..35b8749 100755 > ---

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 2/2] configure.ac update version

2016-08-10 Thread Anders Roxell
On 10 August 2016 at 09:33, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 08/09/16 22:26, Anders Roxell wrote: >> >> On 9 August 2016 at 08:54, Maxim Uvarov <maxim.uva...@linaro.org> wrote: >>> >>> On 08/09/16 02:36, Mike Holmes wrote: >>

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 2/2] configure.ac update version

2016-08-09 Thread Anders Roxell
On 9 August 2016 at 08:54, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 08/09/16 02:36, Mike Holmes wrote: >> >> >> >> On 8 August 2016 at 03:01, Maxim Uvarov <maxim.uva...@linaro.org >> <mailto:maxim.uva...@linaro.org>> wrot

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 1/2] scripts/git_hash.sh: to support monarch_lts tags

2016-08-09 Thread Anders Roxell
On 8 August 2016 at 09:50, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 08/05/16 21:33, Anders Roxell wrote: >> >> On 2016-08-03 11:40, Maxim Uvarov wrote: >>> >>> Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org> >>> --- >>>

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 1/2] scripts/git_hash.sh: to support monarch_lts tags

2016-08-05 Thread Anders Roxell
On 2016-08-03 11:40, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > scripts/git_hash.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh > index 336eb01..83123f4 100755 > ---

Re: [lng-odp] [PATCH] helper/hashtable: use static to hide private functions

2016-08-03 Thread Anders Roxell
On 2 August 2016 at 17:27, Mike Holmes <mike.hol...@linaro.org> wrote: > On 2 August 2016 at 04:44, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > >> On 08/02/16 10:54, Anders Roxell wrote: >> >>> Signed-off-by: Anders Roxell <anders.rox...@linaro

Re: [lng-odp] [MONARCH_LTS PATCHv2] changelog: update for v1.11.0.0

2016-08-02 Thread Anders Roxell
On 2 August 2016 at 20:53, Bill Fischofer wrote: > Signed-off-by: Bill Fischofer > --- > CHANGELOG | 114 > ++ > 1 file changed, 114 insertions(+) > > diff --git a/CHANGELOG

[lng-odp] [PATCH] helper/hashtable: use static to hide private functions

2016-08-02 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- helper/hashtable.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/helper/hashtable.c b/helper/hashtable.c index 8bb1ae5..f17b80f 100644 --- a/helper/hashtable.c +++ b/helper/hashtable.c @@ -164,7

[lng-odp] [PATCHv3] linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc

2016-07-22 Thread Anders Roxell
Make it easier for other platforms like odp-dpdk to reuse arch specific files in EXTRA_DIST for "free". Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/Makefile.inc | 17 +++

[lng-odp] [PATCH] linux-dpdk/m4/configure.m4: move m4_include above ac_check_*

2016-07-21 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/linux-dpdk/m4/configure.m4 | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/platform/linux-dpdk/m4/configure.m4 b/platform/linux-dpdk/m4/configure.m4 index 1d04fd0..f370afb 100644 --- a/pl

Re: [lng-odp] [PATCH 1/2] linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc

2016-07-21 Thread Anders Roxell
On 21 July 2016 at 15:12, Mike Holmes <mike.hol...@linaro.org> wrote: > > > On 21 July 2016 at 05:42, Anders Roxell <anders.rox...@linaro.org> wrote: >> >> Make it easier for other platforms like odp-dpdk to reuse arch specific >> files in EXTRA_DIST for

[lng-odp] [PATCHv2] linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc

2016-07-21 Thread Anders Roxell
Make it easier for other platforms like odp-dpdk to reuse arch specific files in EXTRA_DIST for "free". Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/Makefile.inc | 17 +++

Re: [lng-odp] [API-NEXT PATCHv3 01/18] linux-generic: Makefile: reintroducing lost change for drv

2016-07-21 Thread Anders Roxell
On 21 July 2016 at 14:06, Christophe Milard wrote: > The change done for commit id 1fcd2369be88a6f4f7a7a93e9bb315d0e65ab128 > in the Makefile, and delated after is reintroduced here. > > Signed-off-by: Christophe Milard > --- >

[lng-odp] [PATCH 2/2] platform: Makefile.inc: add wildcards

2016-07-21 Thread Anders Roxell
Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/Makefile.inc | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/platform/Makefile.inc b/platform/Makefile.inc index 370383b..0a

[lng-odp] [PATCH 0/2] make it easy for other platform to reuse the arch dir

2016-07-21 Thread Anders Roxell
included from ./include/odp/api/atomic.h:20:0, Cheers, Anders Anders Roxell (2): linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc platform: Makefile.inc: add wildcards platform/Makefile.inc | 5 + platform/linux-generic/Makefile.am | 17 - 2 files

[lng-odp] [PATCH 1/2] linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc

2016-07-21 Thread Anders Roxell
Make it easier for other platforms like odp-dpdk to reuse arch specific files in EXTRA_DIST for "free". Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org>

[lng-odp] [PATCH 2/2] platform: Makefile.inc: add wildcards

2016-07-21 Thread Anders Roxell
Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/Makefile.inc | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/platform/Makefile.inc b/platform/Makefile.inc index 370383b..0a

[lng-odp] [PATCH 1/2] linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc

2016-07-21 Thread Anders Roxell
Make it easier for other platforms like odp-dpdk to reuse arch specific files in EXTRA_DIST for "free". Reported-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org>

[lng-odp] [PATCH 0/2] make it easy for other platform to reuse the arch dir

2016-07-21 Thread Anders Roxell
included from ./include/odp/api/atomic.h:20:0, Cheers, Anders Anders Roxell (2): linux-gen: Makefile: Move EXTRA_DIST to to platform/Makefile.inc platform: Makefile.inc: add wildcards platform/Makefile.inc | 5 + platform/linux-generic/Makefile.am | 17 - 2 files

Re: [lng-odp] [API-NEXT PATCH 12/18] linux-generic: drv: adding align.h

2016-07-21 Thread Anders Roxell
On 21 July 2016 at 09:20, Maxim Uvarov wrote: > On 07/21/16 10:01, Christophe Milard wrote: >> >> On 21 July 2016 at 08:45, Maxim Uvarov wrote: >>> >>> On 07/20/16 23:07, Christophe Milard wrote: I am not sure I understand the problem ?

Re: [lng-odp] [PATCHv3 0/4] Restructuring tests for clarity and new interfaces.

2016-07-15 Thread Anders Roxell
On 2016-07-15 05:46, Christophe Milard wrote: > Cannot see them here when doing normal builds. you must be more > accurate on what you do. Moreover, these warnings seems to relate to > helper tests, which this serie doesn't touch. > confused. Don't be confused... Maxim commented on the wrong

[lng-odp] [PATCHv2 3/3] example/ipsec: scope ipsec examples

2016-07-14 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- example/ipsec/Makefile.am | 18 +- example/ipsec/{run_ah_in => odp_ipsec_run_ah_in} | 0 example/ipsec/{run_ah_out => odp_ipsec_run_ah_out} | 0 example/ipsec/

[lng-odp] [PATCHv2 2/3] example/time/Makefile: remove incorrect postfix _test

2016-07-14 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- example/time/.gitignore | 2 +- example/time/Makefile.am | 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/example/time/.gitignore b/example/time/.gitignore index 3106aea..938c1aa 100644 --- a/exampl

[lng-odp] [PATCHv2 1/3] helper/test/Makefile: don't install tests

2016-07-14 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- helper/test/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helper/test/Makefile.am b/helper/test/Makefile.am index e9e8785..545db73 100644 --- a/helper/test/Makefile.am +++ b/helper/test/Makef

[lng-odp] [PATCHv2 0/3] cleanups

2016-07-14 Thread Anders Roxell
: library has 'chksum' as canonical name (possible typo) Anders Roxell (3): helper/test/Makefile: don't install tests example/time/Makefile: remove incorrect postfix _test example/ipsec: scope ipsec examples example/ipsec/Makefile.am | 18

[lng-odp] [PATCH 3/3] example/ipsec: scope ipsec examples

2016-07-13 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- example/ipsec/Makefile.am| 18 +- example/ipsec/odp_ipsec_run_ah_in| 12 example/ipsec/odp_ipsec_run_ah_out | 12 example/ipsec/odp_ipsec_run_both_in

[lng-odp] [PATCH 2/3] example/time/Makefile: remove incorrect postfix _test

2016-07-13 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- example/time/.gitignore | 2 +- example/time/Makefile.am | 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/example/time/.gitignore b/example/time/.gitignore index 3106aea..938c1aa 100644 --- a/exampl

[lng-odp] [PATCH 1/3] helper/test/Makefile: don't install tests

2016-07-13 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- helper/test/Makefile.am | 2 -- 1 file changed, 2 deletions(-) diff --git a/helper/test/Makefile.am b/helper/test/Makefile.am index e9e8785..59a427a 100644 --- a/helper/test/Makefile.am +++ b/helper/test/Makefile.am @@ -22,8

[lng-odp] [PATCH 0/3] cleanups

2016-07-13 Thread Anders Roxell
This is preparation cleanup so installed executables are scoped and helpers are removed as unhelpful for the final installation. Anders Roxell (3): helper/test/Makefile: don't install tests example/time/Makefile: remove incorrect postfix _test example/ipsec: scope ipsec examples example

[lng-odp] [PATCHv3] configure: split up libodphelper SO-version from libodp

2016-06-30 Thread Anders Roxell
The libodphelper and libopd need to track their evolving ABI independently and so there needs to be two separate version numbers. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 6 +- helper/Makefile.am | 2 +- 2 files changed, 6 insertions(+), 2 del

[lng-odp] [PATCHv2] configure: split up libodphelper SO-version from libodp

2016-06-28 Thread Anders Roxell
The libodphelper and libopd need to track their evolving ABI independently and so there needs to be two separate version numbers. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 3 +++ helper/Makefile.am | 2 +- 2 files changed, 4 insertions(+), 1 de

Re: [lng-odp] [PATCHv2] update version number from v1.10.0.0 to v1.10.1.0

2016-06-14 Thread Anders Roxell
On 14 Jun 2016 2:22 p.m., "Maxim Uvarov" <maxim.uva...@linaro.org> wrote: > > Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org> Review-by: Anders Roxell <anders.rox...@linaro.org> > --- > configure.ac | 2 +- > include/odp/api/

Re: [lng-odp] [PATCH] update version number from v1.10.0.0 to v1.10.1.0

2016-06-14 Thread Anders Roxell
On 14 June 2016 at 12:46, Maxim Uvarov wrote: > Signed-off-by: Maxim Uvarov > --- > configure.ac | 2 +- > include/odp/api/spec/version.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure.ac

[lng-odp] [PATCH] example/timer: don't run test if configure flag not set

2016-06-10 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- example/timer/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/example/timer/Makefile.am b/example/timer/Makefile.am index 1c733d3..edb8b2c 100644 --- a/example/timer/Makefile.am +++ b/example/timer/Makefile.am @@

[lng-odp] [PATCHv2] configure: split up libodphelper SO-version from libodp

2016-06-08 Thread Anders Roxell
The libodphelper and libopd need to track their evolving ABI independently and so there needs to be two separate version numbers. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 3 +++ helper/Makefile.am | 2 +- 2 files changed, 4 insertions(+), 1 de

[lng-odp] [PATCH] configure: split up libodphelper SO-version from libodp

2016-06-07 Thread Anders Roxell
They can be updated separately so no need to bump both versions at the same time, specially when there is no change in one of them. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 3 +++ helper/Makefile.am | 2 +- 2 files changed, 4 insertions(+), 1 de

Re: [lng-odp] [PATCH] linux-generic/test/Makefile: OOT ring build error

2016-06-05 Thread Anders Roxell
On 5 June 2016 at 22:11, Anders Roxell <anders.rox...@linaro.org> wrote: > Add a missing include path to be able to do out-of-tree builds. > > Making all in ring > make[2]: Entering directory > '/media/data/src/odp/crap/platform/linux-generic/test/ring' > CC libte

[lng-odp] [PATCH] linux-generic/test/Makefile: OOT ring build error

2016-06-05 Thread Anders Roxell
]: *** [libtestring_la-ring_stress.lo] Error 1 Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/linux-generic/test/Makefile.inc | 1 + 1 file changed, 1 insertion(+) diff --git a/platform/linux-generic/test/Makefile.inc b/platform/linux-generic/test/Makefile.inc index 2

[lng-odp] [PATCH] linux-generic: include: drv: std_types.h: fix cyclic include

2016-05-23 Thread Anders Roxell
Reported-by: Mike Holmes <mike.hol...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/linux-generic/include/odp/drv/std_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/include/odp/drv/std_types.h b/pl

[lng-odp] [PATCH] helper/test: add missing scripts to EXTRA_DIST

2016-05-23 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- helper/test/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/helper/test/Makefile.am b/helper/test/Makefile.am index 8e65948..e9e8785 100644 --- a/helper/test/Makefile.am +++ b/helper/test/Makefile.am @@ -24,6

Re: [lng-odp] Fedora repo changes

2016-05-16 Thread Anders Roxell
On 11 May 2016 at 21:53, Matt Spencer wrote: > Hi Guys > > Thought I would use some down time to have a play with ODP. On a vanilla > Fedora 23 Server install, there is an issue with the current odp.repo in > that the default behaviour of ‘def’ is to gpg check the

Re: [lng-odp] lng-odp mailman settings

2016-05-16 Thread Anders Roxell
On 2016-05-16 08:27, Savolainen, Petri (Nokia - FI/Espoo) wrote: > > > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Bill > Fischofer > Sent: Saturday, May 14, 2016 1:55 AM > To: Brian Brooks > Cc: lng-odp > Subject:

[lng-odp] [PATCH] scripts/git_hash: match more digits

2016-05-02 Thread Anders Roxell
Make it possible to match more digits than groups of one. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- scripts/git_hash.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh index 6ba265e..336eb01 100755 --- a/s

Re: [lng-odp] odp_api.h missing in ODP 1.9 package

2016-05-02 Thread Anders Roxell
On 2 May 2016 at 12:33, Oriol Arcas wrote: > Hello, > > I noticed that odp_api.h is missing in the new packages for ODP 1.9. > > In ODP 1.8: > > $> dpkg-deb -c > libodp108-dev_1.8.0.0.git28.g4fc79fc-1.linarojessie.1_amd64.deb | grep > odp_api.h > -rw-r--r-- root/root

Re: [lng-odp] [ODP-CHECK PATCHv2] odp-linux: add dpdk pktio testing

2016-04-23 Thread Anders Roxell
On 2016-04-15 22:55, Maxim Uvarov wrote: > Add dpdk pktio compilation and run under native make check. Great, this should be a new variable, because apply-and-build.sh should be able to verify patches on the odp-dpdk project as well, see explanation below [1]. > Huge pages have to be mounted: >

Re: [lng-odp] [API-NEXT PATCHv5] api: make only the API visible

2016-04-18 Thread Anders Roxell
On 18 April 2016 at 18:47, Bill Fischofer <bill.fischo...@linaro.org> wrote: > From: Anders Roxell <anders.rox...@linaro.org> > > Internal functions should not be part of symbols that are visible > outside the library. Using -fvisibility=hidden hides all internal > fu

[lng-odp] [API-NEXT PATCHv5] api: make only the API visible

2016-04-18 Thread Anders Roxell
Internal functions should not be part of symbols that are visible outside the library. Using -fvisibility=hidden hides all internal functions from the public ABI. Suggested-by: Ricardo Salveti <ricardo.salv...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> -

Re: [lng-odp] [API-NEXT PATCHv4] api: make only the API visible

2016-04-15 Thread Anders Roxell
cardo.salv...@linaro.org > <mailto:ricardo.salv...@linaro.org>> wrote: > > On Wed, Apr 13, 2016 at 3:51 PM, Bill Fischofer > <bill.fischo...@linaro.org <mailto:bill.fischo...@linaro.org>> wrote: > > On Wed, Apr 13, 2016 at 1:47 PM, Ricardo Salveti >

[lng-odp] [PATCH] linux-generic: add arch specific *.h files to EXTRA_DIST

2016-04-13 Thread Anders Roxell
To make the build from a tarball work again from an architecture that you didn't create the tarball from. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/linux-generic/Makefile.am | 4 1 file changed, 4 insertions(+) diff --git a/platform/linux-generic/Makefile

[lng-odp] [API-NEXT PATCHv4] api: make only the API visible

2016-04-13 Thread Anders Roxell
Internal functions should not be part of symbols that are visible outside the library. Using -fvisibility=hidden hides all internal functions from the public ABI. Suggested-by: Ricardo Salveti <ricardo.salv...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> ---

[lng-odp] [PATCHv3 3/3] linux-generic: use hidden on internal functions

2016-04-12 Thread Anders Roxell
Internal functions should not be visible in the ABI. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- .../linux-generic/include/odp_buffer_internal.h| 7 +- .../include/odp_classification_internal.h | 42 ++-- platform/linux-generic/include/odp_inte

[lng-odp] [PATCHv3 2/3] linux-generic/include/odp_internal: add hidden attribute

2016-04-12 Thread Anders Roxell
The hidden attribute allows functions to be hidden from the public ABI. Internal functions should not be part of symbols that are visible outside the library. Suggested-by: Ricardo Salveti <ricardo.salv...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- pl

[lng-odp] [PATCHv3 1/3] linux-generic: odp_schedule: drop prefix odp on internal functions

2016-04-12 Thread Anders Roxell
Internal functions shouldn't have use the prefix odp only the public API. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/linux-generic/include/odp_schedule_internal.h | 2 +- platform/linux-generic/odp_schedule.c | 6 +++--- 2 files changed, 4 inse

[lng-odp] [PATCHv3 0/3] hide internal functions

2016-04-12 Thread Anders Roxell
Hi, Went through all internal functions so they aren't visible from the outside library. Cheers, Anders Anders Roxell (3): linux-generic: odp_schedule: drop prefix odp on internal functions linux-generic/include/odp_internal: add hidden attribute linux-generic: use hidden on internal

[lng-odp] [PATCH] platform/Makefile.inc: remove incorrect install to share dir

2016-04-11 Thread Anders Roxell
Remove libodp-linux.la from /usr/share/... its already installed in /usr/lib/ Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- platform/Makefile.inc | 2 -- 1 file changed, 2 deletions(-) diff --git a/platform/Makefile.inc b/platform/Makefile.inc index 1cb7a71..160ad4c

Re: [lng-odp] [PATCHv2] linux-generic/include/odp_packet_io_internal: add visability hidden

2016-04-11 Thread Anders Roxell
On 9 April 2016 at 20:07, Mike Holmes <mike.hol...@linaro.org> wrote: > > > On 8 April 2016 at 17:14, Anders Roxell <anders.rox...@linaro.org> wrote: >> >> Internal functions should not be part of symbols is visible outside the >> library. >> &

Re: [lng-odp] [PATCHv2] linux-generic/include/odp_packet_io_internal: add visability hidden

2016-04-11 Thread Anders Roxell
g works without a problem, so I don't think there is anything to do. Cheers, Anders > > On Fri, Apr 8, 2016 at 4:14 PM, Anders Roxell <anders.rox...@linaro.org> > wrote: >> >> Internal functions should not be part of symbols is visible outside the >> library. &

[lng-odp] [PATCHv2] linux-generic/include/odp_packet_io_internal: add visability hidden

2016-04-08 Thread Anders Roxell
Internal functions should not be part of symbols is visible outside the library. Suggested-by: Ricardo Salveti <ricardo.salv...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- .../linux-generic/include/odp_packet_io_internal.h | 23 +- 1

Re: [lng-odp] [PATCH] linux-generic/odp_impl: removing __DATE__ and __TIME__

2016-04-07 Thread Anders Roxell
o.org/show_bug.cgi?id=2154 > > Signed-off-by: Ricardo Salveti <ricardo.salv...@linaro.org> Reviewed-by: Anders Roxell <anders.rox...@linaro.org> > --- > platform/linux-generic/odp_impl.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/platform/linux-gen

Re: [lng-odp] [PATCHv3] validation: configure: move cflags guards to test/m4

2016-04-07 Thread Anders Roxell
o.org> > Suggested-by: Anders Roxell <anders.rox...@linaro.org> > Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org> if fixing the changelog. Reviewed-by: Anders Roxell <anders.rox...@linaro.org> > --- > v3: drop patch 1. (btw there were no reject). > &

Re: [lng-odp] [PATCH 1/2] configure: support out-of-tree CUnit

2016-04-07 Thread Anders Roxell
On 7 April 2016 at 15:17, Maxim Uvarov wrote: > From: Brian Brooks > > If --with-cunit-path=DIR is used, skip the AC_CHECK_LIB and modify the linker > flags accordingly. > > Signed-off-by: Brian Brooks > Signed-off-by:

Re: [lng-odp] [PATCH] configure: reorder m4_includes

2016-04-07 Thread Anders Roxell
>> Maxim >> Uvarov >> Sent: Wednesday, April 06, 2016 2:50 PM >> To: lng-odp@lists.linaro.org >> Subject: Re: [lng-odp] [PATCH] configure: reorder m4_includes >> >> Merged, >> Maxim. >> >> On 04/05/16 20:08, Brian Brooks wrote: >> >

Re: [lng-odp] [PATCH 1/2] configure: support out-of-tree CUnit

2016-04-04 Thread Anders Roxell
On 26 March 2016 at 02:06, Brian Brooks <brian.bro...@linaro.org> wrote: > On 04/01 22:15:34, Anders Roxell wrote: >> On 25 March 2016 at 20:25, Brian Brooks <brian.bro...@linaro.org> wrote: >> > If --with-cunit-path=DIR is used, skip the AC_CHECK_LIB and modi

[lng-odp] [PATCHv2] configure: remove separate so_version file

2016-04-01 Thread Anders Roxell
Set versioning in configure.ac directly, folks familiar with autotools will expect it to be set there. Suggested-by: Fathi Boudra <fathi.bou...@linaro.org> Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- .so_version | 1 - Makefile.am | 2 +- configure

[lng-odp] [PATCH] configure: reorder m4_includes

2016-04-01 Thread Anders Roxell
t; Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- Thanks to Brians patch "[PATCH 2/2] configure: remove dead code", with my suggestion to it, we found out that the order matters. Cheers, Anders configure.ac | 16 1 file changed, 8 insertions(+), 8

Re: [lng-odp] [PATCH 1/2] configure: support out-of-tree CUnit

2016-04-01 Thread Anders Roxell
On 25 March 2016 at 20:25, Brian Brooks wrote: > If --with-cunit-path=DIR is used, skip the AC_CHECK_LIB and modify the linker > flags accordingly. Why should we skip the AC_CHECK_LIB and AC_CHECK_HEADERS step don't feel right. > > Signed-off-by: Brian Brooks

Re: [lng-odp] [PATCH 2/2] configure: remove dead code

2016-04-01 Thread Anders Roxell
On 25 March 2016 at 20:25, Brian Brooks wrote: > Remove dead code which potentially acted as a workaround for making > AC_CHECK_LIB to work with --with-cunit-path. Workaround you may say that. I would put it, one way to do checks for libs and headers that is in the path

[lng-odp] [PATCH] configure: correctly set the SO-version when build OOT

2016-03-31 Thread Anders Roxell
Fix out-of-tree build bug introduced with patch "972f47f configure: disconnect API version with SO version" Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.

Re: [lng-odp] [PATCH] test: run only memcheck

2016-03-30 Thread Anders Roxell
d-off-by: Mike Holmes <mike.hol...@linaro.org> Reviewed-by: Anders Roxell <anders.rox...@linaro.org> > --- > test/Makefile.inc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/test/Makefile.inc b/test/Makefile.inc > index c4399cb..5d

Re: [lng-odp] [PATCH] configure: disconnect API version with SO version

2016-03-30 Thread Anders Roxell
On 30 March 2016 at 21:26, Ricardo Salveti <ricardo.salv...@linaro.org> wrote: > On Wed, Mar 30, 2016 at 4:08 PM, Anders Roxell <anders.rox...@linaro.org> > wrote: >> We used to force applications to rebuild when a ODP released a new >> version that chang

[lng-odp] [PATCH] configure: disconnect API version with SO version

2016-03-30 Thread Anders Roxell
We used to force applications to rebuild when a ODP released a new version that changed one of the first two digits. That shouldn't be needed if we use the SO-verson as it is intended to be used. Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- .so_version | 1 + Makef

[lng-odp] [PATCHv2 06/10] configure: move test_helper to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 13 ++--- helper/m4/configure.m4 | 9 + 2 files changed, 11 insertions(+), 11 deletions(-) create mode 100644 helper/m4/configure.m4 diff --git a/configure.ac b/configure.ac index 0

[lng-odp] [PATCHv2 09/10] configure: move test_vald to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 51 ++- test/m4/configure.m4 | 1 + test/m4/validation.m4 | 44 3 files changed, 47 insertions(+), 49 del

[lng-odp] [PATCHv2 07/10] configure: move docs to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac| 49 - doc/m4/configure.m4 | 38 ++ 2 files changed, 42 insertions(+), 45 deletions(-) diff --git a/configure.ac b/config

[lng-odp] [PATCHv2 04/10] configure: move test_perf to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 19 ++- test/m4/configure.m4 | 2 ++ test/m4/performance.m4 | 9 + 3 files changed, 17 insertions(+), 13 deletions(-) create mode 100644 test/m4/performance.m4 diff

[lng-odp] [PATCHv2 01/10] linux-generic: m4: netmap: set netmap_support=no

2016-03-29 Thread Anders Roxell
Conform with other m4 files Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac| 1 - platform/linux-generic/m4/odp_netmap.m4 | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 31d4d32..7

[lng-odp] [PATCHv2 08/10] configure: move example configure to the others

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 883f52f..e25c7a4 100644 --- a/configure.ac +++ b/configure.ac @@ -85,6 +85,7 @@ AC_SUBST([platform_with_platfor

[lng-odp] [PATCHv2 05/10] configure: move test_cpp to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell <anders.rox...@linaro.org> --- configure.ac | 16 +--- test/m4/configure.m4 | 1 + test/m4/miscellaneous.m4 | 9 + 3 files changed, 11 insertions(+), 15 deletions(-) create mode 100644 test/m4/miscellaneous.m4 diff

  1   2   3   4   5   6   >