Age | Commit message (Collapse) | Author |
|
Increment API version number to reflect various API changes
and additions. Major number is incremented as changes to packet IO
checksum offload specification and timestamp API are not backwards
compatible.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
These warnings have now been fixed. The warning can be enabled
again as an error.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
Reviewed-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
|
|
Make configure --(en|dis)able-doxygen-dot functional. Enable by
default. If dot tool is not found, a warning is emitted and the
feature is automatically disabled.
Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version number to reflect crypto and classifier API
changes.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
|
|
Always build both random module implementations to detect possible build
issues. Renamed random_null implementation to random_std.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Make sure that all autoconf arguments defined by ODP use default value
description similar to built-in arguments. That is: [default=<value>].
Also make sure that all arguments use AS_HELP_STRING for uniform text
formatting.
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Every platform may now add its own configure status by appending to
PLAT_CFG_TEXT autoconf variable. Note that since it is an autoconf
variable, it should be appended by AS_VAR_APPEND.
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Autoconf configuration for OpenSSL is required for the IPSec example as
well as for linux-generic internals. Therefore any other platform added
will by extension require this code if it wants to compile the IPSec
example.
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version to reflect API and documentation cleanups.
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Jerin Jacob <jerinj@marvell.com>
Reviewed-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
|
|
Debian-based Linux distributions have introduced a while ago a very
specific patch to libtool generation which disables libtool generating
dependent libraries list to the linker (distribution libtool.m4 has
link_all_deplibs forced to no). This causes issues when cross-compiling
while using dependent libraries compiled and installed manually to
non-sysroot location (see [1]).
To reproduce the issue, one has to fetch a toolchain which does not
contain pkg-config in it. Debian/Ubuntu aarch64 toolchains do contain
version of that tool (aarch64-linux-gnu-pkg-config) which does not have
the change in question.
There are several solutions to fix this, however this patch opts for
explicitly listing dependent libraries in examples and tests.
It is done by adding the PLAT_DEP_LIBS variable which platform should
fill with libraries that can be added from non-standard locations (such
as libconfig and libcrypto for linux-generic).
Additionally move the non-platform specific autoconf definitions prior
to the platform's configure.m4 so that platforms may adapt to argument
customizations passed by the user.
[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=297726
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version number to reflect API additions and documentation
clarifications.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
GCC version may contain minor and patch version, therefore extract major
version expecting that.
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
|
|
So far the build system (helper, examples, tests) expected that the ODP
library is named libodp-linux. However that's only a recommended library
name. Therefore move the library name specification to the platform
automake files. More precisely platform has to define the following:
1. In configure.m4:
ODP_LIB_NAME=<name>
2. In Makefile.am:
lib_LTLIBRARIES += $(LIB)/lib<name>.la
Where <name> is the library name. Please note that lib_LTLIBRARIES has
to be explicitly named (cannot use ODP_LIB_NAME variable), otherwise
automake will fail to generate compile target.
Trying to perform a substitutions in it is only possible with full
library name substitution (see [1]), however then all possible library
names have to be defined in EXTRA_LTLIBRARIES which cannot contain
substitutions. This would effectively replace one hard-coded place with
another.
[1] https://www.gnu.org/software/automake/manual/
html_node/Conditional-Libtool-Libraries.html
Signed-off-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version number to reflect:
* additions to timer, time and TM APIs
* clarifications in random number API
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
When providing a conf file with "--with-config-file",
- In case of change in this file, rebuild was not doing anything
- In case of change in generic conf file, rebuild was using
generic conf file instead of provided one to regenerate / rebuild
- Generic conf file was installed
Signed-off-by: Thomas Pansiot <thomas.pansiot@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
GCC 10 sometimes gets confused about object sizes and generates bogus
warnings. As a workaround, make -Warray-bounds and -Wstringop-overflow not
generate errors as part of -Werror when GCC version 10 or later is used.
Fixes: https://github.com/OpenDataPlane/odp/issues/959
Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version number to reflect API additions
and documentation clarifications.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Increment API version number to reflect addition of the new
stash API.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Incremented minor version number to reflect packet length
clarifications and addition of packet pool align parameter.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Increment API version number to reflect addition of
ODP_VERSION_API, ODP_VERSION_API_NUM and ODP_CACHE_LINE_ROUNDUP
macros.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Increment the minor API version digit to reflect addition
of odp_packet_input_set() function and compress feature flag.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Increment API version number to reflect API changes:
* cls: PMR values for protocol fields are defined in big endian
* cls: ODP_PMR_CUSTOM_L3 term has been added
* coding style: use 'unsigned int' instead of 'unsigned'
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Trying to build static test applications without static ODP library would
fail.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Rename implementation internal defines so that they stand out from the
API defines.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
The config.h file has been split into three parts:
autoheader_build.h: should NOT be included by ANY source file
autoheader_external.h: can be included by any source file, including
public headers
autoheader_internal.h: can be included by any internal source file, but
NOT by public headers
autoheader_external.h and autoheader_internal.h are generated based on
matching template files (*.in) and include only ODP defines.
autoheader_build.h includes all build defines and should not be included
by any source file.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Fix GCC-9 build by handling "pointer to packed member of
a struct" warnings not as errors. These warnings are generated
e.g. when taking pointers to protocol header structures, which
are defined with packed attribute. These structures are
naturally aligned, so these warnings can be ignored.
chksum.c: In function odph_process_l4_hdr:
chksum.c:133:43: error: taking address of packed member of
struct <anonymous> may result in an unaligned pointer
value [-Werror=address-of-packed-member]
pkt_chksum_ptr = (uint16_t *)(void *)&udp_hdr_ptr->chksum;
^~~~~~~~~~~~~~~~~~~~
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Previously, config.h header was not included in install dir. This caused a
compilation failure when an application tried to include ODP helper
headers.
config.h has been moved to include/odp subdirectory to avoid naming
conflicts and unnecessary includes have been removed.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reported-by: Mikko Parpala <mikko.parpala@nokia.com>
|
|
ODP API already has ODP_UNUSED macro.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Increment helper version number to reflect addition of
ODPH_ASSERT and other odph_debug.h content into helper API.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Test application code needs also debug code (asserts) that
can be turned off/on. Added new preprocessor define (ODPH_DEBUG)
and a configure option to control it. All debug options are
enabled when --enable-debug=full is used.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Added option to build with link time optimization (-flto flag).
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
In addition to CFLAGS print also CXXFLAGS after configure.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Dmitry Eremin-Solenikov <deremin-solenikov@cavium.com>
|
|
If user defines CFLAGS do not add -march option as user defined
flags may contain another -march or other related options.
Automake sets CFLAGS to "-g -O2" by default, so CFLAGS check
must to be done before AM_INIT.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Dmitry Eremin-Solenikov <deremin-solenikov@cavium.com>
|
|
Increment version number to reflect timer API specification
update and additional features.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Bump ODP API minor version, as API changes are backwards compatible.
Signed-off-by: Dmitry Eremin-Solenikov <deremin-solenikov@cavium.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Fix out-of-tree build issue introduced by addition of
the new version.h file.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Increment version number to reflect addition of the new
thread create and join functions.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Added helper library version defines, so that application can
track helper version independent of ODP API version. Added also
function for easy print out of the versions number.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Increment version number to reflect queue API specification
update.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Update version number to reflect init API text modifications.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
|
|
Support platforms that do not use config file.
Signed-off-by: Dmitry Eremin-Solenikov <deremin-solenikov@cavium.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <Stanislaw.Kardach@cavium.com>
|
|
Increment minor version number to reflect backward compatible
API changes.
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
|
|
Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Bill Fischofer <billf@me.com>
|
|
Increment minor version number to reflect backward compatible
API changes.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Stanislaw Kardach <skardach@marvell.com>
|
|
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
|
|
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>
|
|
performance.m4 which provides --enable-test-perf-proc is
not needed now because process mode is tested with standard
tests.
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>
Reviewed-and-tested-by Matias Elo <matias.elo@nokia.com>
|
|
for clang we see strange optimizations for TM code which breaks
code flow execution. There might be number of such places. For
now it's better to disable march=native until we completely
test and validate clang support for this.
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Reviewed-by: Petri Savolainen <petri.savolainen@linaro.org>
|
|
When not building in ABI compatible mode, enable compiler
optimizations for the CPU architecture of the local machine.
Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>
Reviewed-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>
Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
|