qemu-devel
[Top][All Lists]
Advanced

[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: Daniel P . Berrangé
Subject: Re: [Qemu-devel] [PULL 1/1] .travis.yml: add --disable-linux-user for some jobs
Date: Tue, 13 Mar 2018 14:04:26 +0000
User-agent: Mutt/1.9.2 (2017-12-15)

On Tue, Mar 13, 2018 at 02:02:54PM +0000, Alex Bennée wrote:
> 
> 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?

No, I've not gone beyond quickly writing this previous mail  :-)

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



reply via email to

[Prev in Thread] Current Thread [Next in Thread]