The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5: Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100) are available in the Git repository at: https://github.com/stsquad/qemu.git tags/pull-travis-updates-140618-1 for you to fetch changes up to f8309de9b7e7de1bfdfaddf8df4130a06e7e31b7: travis: reduce time taken for trace-backend testing (2018-06-14 20:24:07 +0100) ---------------------------------------------------------------- Travis updates - show config.log when failing - reduce time for gprof build - reduce time for alternate trace builds ---------------------------------------------------------------- Alex Bennée (2): travis: reduce coverage of gprof build travis: reduce time taken for trace-backend testing Daniel P. Berrangé (1): travis: display config.log when configure fails .travis.yml | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) -- 2.17.1
On 15 June 2018 at 12:09, Alex Bennée <alex.bennee@linaro.org> wrote: > The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5: > > Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100) > > are available in the Git repository at: > > https://github.com/stsquad/qemu.git tags/pull-travis-updates-140618-1 > > for you to fetch changes up to f8309de9b7e7de1bfdfaddf8df4130a06e7e31b7: > > travis: reduce time taken for trace-backend testing (2018-06-14 20:24:07 +0100) > > ---------------------------------------------------------------- > Travis updates > > - show config.log when failing > - reduce time for gprof build > - reduce time for alternate trace builds Applied, thanks. -- PMM
From: Daniel P. Berrangé <berrange@redhat.com> When configure fails in CI systems we must be able to see the contents of the config.log file to diagnose the root cause. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Fam Zheng <famz@redhat.com> [AJB: used Eric's suggested {} form] Signed-off-by: Alex Bennée <alex.bennee@linaro.org> diff --git a/.travis.yml b/.travis.yml index 814be151f4..d5e6d50e6d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -69,7 +69,7 @@ before_install: - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ - git submodule update --init --recursive before_script: - - ./configure ${CONFIG} + - ./configure ${CONFIG} || { cat config.log && exit 1; } script: - make ${MAKEFLAGS} && ${TEST_CMD} matrix: @@ -151,4 +151,4 @@ matrix: - CONFIG="--cc=gcc-7 --cxx=g++-7 --disable-pie --disable-linux-user" - TEST_CMD="" before_script: - - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -fsanitize=thread -fuse-ld=gold" || cat config.log + - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; } -- 2.17.1
This build is regularly timing out and even switching off linux-user wasn't enough. Instead explicitly choose a target list of broadly the "major" architectures. This is enough to check the gprof build machinery works without worrying about the actual coverage results. I did try various YAML constructs for specifying CONFIG with continuation but couldn't get any of them to work hence the very long line. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> diff --git a/.travis.yml b/.travis.yml index d5e6d50e6d..c200dde534 100644 --- a/.travis.yml +++ b/.travis.yml @@ -80,7 +80,7 @@ matrix: - env: CONFIG="--disable-user" compiler: clang # gprof/gcov are GCC features - - env: CONFIG="--enable-gprof --enable-gcov --disable-pie --disable-linux-user" + - env: CONFIG="--enable-gprof --enable-gcov --disable-pie --target-list=aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu" compiler: gcc # We manually include builds which we disable "make check" for - env: CONFIG="--enable-debug --enable-tcg-interpreter" -- 2.17.1
These builds are reaching regular timeouts and probably don't need to be so widely exercised. ftrace and ust in particular are used in conjunction with whole system profiling which makes most sense with KVM setups, hence the native softmmu target. We also expand simple to cover the multiple log backends while restricting its scope to user-mode testing only. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> diff --git a/.travis.yml b/.travis.yml index c200dde534..fabfe9ec34 100644 --- a/.travis.yml +++ b/.travis.yml @@ -86,13 +86,14 @@ matrix: - env: CONFIG="--enable-debug --enable-tcg-interpreter" TEST_CMD="" compiler: gcc - - env: CONFIG="--enable-trace-backends=simple" + # We don't need to exercise every backend with every front-end + - env: CONFIG="--enable-trace-backends=log,simple,syslog --disable-system" TEST_CMD="" compiler: gcc - - env: CONFIG="--enable-trace-backends=ftrace" + - env: CONFIG="--enable-trace-backends=ftrace --target-list=x86_64-softmmu" TEST_CMD="" compiler: gcc - - env: CONFIG="--enable-trace-backends=ust" + - env: CONFIG="--enable-trace-backends=ust --target-list=x86_64-softmmu" TEST_CMD="" compiler: gcc - env: CONFIG="--disable-tcg" -- 2.17.1
© 2016 - 2024 Red Hat, Inc.