| Message ID | 20211103170558.717981-1-alex.bennee@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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....
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
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~
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