mbox series

[PULL,for,6.2,00/21] testing, plugin and gdbstub updates

Message ID 20211103170558.717981-1-alex.bennee@linaro.org
Headers show
Series testing, plugin and gdbstub updates | expand

Message

Alex Bennée Nov. 3, 2021, 5:05 p.m. UTC
The following changes since commit e86e00a2493254d072581960b48461eb96481e45:

  Merge remote-tracking branch 'remotes/berrange/tags/hmp-x-qmp-620-pull-request' into staging (2021-11-03 08:04:32 -0400)

are available in the Git repository at:

  https://github.com/stsquad/qemu.git tags/pull-for-6.2-031121-2

for you to fetch changes up to 8b964f7f800eda16c489a7a8a0f1d88da1c34c1b:

  tests/vm/openbsd: Update to release 7.0 (2021-11-03 14:54:21 +0000)

----------------------------------------------------------------
Testing, gdbstub and plugin updates for 6.2

 - update hexagon compiler to binary release
 - add microblaze and nios2 compiler docker images
 - fix test cross compiler detection for some targets
 - don't try and link ebf to user targets
 - add L2 tracking to cache plugin
 - exit cleanly on C-a x
 - clean up debug output in check-tcg
 - switch to thread on break in gdbstub
 - update openbsd VM to 7.0

----------------------------------------------------------------
Alex Bennée (7):
      tests/docker: split PARTIAL into PARTIAL and VIRTUAL images
      tests/tcg: enable debian-nios2-cross for test building
      chardev: don't exit() straight away on C-a x
      tests/plugins: extend the insn plugin to track opcode sizes
      plugins: try and make plugin_insn_append more ergonomic
      tests/tcg: remove duplicate EXTRA_RUNS
      tests/tcg: remove debug polluting make output

Mahmoud Mandour (5):
      plugins/cache: freed heap-allocated mutexes
      plugins/cache: implement unified L2 cache emulation
      plugins/cache: split command line arguments into name and value
      plugins/cache: make L2 emulation optional through args
      docs/tcg-plugins: add L2 arguments to cache docs

Paolo Bonzini (1):
      ebpf: really include it only in system emulators

Pavel Labath (1):
      gdbstub: Switch to the thread receiving a signal

Philippe Mathieu-Daudé (1):
      tests/tcg: Fix some targets default cross compiler path

Richard Henderson (6):
      tests/docker: Update debian-hexagon-cross to a newer toolchain
      gitlab-ci: Remove special casing for hexagon testing
      tests/docker: Add debian-nios2-cross image
      tests/docker: Add debian-microblaze-cross image
      tests/tcg: Enable container_cross_cc for microblaze
      tests/vm/openbsd: Update to release 7.0

 docs/devel/tcg-plugins.rst                         |  20 +-
 meson.build                                        |   2 -
 include/exec/plugin-gen.h                          |  12 +-
 include/qemu/plugin.h                              |   7 +-
 accel/tcg/plugin-gen.c                             |   3 +-
 accel/tcg/translator.c                             |   2 +-
 chardev/char-mux.c                                 |   3 +-
 contrib/plugins/cache.c                            | 318 +++++++++++++++------
 gdbstub.c                                          |   8 +-
 stubs/qmp-quit.c                                   |   8 +
 tests/plugin/insn.c                                |  37 ++-
 .gitlab-ci.d/buildtest.yml                         |   4 -
 .gitlab-ci.d/container-cross.yml                   |  27 +-
 MAINTAINERS                                        |   4 +-
 stubs/meson.build                                  |   1 +
 tests/docker/Makefile.include                      |  59 ++--
 .../docker/dockerfiles/debian-hexagon-cross.docker |  47 +--
 .../build-toolchain.sh                             | 141 ---------
 .../debian-microblaze-cross.d/build-toolchain.sh   |  88 ++++++
 .../debian-nios2-cross.d/build-toolchain.sh        |  87 ++++++
 tests/docker/dockerfiles/debian-toolchain.docker   |  36 +++
 tests/tcg/configure.sh                             |  26 +-
 tests/tcg/hexagon/Makefile.target                  |   6 +
 tests/tcg/multiarch/Makefile.target                |  13 +-
 .../multiarch/gdbstub/test-thread-breakpoint.py    |  60 ++++
 tests/tcg/nios2/Makefile.target                    |  11 +
 tests/tcg/sh4/Makefile.target                      |   6 +
 tests/vm/openbsd                                   |   7 +-
 28 files changed, 688 insertions(+), 355 deletions(-)
 create mode 100644 stubs/qmp-quit.c
 delete mode 100755 tests/docker/dockerfiles/debian-hexagon-cross.docker.d/build-toolchain.sh
 create mode 100755 tests/docker/dockerfiles/debian-microblaze-cross.d/build-toolchain.sh
 create mode 100755 tests/docker/dockerfiles/debian-nios2-cross.d/build-toolchain.sh
 create mode 100644 tests/docker/dockerfiles/debian-toolchain.docker
 create mode 100644 tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py
 create mode 100644 tests/tcg/nios2/Makefile.target

-- 
2.30.2

Comments

Alex Bennée Nov. 3, 2021, 5:20 p.m. UTC | #1
Alex Bennée <alex.bennee@linaro.org> writes:

> The following changes since commit e86e00a2493254d072581960b48461eb96481e45:

>

>   Merge remote-tracking branch 'remotes/berrange/tags/hmp-x-qmp-620-pull-request' into staging (2021-11-03 08:04:32 -0400)

>

<snip>
>       tests/docker: Update debian-hexagon-cross to a newer toolchain


Hmm it seems like we can blame the toolchain:

  Subject: Hexagon toolchain update vs linux-user signals
  From: Richard Henderson <richard.henderson@linaro.org>

  Message-ID: <4f65337b-b9ed-dc4c-ac09-025bef5eaa4c@linaro.org>
  Date: Wed, 3 Nov 2021 11:22:03 -0400

NACK this PR and I'll roll v2 tomorrow....

-- 
Alex Bennée
Taylor Simpson Nov. 3, 2021, 7:32 p.m. UTC | #2
We'll investigate the toolchain issue.  This is compiling tests/tcg/multiarch/signals.c, right?

Also, I'll do as Richard suggests in his email and respin the pull with the HVX tests disabled in the makefile.

Thanks,
Taylor


> -----Original Message-----

> From: Alex Bennée <alex.bennee@linaro.org>

> Sent: Wednesday, November 3, 2021 12:21 PM

> To: peter.maydell@linaro.org

> Cc: richard.henderson@linaro.org; qemu-devel@nongnu.org; Alex Bennée

> <alex.bennee@linaro.org>; Taylor Simpson <tsimpson@quicinc.com>

> Subject: Re: [PULL for 6.2 00/21] testing, plugin and gdbstub updates

> 

> WARNING: This email originated from outside of Qualcomm. Please be wary

> of any links or attachments, and do not enable macros.

> 

> Alex Bennée <alex.bennee@linaro.org> writes:

> 

> > The following changes since commit

> e86e00a2493254d072581960b48461eb96481e45:

> >

> >   Merge remote-tracking branch 'remotes/berrange/tags/hmp-x-qmp-620-

> pull-request' into staging (2021-11-03 08:04:32 -0400)

> >

> <snip>

> >       tests/docker: Update debian-hexagon-cross to a newer toolchain

> 

> Hmm it seems like we can blame the toolchain:

> 

>   Subject: Hexagon toolchain update vs linux-user signals

>   From: Richard Henderson <richard.henderson@linaro.org>

>   Message-ID: <4f65337b-b9ed-dc4c-ac09-025bef5eaa4c@linaro.org>

>   Date: Wed, 3 Nov 2021 11:22:03 -0400

> 

> NACK this PR and I'll roll v2 tomorrow....

> 

> --

> Alex Bennée
Richard Henderson Nov. 4, 2021, 4:45 a.m. UTC | #3
On 11/3/21 3:32 PM, Taylor Simpson wrote:
> We'll investigate the toolchain issue.  This is compiling tests/tcg/multiarch/signals.c, right?


Technically, yes.  But the error I identified is within the packaged musl libc.a.


r~