[PULL 00/12] some testing and plugin updates

Alex Bennée posted 12 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211008122556.757252-1-alex.bennee@linaro.org
There is a newer version of this series
configure                                          |   6 +-
include/tcg/tcg.h                                  |   6 -
accel/tcg/plugin-gen.c                             | 157 +++++++++++----------
tests/tcg/multiarch/{ => libs}/float_helpers.c     |   2 +-
tests/tcg/multiarch/{ => linux}/linux-test.c       |   0
tests/tcg/multiarch/sha1.c                         |   1 -
.github/lockdown.yml                               |  34 -----
.github/workflows/lockdown.yml                     |  30 ++++
.gitlab-ci.d/cirrus.yml                            |   2 +-
.gitlab-ci.d/cirrus/build.yml                      |   1 +
.gitlab-ci.d/container-cross.yml                   |   3 +-
.gitlab-ci.d/crossbuilds.yml                       |  19 +++
.gitlab-ci.d/static_checks.yml                     |   2 +-
.travis.yml                                        |  23 ---
plugins/qemu-plugins.symbols                       |   3 +
tests/docker/Makefile.include                      |   6 +-
tests/docker/common.rc                             |  10 +-
tests/docker/dockerfiles/debian-native.docker      |  49 +++++++
.../docker/dockerfiles/debian-riscv64-cross.docker |  46 +++++-
tests/tcg/multiarch/Makefile.target                |  15 +-
tests/tcg/x86_64/Makefile.target                   |   4 +
21 files changed, 263 insertions(+), 156 deletions(-)
rename tests/tcg/multiarch/{ => libs}/float_helpers.c (99%)
rename tests/tcg/multiarch/{ => linux}/linux-test.c (100%)
delete mode 100644 .github/lockdown.yml
create mode 100644 .github/workflows/lockdown.yml
create mode 100644 tests/docker/dockerfiles/debian-native.docker
[PULL 00/12] some testing and plugin updates
Posted by Alex Bennée 2 years, 6 months ago
The following changes since commit 14f12119aa675e9e28207a48b0728a2daa5b88d6:

  Merge remote-tracking branch 'remotes/vsementsov/tags/pull-jobs-2021-10-07-v2' into staging (2021-10-07 10:26:35 -0700)

are available in the Git repository at:

  https://github.com/stsquad/qemu.git tags/pull-for-6.2-081021-1

for you to fetch changes up to 755c4aefd86f8b8eda1eb37f20024765c70ebbcb:

  tests/docker: add a debian-native image and make available (2021-10-08 12:55:27 +0100)

----------------------------------------------------------------
Some testing and plugin updates:

  - don't override the test compiler when specified
  - split some multiarch tests by guest OS
  - add riscv64 docker image and cross-compile tests
  - drop release tarball test from Travis
  - skip check-patch on master repo
  - fix passing of TEST_TARGETS to cirrus
  - fix missing symbols in plugins
  - refactor plugin instruction boundary detection
  - update github repo lockdown
  - add a debian-native test image for multi-arch builds

----------------------------------------------------------------
Alex Bennée (7):
      configure: don't override the selected host test compiler if defined
      tests/tcg/sha1: remove endian include
      tests/tcg: move some multiarch files and make conditional
      tests/docker: promote debian-riscv64-cross to a full image
      accel/tcg: re-factor plugin_inject_cb so we can assert insn_idx is valid
      .github: move repo lockdown to the v2 configuration
      tests/docker: add a debian-native image and make available

Daniel P. Berrangé (2):
      gitlab: skip the check-patch job on the upstream repo
      gitlab: fix passing of TEST_TARGETS env to cirrus

Lukas Jünger (1):
      plugins/: Add missing functions to symbol list

Richard Henderson (1):
      gitlab: Add cross-riscv64-system, cross-riscv64-user

Thomas Huth (1):
      travis.yml: Remove the "Release tarball" job

 configure                                          |   6 +-
 include/tcg/tcg.h                                  |   6 -
 accel/tcg/plugin-gen.c                             | 157 +++++++++++----------
 tests/tcg/multiarch/{ => libs}/float_helpers.c     |   2 +-
 tests/tcg/multiarch/{ => linux}/linux-test.c       |   0
 tests/tcg/multiarch/sha1.c                         |   1 -
 .github/lockdown.yml                               |  34 -----
 .github/workflows/lockdown.yml                     |  30 ++++
 .gitlab-ci.d/cirrus.yml                            |   2 +-
 .gitlab-ci.d/cirrus/build.yml                      |   1 +
 .gitlab-ci.d/container-cross.yml                   |   3 +-
 .gitlab-ci.d/crossbuilds.yml                       |  19 +++
 .gitlab-ci.d/static_checks.yml                     |   2 +-
 .travis.yml                                        |  23 ---
 plugins/qemu-plugins.symbols                       |   3 +
 tests/docker/Makefile.include                      |   6 +-
 tests/docker/common.rc                             |  10 +-
 tests/docker/dockerfiles/debian-native.docker      |  49 +++++++
 .../docker/dockerfiles/debian-riscv64-cross.docker |  46 +++++-
 tests/tcg/multiarch/Makefile.target                |  15 +-
 tests/tcg/x86_64/Makefile.target                   |   4 +
 21 files changed, 263 insertions(+), 156 deletions(-)
 rename tests/tcg/multiarch/{ => libs}/float_helpers.c (99%)
 rename tests/tcg/multiarch/{ => linux}/linux-test.c (100%)
 delete mode 100644 .github/lockdown.yml
 create mode 100644 .github/workflows/lockdown.yml
 create mode 100644 tests/docker/dockerfiles/debian-native.docker

-- 
2.30.2


Re: [PULL 00/12] some testing and plugin updates
Posted by Richard Henderson 2 years, 6 months ago
On 10/8/21 5:25 AM, Alex Bennée wrote:
> The following changes since commit 14f12119aa675e9e28207a48b0728a2daa5b88d6:
> 
>    Merge remote-tracking branch 'remotes/vsementsov/tags/pull-jobs-2021-10-07-v2' into staging (2021-10-07 10:26:35 -0700)
> 
> are available in the Git repository at:
> 
>    https://github.com/stsquad/qemu.git tags/pull-for-6.2-081021-1
> 
> for you to fetch changes up to 755c4aefd86f8b8eda1eb37f20024765c70ebbcb:
> 
>    tests/docker: add a debian-native image and make available (2021-10-08 12:55:27 +0100)
> 
> ----------------------------------------------------------------
> Some testing and plugin updates:
> 
>    - don't override the test compiler when specified
>    - split some multiarch tests by guest OS
>    - add riscv64 docker image and cross-compile tests
>    - drop release tarball test from Travis
>    - skip check-patch on master repo
>    - fix passing of TEST_TARGETS to cirrus
>    - fix missing symbols in plugins
>    - refactor plugin instruction boundary detection
>    - update github repo lockdown
>    - add a debian-native test image for multi-arch builds
> 
> ----------------------------------------------------------------
> Alex Bennée (7):
>        configure: don't override the selected host test compiler if defined
>        tests/tcg/sha1: remove endian include
>        tests/tcg: move some multiarch files and make conditional
>        tests/docker: promote debian-riscv64-cross to a full image
>        accel/tcg: re-factor plugin_inject_cb so we can assert insn_idx is valid
>        .github: move repo lockdown to the v2 configuration
>        tests/docker: add a debian-native image and make available
> 
> Daniel P. Berrangé (2):
>        gitlab: skip the check-patch job on the upstream repo
>        gitlab: fix passing of TEST_TARGETS env to cirrus
> 
> Lukas Jünger (1):
>        plugins/: Add missing functions to symbol list
> 
> Richard Henderson (1):
>        gitlab: Add cross-riscv64-system, cross-riscv64-user
> 
> Thomas Huth (1):
>        travis.yml: Remove the "Release tarball" job
> 
>   configure                                          |   6 +-
>   include/tcg/tcg.h                                  |   6 -
>   accel/tcg/plugin-gen.c                             | 157 +++++++++++----------
>   tests/tcg/multiarch/{ => libs}/float_helpers.c     |   2 +-
>   tests/tcg/multiarch/{ => linux}/linux-test.c       |   0
>   tests/tcg/multiarch/sha1.c                         |   1 -
>   .github/lockdown.yml                               |  34 -----
>   .github/workflows/lockdown.yml                     |  30 ++++
>   .gitlab-ci.d/cirrus.yml                            |   2 +-
>   .gitlab-ci.d/cirrus/build.yml                      |   1 +
>   .gitlab-ci.d/container-cross.yml                   |   3 +-
>   .gitlab-ci.d/crossbuilds.yml                       |  19 +++
>   .gitlab-ci.d/static_checks.yml                     |   2 +-
>   .travis.yml                                        |  23 ---
>   plugins/qemu-plugins.symbols                       |   3 +
>   tests/docker/Makefile.include                      |   6 +-
>   tests/docker/common.rc                             |  10 +-
>   tests/docker/dockerfiles/debian-native.docker      |  49 +++++++
>   .../docker/dockerfiles/debian-riscv64-cross.docker |  46 +++++-
>   tests/tcg/multiarch/Makefile.target                |  15 +-
>   tests/tcg/x86_64/Makefile.target                   |   4 +
>   21 files changed, 263 insertions(+), 156 deletions(-)
>   rename tests/tcg/multiarch/{ => libs}/float_helpers.c (99%)
>   rename tests/tcg/multiarch/{ => linux}/linux-test.c (100%)
>   delete mode 100644 .github/lockdown.yml
>   create mode 100644 .github/workflows/lockdown.yml
>   create mode 100644 tests/docker/dockerfiles/debian-native.docker

I'm seeing:

   TEST    threadcount-with-libmem.so on s390x
**
ERROR:../accel/tcg/plugin-gen.c:795:plugin_gen_inject: assertion failed: (insn_idx >= 0)
qemu: uncaught target signal 11 (Segmentation fault) - core dumped
timeout: the monitored command dumped core
Segmentation fault

on the build-user job.

https://gitlab.com/qemu-project/qemu/-/jobs/1662530140


r~