This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "".
The branch, master has been updated via 56834f66e317f4bf941de808cb6529bad3d3baa6 (commit) via 7a16f52fb6fbaadbf405bfb2c0f7b9c0ce58b0a9 (commit) via e491dad2b1c781008cd3e7904137adc2f09fd076 (commit) via 66feb78128fa440c2a5a1b0f9bd513949f25f5e8 (commit) via b820eb79b68dbdd22ea8d84b5b88778942f0652a (commit) from d1df2b5511707783c2be6eda6c0d64663538bdfa (commit)
Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below.
- Log ----------------------------------------------------------------- commit 56834f66e317f4bf941de808cb6529bad3d3baa6 Author: Petri Savolainen petri.savolainen@nokia.com Date: Tue Jun 18 15:28:12 2019 +0300
travis: add -O3 and -O0 tests for Ubuntu 18.04
Ubuntu 18.04 has newer compiler version than Ubuntu 16.04.
Signed-off-by: Petri Savolainen petri.savolainen@nokia.com Reviewed-by: Matias Elo matias.elo@nokia.com
diff --git a/.travis.yml b/.travis.yml index 82ff4bb26..9fbc1e47e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -56,6 +56,8 @@ env: matrix: - CHECK=0 CONF="CFLAGS=-O3" - CHECK=0 CONF="CFLAGS=-O0 --enable-debug --enable-debug-print" + - CHECK=0 OS=ubuntu_18.04 CONF="CFLAGS=-O3" + - CHECK=0 OS=ubuntu_18.04 CONF="CFLAGS=-O0 --enable-debug --enable-debug-print" - CHECK=0 ARCH=arm64 - CHECK=0 ARCH=armhf - CHECK=0 ARCH=powerpc
commit 7a16f52fb6fbaadbf405bfb2c0f7b9c0ce58b0a9 Author: Petri Savolainen petri.savolainen@nokia.com Date: Tue Jun 18 14:53:17 2019 +0300
travis: remove extra double quotes
Quotes are not needed with simple strings (no space characters). For uniformity, quotes are used always with CONF variable.
Signed-off-by: Petri Savolainen petri.savolainen@nokia.com Reviewed-by: Matias Elo matias.elo@nokia.com
diff --git a/.travis.yml b/.travis.yml index 6ae5895ec..82ff4bb26 100644 --- a/.travis.yml +++ b/.travis.yml @@ -49,21 +49,21 @@ env: # you need to generate a new one at https://codecov.io specific for your # repo. - CODECOV_TOKEN=a733c34c-5f5c-4ff1-af4b-e9f5edb1ab5e - - OS="ubuntu_16.04" + - OS=ubuntu_16.04 - ARCH=x86_64 - CHECK=1 - NETMAP=0 matrix: - CHECK=0 CONF="CFLAGS=-O3" - CHECK=0 CONF="CFLAGS=-O0 --enable-debug --enable-debug-print" - - CHECK=0 ARCH="arm64" - - CHECK=0 ARCH="armhf" - - CHECK=0 ARCH="powerpc" - - CHECK=0 ARCH="i386" - - CHECK=0 ARCH="arm64" CONF="--disable-abi-compat" - - CHECK=0 ARCH="armhf" CONF="--disable-abi-compat" - - CHECK=0 ARCH="powerpc" CONF="--disable-abi-compat" - - CHECK=0 ARCH="i386" CONF="--disable-abi-compat" + - CHECK=0 ARCH=arm64 + - CHECK=0 ARCH=armhf + - CHECK=0 ARCH=powerpc + - CHECK=0 ARCH=i386 + - CHECK=0 ARCH=arm64 CONF="--disable-abi-compat" + - CHECK=0 ARCH=armhf CONF="--disable-abi-compat" + - CHECK=0 ARCH=powerpc CONF="--disable-abi-compat" + - CHECK=0 ARCH=i386 CONF="--disable-abi-compat" - CONF="" - CONF="--disable-abi-compat" - CONF="--enable-deprecated" @@ -73,16 +73,16 @@ env: - CONF="--disable-host-optimization" - CONF="--disable-host-optimization --disable-abi-compat" - CHECK=0 CONF="--enable-pcapng-support" - - CHECK=0 OS="centos_7" + - CHECK=0 OS=centos_7 - CONF="--without-openssl --without-pcap" - - OS="ubuntu_18.04" + - OS=ubuntu_18.04
matrix: exclude: - compiler: gcc - env: CHECK=0 ARCH="arm64" + env: CHECK=0 ARCH=arm64 - compiler: gcc - env: CHECK=0 ARCH="i386" + env: CHECK=0 ARCH=i386
install: - if [ ${NETMAP} -eq 1 ] ; then
commit e491dad2b1c781008cd3e7904137adc2f09fd076 Author: Petri Savolainen petri.savolainen@nokia.com Date: Tue Jun 18 13:50:20 2019 +0300
travis: add -O3 and -O0 build tests
Sometimes build fails with compiler optimization level -O3 or -O0, although it builds correctly with -O2 level. Check that code builds with -O3 and -O0 levels.
Signed-off-by: Petri Savolainen petri.savolainen@nokia.com Reviewed-by: Matias Elo matias.elo@nokia.com
diff --git a/.travis.yml b/.travis.yml index 51cf2903a..6ae5895ec 100644 --- a/.travis.yml +++ b/.travis.yml @@ -54,9 +54,8 @@ env: - CHECK=1 - NETMAP=0 matrix: - - CONF="" - - CONF="--disable-abi-compat" - - NETMAP=1 CONF="" + - CHECK=0 CONF="CFLAGS=-O3" + - CHECK=0 CONF="CFLAGS=-O0 --enable-debug --enable-debug-print" - CHECK=0 ARCH="arm64" - CHECK=0 ARCH="armhf" - CHECK=0 ARCH="powerpc" @@ -65,8 +64,11 @@ env: - CHECK=0 ARCH="armhf" CONF="--disable-abi-compat" - CHECK=0 ARCH="powerpc" CONF="--disable-abi-compat" - CHECK=0 ARCH="i386" CONF="--disable-abi-compat" + - CONF="" + - CONF="--disable-abi-compat" - CONF="--enable-deprecated" - CONF="--enable-dpdk-zero-copy --disable-static-applications" + - NETMAP=1 CONF="" - NETMAP=1 CONF="--disable-static-applications" - CONF="--disable-host-optimization" - CONF="--disable-host-optimization --disable-abi-compat"
commit 66feb78128fa440c2a5a1b0f9bd513949f25f5e8 Author: Petri Savolainen petri.savolainen@nokia.com Date: Tue Jun 18 13:03:25 2019 +0300
travis: set default ARCH to x86_64
Test cases are build for x86_64 by default. ARCH needs to be set only when building for another CPU architecture.
Signed-off-by: Petri Savolainen petri.savolainen@nokia.com Reviewed-by: Matias Elo matias.elo@nokia.com
diff --git a/.travis.yml b/.travis.yml index a32b48e0b..51cf2903a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -50,6 +50,7 @@ env: # repo. - CODECOV_TOKEN=a733c34c-5f5c-4ff1-af4b-e9f5edb1ab5e - OS="ubuntu_16.04" + - ARCH=x86_64 - CHECK=1 - NETMAP=0 matrix: @@ -69,8 +70,8 @@ env: - NETMAP=1 CONF="--disable-static-applications" - CONF="--disable-host-optimization" - CONF="--disable-host-optimization --disable-abi-compat" - - CHECK=0 ARCH="x86_64" CONF="--enable-pcapng-support" - - CHECK=0 ARCH="x86_64" OS="centos_7" + - CHECK=0 CONF="--enable-pcapng-support" + - CHECK=0 OS="centos_7" - CONF="--without-openssl --without-pcap" - OS="ubuntu_18.04"
commit b820eb79b68dbdd22ea8d84b5b88778942f0652a Author: Petri Savolainen petri.savolainen@nokia.com Date: Tue Jun 18 11:01:54 2019 +0300
linux-gen: schedule_sp: add extra check to suppress compiler warning
GCC 7.4 with -O3 optimization level warns (falsely) about array bounds. Suppress the warning with an extra check.
Signed-off-by: Petri Savolainen petri.savolainen@nokia.com Reviewed-by: Matias Elo matias.elo@nokia.com
diff --git a/platform/linux-generic/odp_schedule_sp.c b/platform/linux-generic/odp_schedule_sp.c index 79e985c09..106874552 100644 --- a/platform/linux-generic/odp_schedule_sp.c +++ b/platform/linux-generic/odp_schedule_sp.c @@ -297,6 +297,12 @@ static void remove_group(sched_group_t *sched_group, int thr, int group)
num = thr_group->num_group;
+ /* Extra array bounds check to suppress warning on GCC 7.4 with -O3 */ + if (num >= NUM_GROUP) { + ODP_ERR("Too many groups"); + return; + } + for (i = 0; i < num; i++) { if (thr_group->group[i] == group) { found = 1;
-----------------------------------------------------------------------
Summary of changes: .travis.yml | 35 ++++++++++++++++++-------------- platform/linux-generic/odp_schedule_sp.c | 6 ++++++ 2 files changed, 26 insertions(+), 15 deletions(-)
hooks/post-receive