[Qemu-commits] [qemu/qemu] 6ec0a0: hw/i386/x86-iommu: Add missing stubs

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 6ec0a06366259907650cec999a10bf055828d40a
      
https://github.com/qemu/qemu/commit/6ec0a06366259907650cec999a10bf055828d40a
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M hw/i386/x86-iommu-stub.c

  Log Message:
  -----------
  hw/i386/x86-iommu: Add missing stubs

In commit 6c730e4af9 we introduced a stub to build the MicroVM
machine without Intel IOMMU. This stub is incomplete for the
other PC machines. Add the missing stubs.

Fixes: 6c730e4af9
Reported-by: Travis-CI
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Wainer dos Santos Moschetta <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>


  Commit: 22db66a436346fcccddf7617bacacdc926d2680d
      
https://github.com/qemu/qemu/commit/22db66a436346fcccddf7617bacacdc926d2680d
  Author: Gerd Hoffmann <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/vm/openbsd

  Log Message:
  -----------
  tests/vm: update openbsd to release 6.6

Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>


  Commit: bc2bf7f3480c51c3e7f38bdab1542deec765601e
      
https://github.com/qemu/qemu/commit/bc2bf7f3480c51c3e7f38bdab1542deec765601e
  Author: Gerd Hoffmann <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/vm/freebsd

  Log Message:
  -----------
  freebsd: use python37

FreeBSD seems to use python37 by default now, which breaks the build
script.  Add python to the package list, to explicitly pick the version,
and also adapt the configure command line.

Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>


  Commit: 67892c9537d9ac50093438e77aa391f8d3e75930
      
https://github.com/qemu/qemu/commit/67892c9537d9ac50093438e77aa391f8d3e75930
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis.yml: avocado: Print logs of non-pass tests only

The acceptance tests build on Travis is configured to print
the entire Avocado's job log in case any test fail. Usually one is
interested on failed tests only though. So this change the Travis
configuration in order to show the log of tests which status is
different from 'PASS' and 'SKIP' only. Note that 'CANCEL'-ed tests
will have the log printed too because it can help to debug some
condition on CI environment which is not being fulfilled.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>


  Commit: 3b35388358f7d8045028565ae4c8cddee093f101
      
https://github.com/qemu/qemu/commit/3b35388358f7d8045028565ae4c8cddee093f101
  Author: Wainer dos Santos Moschetta <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis.yml: Detach build and test steps

Currently build and test commands are a single step in a
Travis's `script` block. In order to see the output
of the tests one needs to scroll down the log to find where
the build messages ended and the limit is not clear. If
they were in different steps then Travis would print the
result build command, which can be easily grep'ed.

So this change is made to detach those commands
to ease the visualization of the output.

Note that all steps on the `script` block is executed regardless
if one previous has failed. To overcome it, let's save the
return code of the build then check whether succeed or failed on
the test step.

Signed-off-by: Wainer dos Santos Moschetta <address@hidden>
Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>


  Commit: 6e3bd8a072d79b30c842ab67a0656e3a399e92f3
      
https://github.com/qemu/qemu/commit/6e3bd8a072d79b30c842ab67a0656e3a399e92f3
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis.yml: duplicate before_script for MacOSX

Rather than keep the hack in the global code lets "Think Different"
and have a special copy for MacOSX.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: 289c8e592127eedb192d0f508f62f9f6531c0449
      
https://github.com/qemu/qemu/commit/289c8e592127eedb192d0f508f62f9f6531c0449
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  travis.yml: install homebrew python for OS X

Our python3 requirements now outstrip those of the build. While we are
at it we can move more of the special casing for Mac into the one
build we have.

Signed-off-by: Alex Bennée <address@hidden>


  Commit: 3618e3a6403079383904a1a61d53d0a57508b746
      
https://github.com/qemu/qemu/commit/3618e3a6403079383904a1a61d53d0a57508b746
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/Makefile.include
    M tests/fp/Makefile

  Log Message:
  -----------
  testing: don't nest build for fp-test

Re-calling the main make is counter-productive and really messes up
with parallel builds. Just ensure we have built the pre-requisites
before we build the fp-test bits. If the user builds manually just
complain if the parent build hasn't got the bits we need.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Emilio G. Cota <address@hidden>


  Commit: b906acbb3aceed5b1eca30d9d365d5bd7431400b
      
https://github.com/qemu/qemu/commit/b906acbb3aceed5b1eca30d9d365d5bd7431400b
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M target/arm/helper.c

  Log Message:
  -----------
  target/arm: remove unused EXCP_SEMIHOST leg

All semihosting exceptions are dealt with earlier in the common code
so we should never get here.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Reviewed-by: Keith Packard <address@hidden>
Tested-by: Keith Packard <address@hidden>


  Commit: 4ff5ef9e911c670ca10cdd36dd27c5395ec2c753
      
https://github.com/qemu/qemu/commit/4ff5ef9e911c670ca10cdd36dd27c5395ec2c753
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M linux-user/aarch64/cpu_loop.c
    M linux-user/arm/cpu_loop.c
    M target/arm/helper.c
    M target/arm/m_helper.c
    M target/arm/translate-a64.c
    M target/arm/translate.c

  Log Message:
  -----------
  target/arm: only update pc after semihosting completes

Before we introduce blocking semihosting calls we need to ensure we
can restart the system on semi hosting exception. To be able to do
this the EXCP_SEMIHOST operation should be idempotent until it finally
completes. Practically this means ensureing we only update the pc
after the semihosting call has completed.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Reviewed-by: Keith Packard <address@hidden>
Tested-by: Keith Packard <address@hidden>


  Commit: 8de702cb677c8381fb702cae252d6b69aa4c653b
      
https://github.com/qemu/qemu/commit/8de702cb677c8381fb702cae252d6b69aa4c653b
  Author: Keith Packard <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M hw/semihosting/console.c
    M include/hw/semihosting/console.h
    M include/hw/semihosting/semihost.h
    M linux-user/arm/semihost.c
    M stubs/semihost.c
    M target/arm/arm-semi.c
    M vl.c

  Log Message:
  -----------
  semihosting: add qemu_semihosting_console_inc for SYS_READC

Provides a blocking call to read a character from the console using
semihosting.chardev, if specified. This takes some careful command
line options to use stdio successfully as the serial ports, monitor
and semihost all want to use stdio. Here's a sample set of command
line options which share stdio between semihost, monitor and serial
ports:

        qemu \
        -chardev stdio,mux=on,id=stdio0 \
        -serial chardev:stdio0 \
        -semihosting-config enable=on,chardev=stdio0 \
        -mon chardev=stdio0,mode=readline

This creates a chardev hooked to stdio and then connects all of the
subsystems to it. A shorter mechanism would be good to hear about.

Signed-off-by: Keith Packard <address@hidden>
Message-Id: <address@hidden>
[AJB: fixed up deadlock, minor commit title reword]
Signed-off-by: Alex Bennée <address@hidden>
Cc: Paolo Bonzini <address@hidden>
Reviewed-by: Keith Packard <address@hidden>
Tested-by: Keith Packard <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: 9c354591cae35d91e97244469904ab4483482d34
      
https://github.com/qemu/qemu/commit/9c354591cae35d91e97244469904ab4483482d34
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/tcg/aarch64/Makefile.softmmu-target
    A tests/tcg/aarch64/system/semiconsole.c

  Log Message:
  -----------
  tests/tcg: add a dumb-as-bricks semihosting console test

We don't run this during check-tcg as we would need to check stuff is
echoed back. However we can still build the binary so people can test
it manually.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: f05938090b966e58739d98fe6c755b72234d34ba
      
https://github.com/qemu/qemu/commit/f05938090b966e58739d98fe6c755b72234d34ba
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/tcg/arm/Makefile.target
    A tests/tcg/arm/semicall.h
    M tests/tcg/arm/semihosting.c

  Log Message:
  -----------
  tests/tcg: extract __semi_call into a header and expand

There are two types of ARM semicall - lets test them both. Putting the
logic in a header will make re-using the functions easier later.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: 486e58b188c1b093a8f64d4b5cd11ff5c3514cb2
      
https://github.com/qemu/qemu/commit/486e58b188c1b093a8f64d4b5cd11ff5c3514cb2
  Author: Alex Bennée <address@hidden>
  Date:   2020-01-09 (Thu, 09 Jan 2020)

  Changed paths:
    M tests/tcg/aarch64/Makefile.target
    M tests/tcg/arm/Makefile.target
    A tests/tcg/arm/semiconsole.c

  Log Message:
  -----------
  tests/tcg: add user version of dumb-as-bricks semiconsole test

There are linux-user users of semihosting so we'd better check things
work for them as well.

Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: f38a71b01f839c7b65ea73ddd507903cb9489ed6
      
https://github.com/qemu/qemu/commit/f38a71b01f839c7b65ea73ddd507903cb9489ed6
  Author: Peter Maydell <address@hidden>
  Date:   2020-01-10 (Fri, 10 Jan 2020)

  Changed paths:
    M .travis.yml
    M hw/i386/x86-iommu-stub.c
    M hw/semihosting/console.c
    M include/hw/semihosting/console.h
    M include/hw/semihosting/semihost.h
    M linux-user/aarch64/cpu_loop.c
    M linux-user/arm/cpu_loop.c
    M linux-user/arm/semihost.c
    M stubs/semihost.c
    M target/arm/arm-semi.c
    M target/arm/helper.c
    M target/arm/m_helper.c
    M target/arm/translate-a64.c
    M target/arm/translate.c
    M tests/Makefile.include
    M tests/fp/Makefile
    M tests/tcg/aarch64/Makefile.softmmu-target
    M tests/tcg/aarch64/Makefile.target
    A tests/tcg/aarch64/system/semiconsole.c
    M tests/tcg/arm/Makefile.target
    A tests/tcg/arm/semicall.h
    A tests/tcg/arm/semiconsole.c
    M tests/tcg/arm/semihosting.c
    M tests/vm/freebsd
    M tests/vm/openbsd
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/stsquad/tags/pull-testing-and-semihosting-090120-2' into staging

Testing fixes and semiconsole support:

  - build fix (missing x86-iommu stubs)
  - python fixes for freebsd and OSX
  - nicer reporting of acceptance failures
  - fix build nesting of fp-test (breaks bsds)
  - semihosting clean-ups
  - support for blocking semihosting console

# gpg: Signature made Thu 09 Jan 2020 11:42:51 GMT
# gpg:                using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) <address@hidden>" 
[full]
# Primary key fingerprint: 6685 AE99 E751 67BC AFC8  DF35 FBD0 DB09 5A9E 2A44

* remotes/stsquad/tags/pull-testing-and-semihosting-090120-2:
  tests/tcg: add user version of dumb-as-bricks semiconsole test
  tests/tcg: extract __semi_call into a header and expand
  tests/tcg: add a dumb-as-bricks semihosting console test
  semihosting: add qemu_semihosting_console_inc for SYS_READC
  target/arm: only update pc after semihosting completes
  target/arm: remove unused EXCP_SEMIHOST leg
  testing: don't nest build for fp-test
  travis.yml: install homebrew python for OS X
  travis.yml: duplicate before_script for MacOSX
  travis.yml: Detach build and test steps
  travis.yml: avocado: Print logs of non-pass tests only
  freebsd: use python37
  tests/vm: update openbsd to release 6.6
  hw/i386/x86-iommu: Add missing stubs

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/0dba4897be3a...f38a71b01f83