Re: [Qemu-devel] [PATCH v2 5/5] .travis.yml: reduce target list on core configure tweaks

From: David Gibson <david@gibson.dropbear.id.au>
To: "Alex Bennée" <alex.bennee@linaro.org>
Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 5/5] .travis.yml: reduce target list on core configure tweaks
Date: Thu, 24 Mar 2016 11:22:20 +1100	[thread overview]
Message-ID: <20160324002219.GB23586@voom.redhat.com> (raw)
In-Reply-To: <1458753657-21664-6-git-send-email-alex.bennee@linaro.org>

[-- Attachment #1: Type: text/plain, Size: 3098 bytes --]

On Wed, Mar 23, 2016 at 05:20:57PM +0000, Alex Bennée wrote:
> A number of configure options only really affect the core code and any
> arch specific stuff should be flushed out by other builds:
> 
>   - trace-backends, log build is all targets, others can be less
>   - --disable-build, ensuring disabling stuff doesn't break host
>   - co-routine, default already built with all, common API
>   - tcg interpreter is front end agnostic
> 
> I've used the ./configure --target-list stem support to do all of one
> arch type in each reduced build.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

AFAICT compiling extra targets is very unlikely to detect extra
problems with those config options.

> ---
>  .travis.yml | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 50ac17f..d13bffc 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -45,10 +45,10 @@ env:
>    matrix:
>      - CONFIG=""
>      - CONFIG="--enable-debug --enable-debug-tcg --enable-trace-backends=log"
> -    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb"
> -    - CONFIG="--enable-modules"
> -    - CONFIG="--with-coroutine=ucontext"
> -    - CONFIG="--with-coroutine=sigaltstack"
> +    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --target-list=86"
> +    - CONFIG="--enable-modules --target-list=86"
> +    - CONFIG="--with-coroutine=ucontext --target-list=arm"
> +    - CONFIG="--with-coroutine=sigaltstack --target-list=ppc"
>  git:
>    # we want to do this ourselves
>    submodules: false
> @@ -70,19 +70,19 @@ matrix:
>      - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
>        compiler: gcc
>      # We manually include builds which we disable "make check" for
> -    - env: CONFIG="--enable-debug --enable-tcg-interpreter"
> +    - env: CONFIG="--enable-debug --enable-tcg-interpreter --target-list=mips"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=simple"
> +    - env: CONFIG="--enable-trace-backends=simple --target-list=alpha"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=ftrace"
> +    - env: CONFIG="--enable-trace-backends=ftrace --target-list=s390"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=ust"
> +    - env: CONFIG="--enable-trace-backends=ust --target-list=sparc"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--with-coroutine=gthread"
> +    - env: CONFIG="--with-coroutine=gthread --target-list=sh4"
>             TEST_CMD=""
>        compiler: gcc
>      - env: CONFIG=""

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]