On 27/11/2019 19.38, Alex Bennée wrote:

Thomas Huth <th...@redhat.com> writes:

So far we only have compile coverage for tci. But since commit
2f160e0f9797c7522bfd0d09218d0c9340a5137c ("tci: Add implementation
for INDEX_op_ld16u_i64") has been included, we can also run the
x86 TCG tests with tci, so let's enable them in Travis now.

Signed-off-by: Thomas Huth <th...@redhat.com>
---
  .travis.yml | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index c09b6a0014..b0b634d484 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -218,7 +218,7 @@ matrix:
      # We manually include builds which we disable "make check" for

this comment is out of date now (or rather has been for a while)

      - env:
          - CONFIG="--enable-debug --enable-tcg-interpreter"

Perhaps as linux-user doesn't add much to testing the TCG we could limit
by MAIN_SOFTMMU_TARGETS here?

Limiting is a good idea, but I think we should use the architectures that are supported by tests/boot-serial-test.c instead.

-        - TEST_CMD=""
+        - TEST_CMD="make run-tcg-tests-x86_64-softmmu V=1"

How about "make check-qtest check-tcg"
> Which will exercise the moderate boot code tests of various
architectures as well as x86_64 (if we include the --disable-docker
configure stanza)

I tried "check-tcg" before switching to "run-tcg-tests-x86_64-softmmu", but "check-tcg" then resulted in a docker error:

 https://travis-ci.com/huth/qemu/jobs/260949876#L11974

... so looks like we indeed have to use --disable-docker in that case.

I'll have a try and send a v2 if that works out fine.

 Thomas


Reply via email to