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",
; + }
> + }
> + 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
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
> +++
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
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
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
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.
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.
>
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
> +++
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.
>
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
> ++
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
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
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
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
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
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?
>>>> > >
>>
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
> >>
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
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
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
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
>
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
> ---
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:
>>
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
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>
>>> ---
>>>
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
> ---
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
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
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
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 +++
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
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
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 +++
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
> ---
>
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
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
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>
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
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>
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
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 ?
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
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/
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
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
: 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
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
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
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
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
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
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
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/
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
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
@@
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
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
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
]: *** [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
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
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
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
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:
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
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
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:
>
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
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>
-
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
>
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
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>
---
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
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
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
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
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
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.
>>
&
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.
&
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
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
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).
>
&
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:
>> 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:
>> >
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
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
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
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
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
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.
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
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
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
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
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
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
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
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
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
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 - 100 of 557 matches
Mail list logo