All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: "Alex Bennée" <alex.bennee@linaro.org>
Cc: cota@braap.org, berrange@redhat.com, f4bug@amsat.org,
	richard.henderson@linaro.org, balrogg@gmail.com,
	aurelien@aurel32.net, agraf@suse.de, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH for 3.0 00/10] various docker fixes
Date: Tue, 10 Jul 2018 15:06:28 +0800	[thread overview]
Message-ID: <20180710070628.GN17581@lemon.usersys.redhat.com> (raw)
In-Reply-To: <20180709152117.21585-1-alex.bennee@linaro.org>

On Mon, 07/09 16:21, Alex Bennée wrote:
> Hi,
> 
> The addition of the cross compilers for check-tcg broke the ability to
> run "make docker-test".  In truth this was probably already broken as
> it attempts to run every single test with every single docker image
> which isn't something that gets done on a regular basis.
> 
> Anyway the patches clean up the ability to do that in a sane way
> although it still takes a long time to run the full test set.
> 
> To help with running a better subset I've expanded the individual
> tests so you can now run a line like:
> 
>   make docker-test-build TARGET_LIST=aarch64-softmmu J=30
> 
> To make sure your favourite architecture still builds everywhere.
> 
> There is now also a docker-test-unit which just runs the unit tests
> although we have to do a little re-factoring to make sure we don't
> attempt to run "make check" steps when the docker image isn't capable
> of it. This is also needed to make sure the other test-FOO build tests
> don't choke on the check step.
> 
> Finally there is a minor tweak for .gitignore and a fix for docker.py
> throwing backtraces when we attempt to calculate SID_AGE. A cleaner
> re-factoring can be left for a future release.

Thanks for the clean ups!

Looks good in general. The only thing I notice is the test-unit script.

Fam

      parent reply	other threads:[~2018-07-10  7:06 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-09 15:21 [Qemu-devel] [PATCH for 3.0 00/10] various docker fixes Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 01/10] tests/.gitignore: don't ignore docker tests Alex Bennée
2018-07-10  6:57   ` Fam Zheng
2018-07-10  7:54     ` Alex Bennée
2018-07-10  8:36       ` Fam Zheng
2018-07-10 20:50   ` Philippe Mathieu-Daudé
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 02/10] docker: debian-tricore add git tools Alex Bennée
2018-07-10 20:44   ` Philippe Mathieu-Daudé
2018-07-11  0:07     ` Philippe Mathieu-Daudé
2018-07-12 10:44     ` Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 03/10] docker: fail more gracefully on docker.py check Alex Bennée
2018-07-10 20:44   ` Philippe Mathieu-Daudé
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 04/10] docker: split configure_qemu from build_qemu Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 05/10] docker: move make check into check_qemu helper Alex Bennée
2018-07-10  6:59   ` Fam Zheng
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 06/10] docker: gracefully skip check_qemu Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 07/10] docker: Makefile.include don't include partial images Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 08/10] docker: disable debian-powerpc-user-cross Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 09/10] docker: add test-unit runner Alex Bennée
2018-07-10  7:04   ` Fam Zheng
2018-07-12 10:20     ` Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 10/10] docker: add expansion for docker-test-FOO to Makefile.include Alex Bennée
2018-07-10 20:49   ` Philippe Mathieu-Daudé
2018-07-10 21:04     ` Alex Bennée
2018-07-11  1:31       ` Fam Zheng
2018-07-12  8:41         ` Alex Bennée
2018-07-10  7:06 ` Fam Zheng [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180710070628.GN17581@lemon.usersys.redhat.com \
    --to=famz@redhat.com \
    --cc=agraf@suse.de \
    --cc=alex.bennee@linaro.org \
    --cc=aurelien@aurel32.net \
    --cc=balrogg@gmail.com \
    --cc=berrange@redhat.com \
    --cc=cota@braap.org \
    --cc=f4bug@amsat.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.