[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 1/1] .travis.yml: add --disable-linux-user for so
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PULL 1/1] .travis.yml: add --disable-linux-user for some jobs |
Date: |
Tue, 13 Mar 2018 14:02:54 +0000 |
User-agent: |
mu4e 1.1.0; emacs 26.0.91 |
Daniel P. Berrangé <address@hidden> writes:
> On Fri, Feb 09, 2018 at 02:15:14PM +0000, Alex Bennée wrote:
>> The modules and co-routine builds are only really relevant to softmmu
>> builds and regularly timeout on Travis. Let's disable linux-user
>> builds here for more headroom.
>>
>> Signed-off-by: Alex Bennée <address@hidden>
>> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>> Reviewed-by: Daniel P. Berrange <address@hidden>
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index 01a57399b5..0dd5020552 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -52,9 +52,9 @@ env:
>> - 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="--enable-modules --disable-linux-user"
>> + - CONFIG="--with-coroutine=ucontext --disable-linux-user"
>> + - CONFIG="--with-coroutine=sigaltstack --disable-linux-user"
>
> Since this merged, the 4th, 5th & 6th jobs are now reliably completing in
> time, but the 1st and 3rd jobs are hitting timeouts on the majority of
> runs :-(
Yeah it's certainly a problem as QEMU has grown. The original
.travis.yml split things up across architecture lines but we removed
that in favour of "feature" selection.
> We've already got a lot of jobs, but for sake of reliability should we
> consider splitting the 1st & 3rd jobs. Add --disable-linux-user to both
> of the existing jobs, and then adding 2 new jobs with --disable-system
> and --disable-tools set ?
Good idea. I don't suppose you've already tried this?
>
> Regards,
> Daniel
--
Alex Bennée