[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 :|