[Qemu-commits] [qemu/qemu] 36a7ab: tests/acceptance: Increase the timeout in the repl...

[Peter Maydell]

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 36a7ab5f04be1d7485b93e0b76eb0ccc5af81248
      
https://github.com/qemu/qemu/commit/36a7ab5f04be1d7485b93e0b76eb0ccc5af81248
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/acceptance/replay_kernel.py

  Log Message:
  -----------
  tests/acceptance: Increase the timeout in the replay tests

Our gitlab-CI just showed a failed test_ppc_mac99 since it was apparently
killed some few seconds before the test finished. Allow it some more
time to complete.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Acked-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210127065222.48650-1-thuth@redhat.com>


  Commit: 4d8f630915c0b3d5de430c10797d0be1c12a2b2c
      
https://github.com/qemu/qemu/commit/4d8f630915c0b3d5de430c10797d0be1c12a2b2c
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/docker/docker.py

  Log Message:
  -----------
  tests/docker: Fix _get_so_libs() for docker-binfmt-image

Fix a variable rename mistake from commit 5e33f7fead5:

  Traceback (most recent call last):
    File "./tests/docker/docker.py", line 710, in <module>
      sys.exit(main())
    File "./tests/docker/docker.py", line 706, in main
      return args.cmdobj.run(args, argv)
    File "./tests/docker/docker.py", line 489, in run
      _copy_binary_with_libs(args.include_executable,
    File "./tests/docker/docker.py", line 149, in _copy_binary_with_libs
      libs = _get_so_libs(src)
    File "./tests/docker/docker.py", line 123, in _get_so_libs
      libs.append(s.group(1))
  NameError: name 's' is not defined

Fixes: 5e33f7fead5 ("tests/docker: better handle symlinked libs")
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210119050149.516910-1-f4bug@amsat.org>
Message-Id: <20210202134001.25738-2-alex.bennee@linaro.org>


  Commit: dc23bbc3df448d91727535067a05f686b8b29e63
      
https://github.com/qemu/qemu/commit/dc23bbc3df448d91727535067a05f686b8b29e63
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/docker/Makefile.include

  Log Message:
  -----------
  tests/docker: Fix typo in help message

To have the variable properly passed, we need to set it,
ie. NOUSER=1. Fix the message displayed by 'make docker'.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210119052120.522069-1-f4bug@amsat.org>
Message-Id: <20210202134001.25738-3-alex.bennee@linaro.org>


  Commit: dffccf3d34467d7280212445e521f458d31ac6a2
      
https://github.com/qemu/qemu/commit/dffccf3d34467d7280212445e521f458d31ac6a2
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/docker/docker.py

  Log Message:
  -----------
  tests/docker: make _copy_with_mkdir accept missing files

Depending on the linker/ldd setup we might get a file with no path.
Typically this is the psuedo library linux-vdso.so which doesn't
actually exist on the disk. Rather than try and catch these distro
specific edge cases just shout about it and try and continue.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20210202134001.25738-4-alex.bennee@linaro.org>


  Commit: 3971c70f159e00fa0a311b0a39c3baa1f2d814f4
      
https://github.com/qemu/qemu/commit/3971c70f159e00fa0a311b0a39c3baa1f2d814f4
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/docker/docker.py

  Log Message:
  -----------
  tests/docker: preserve original name when copying libs

While it is important we chase down the symlinks to copy the correct
data we can confuse the kernel by renaming the interpreter to what is
in the binary. Extend _copy_with_mkdir to preserve the original name
of the file when asked.

Fixes: 5e33f7fead ("tests/docker: better handle symlinked libs")
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210202134001.25738-5-alex.bennee@linaro.org>


  Commit: 6147c2495d1a362cb78cdbd6a321ddae31e13f9d
      
https://github.com/qemu/qemu/commit/6147c2495d1a362cb78cdbd6a321ddae31e13f9d
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

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

  Log Message:
  -----------
  tests/docker: alias docker-help target for consistency

We have a bunch of -help targets so this will save some cognitive
dissonance. Keep the original for those with muscle memory.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20210202134001.25738-6-alex.bennee@linaro.org>


  Commit: ddd5ed8331652cd77546c331caee49d76fffe4a4
      
https://github.com/qemu/qemu/commit/ddd5ed8331652cd77546c331caee49d76fffe4a4
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/docker/Makefile.include
    M tests/docker/docker.py
    A tests/docker/dockerfiles/empty.docker

  Log Message:
  -----------
  tests/docker: add a docker-exec-copy-test

This provides test machinery for checking the QEMU copying logic works
properly. It takes considerably less time to run than starting a
debootstrap only for it to fail later. I considered adding a remove
command to docker.py but figured that might be gold plating given the
relative size of the containers compared to the ones with actual stuff
in them.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210202134001.25738-7-alex.bennee@linaro.org>


  Commit: 2df52b9bfdf0c4f4fc4103e837d6f59488832795
      
https://github.com/qemu/qemu/commit/2df52b9bfdf0c4f4fc4103e837d6f59488832795
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: make version_ge more tolerant of shady version input

When checking GDB versions we have to tolerate all sorts of random
distro extensions to the version string. While we already attempt to
do some of that before we call version_ge is makes sense to try and
regularise the first input by stripping extraneous -'s. While we at it
convert the old-style shell quoting into a cleaner form t shut up my
editors linter lest it confuse me by underlining the whole line.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20210202134001.25738-8-alex.bennee@linaro.org>


  Commit: d6a66c811edacd59b7b414474f30cc5a4dce32a1
      
https://github.com/qemu/qemu/commit/d6a66c811edacd59b7b414474f30cc5a4dce32a1
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: bump the minimum gdb version for check-tcg to 9.1

For SVE, currently the bulk of the GDB TCG tests, we need at least GDB
9.1 to support the "ieee_half" data type we report. This only affects
when GDB tests are run; users can still use lower versions of gdb as
long as they aren't talking to an SVE enabled model. The work around
is to either get a newer gdb or disable SVE for their CPU model.

Reported-by: Claudio Fontana <cfontana@suse.de>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Luis Machado <luis.machado@linaro.org>
Message-Id: <20210202134001.25738-9-alex.bennee@linaro.org>


  Commit: 46bae04a860ebfe331fc6147ed20bcb8d361755d
      
https://github.com/qemu/qemu/commit/46bae04a860ebfe331fc6147ed20bcb8d361755d
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/tcg/multiarch/Makefile.target

  Log Message:
  -----------
  tests/tcg: don't silently skip the gdb tests

Otherwise people won't know what they are missing.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210202134001.25738-10-alex.bennee@linaro.org>


  Commit: 6e3dd7571705e7a396b21a39894903083bfb23f9
      
https://github.com/qemu/qemu/commit/6e3dd7571705e7a396b21a39894903083bfb23f9
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M gdbstub.c

  Log Message:
  -----------
  gdbstub: Fix handle_query_xfer_auxv

The main problem was that we were treating a guest address
as a host address with a mere cast.

Use the correct interface for accessing guest memory.  Do not
allow offset == auxv_len, which would result in an empty packet.

Fixes: 51c623b0de1 ("gdbstub: add support to Xfer:auxv:read: packet")
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210128201831.534033-1-richard.henderson@linaro.org>
Message-Id: <20210202134001.25738-11-alex.bennee@linaro.org>


  Commit: 2a86d66be1a33c2db25e4774def415784bf44131
      
https://github.com/qemu/qemu/commit/2a86d66be1a33c2db25e4774def415784bf44131
  Author: Stefan Weil <sw@weilnetz.de>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/tcg/Makefile.qemu

  Log Message:
  -----------
  tests/tcg: Replace /bin/true by true (required on macOS)

/bin/true is missing on macOS, but simply "true" is available as a shell 
builtin.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20210128135627.2067003-1-sw@weilnetz.de>
Message-Id: <20210202134001.25738-12-alex.bennee@linaro.org>


  Commit: 47e3424ac97fdc3d11186cf15a686dfee8e8777a
      
https://github.com/qemu/qemu/commit/47e3424ac97fdc3d11186cf15a686dfee8e8777a
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M scripts/mtest2make.py
    M tests/Makefile.include

  Log Message:
  -----------
  scripts/mtest2make.py: export all-%s-targets variable and use it

There are some places where the conditional makefile support is the
simplest solution. Now we don't expose CONFIG_TCG as a variable create
a new one that can be checked for the check-help output.

As check-tcg is a PHONY target we re-use check-softfloat to gate that
as well.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210202134001.25738-13-alex.bennee@linaro.org>


  Commit: c401c058a1cdb131163751f29df9151290d842b0
      
https://github.com/qemu/qemu/commit/c401c058a1cdb131163751f29df9151290d842b0
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M tests/Makefile.include

  Log Message:
  -----------
  tests/Makefile.include: don't use TARGET_DIRS for check-tcg

TARGET_DIRS reflects what we wanted to configure which in the normal
case is all our targets. However once meson has pared-down our target
list due to missing features we need to check the final list of
ninja-targets. This prevents check-tcg barfing on a --disable-tcg
build.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20210202134001.25738-14-alex.bennee@linaro.org>


  Commit: a5dbb175077c93c8d2e1021757f8b9a593fc2b5e
      
https://github.com/qemu/qemu/commit/a5dbb175077c93c8d2e1021757f8b9a593fc2b5e
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M docs/system/arm/vexpress.rst

  Log Message:
  -----------
  docs/system: document an example vexpress-a15 invocation

The wiki and the web are curiously absent of the right runes to boot a
vexpress model so I had to work from first principles to work it out.
Use the more modern -drive notation so alternative backends can be
used (unlike the hardwired -sd mode).

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Anders Roxell <anders.roxell@linaro.org>
Message-Id: <20210202134001.25738-15-alex.bennee@linaro.org>


  Commit: d994cc54498e8952113110b07a76dbfecd46a909
      
https://github.com/qemu/qemu/commit/d994cc54498e8952113110b07a76dbfecd46a909
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M docs/system/arm/versatile.rst

  Log Message:
  -----------
  docs/system: document an example booting the versatilepb machine

There is a bit more out there including Aurelien's excellent write up
and older Debian images here:

  https://www.aurel32.net/info/debian_arm_qemu.php
  https://people.debian.org/~aurel32/qemu/armel/

However the web is transitory and git is forever so lets add something
to the fine manual.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Anders Roxell <anders.roxell@linaro.org>
Cc: Aurelien Jarno <aurelien@aurel32.net>
Message-Id: <20210202134001.25738-16-alex.bennee@linaro.org>


  Commit: 8eef07b4d3c2f1048012f6a5174671bc167b4604
      
https://github.com/qemu/qemu/commit/8eef07b4d3c2f1048012f6a5174671bc167b4604
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M Makefile
    M configure
    M docs/system/arm/versatile.rst
    M docs/system/arm/vexpress.rst
    M gdbstub.c
    M scripts/mtest2make.py
    M tests/Makefile.include
    M tests/acceptance/replay_kernel.py
    M tests/docker/Makefile.include
    M tests/docker/docker.py
    A tests/docker/dockerfiles/empty.docker
    M tests/tcg/Makefile.qemu
    M tests/tcg/multiarch/Makefile.target

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/stsquad/tags/pull-testing-gdbstub-docs-080221-1' into staging

Testing, gdbstub and doc tweaks:

  - increase timeout on replay kernel acceptance test
  - fixes for binfmt_misc docker images
  - better gdb version detection
  - don't silently skip gdb tests
  - fix for gdbstub auxv handling
  - cleaner handling of check-tcg on tcg disabled builds
  - expand vexpress/versitile docs with examples

# gpg: Signature made Mon 08 Feb 2021 11:12:03 GMT
# gpg:                using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44
# gpg: Good signature from "Alex Bennée (Master Work Key) 
<alex.bennee@linaro.org>" [full]
# Primary key fingerprint: 6685 AE99 E751 67BC AFC8  DF35 FBD0 DB09 5A9E 2A44

* remotes/stsquad/tags/pull-testing-gdbstub-docs-080221-1:
  docs/system: document an example booting the versatilepb machine
  docs/system: document an example vexpress-a15 invocation
  tests/Makefile.include: don't use TARGET_DIRS for check-tcg
  scripts/mtest2make.py: export all-%s-targets variable and use it
  tests/tcg: Replace /bin/true by true (required on macOS)
  gdbstub: Fix handle_query_xfer_auxv
  tests/tcg: don't silently skip the gdb tests
  configure: bump the minimum gdb version for check-tcg to 9.1
  configure: make version_ge more tolerant of shady version input
  tests/docker: add a docker-exec-copy-test
  tests/docker: alias docker-help target for consistency
  tests/docker: preserve original name when copying libs
  tests/docker: make _copy_with_mkdir accept missing files
  tests/docker: Fix typo in help message
  tests/docker: Fix _get_so_libs() for docker-binfmt-image
  tests/acceptance: Increase the timeout in the replay tests

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/6f0e9c26dbae...8eef07b4d3c2