[PULL 00/20] CI, qtest, and misc patches

Thomas Huth posted 20 patches 2 years, 11 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210514121518.832729-1-thuth@redhat.com
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Thomas Huth <thuth@redhat.com>, Li-Wen Hsu <lwhsu@freebsd.org>, Willian Rampazzo <willianr@redhat.com>, Tyrone Ting <kfting@nuvoton.com>, "Alex Bennée" <alex.bennee@linaro.org>, Juan Quintela <quintela@redhat.com>, Cornelia Huck <cohuck@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Havard Skinnemoen <hskinnemoen@google.com>, Ed Maste <emaste@freebsd.org>, Yonggang Luo <luoyonggang@gmail.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>
.cirrus.yml                                   |   4 +-
.gitlab-ci.d/containers.yml                   |  76 ++++++-------
.gitlab-ci.yml                                | 104 +++++++++---------
Makefile                                      |   2 +-
configure                                     |   7 ++
docs/devel/qgraph.rst                         |  58 ++++++++++
include/exec/poison.h                         |   6 +
include/sysemu/hax.h                          |   4 +
include/sysemu/hvf.h                          |   4 +
include/sysemu/whpx.h                         |   4 +
migration/meson.build                         |   3 +-
migration/migration.c                         |  15 ---
migration/migration.h                         |   2 +
migration/target.c                            |  25 +++++
pc-bios/s390-ccw/helper.h                     |   2 +-
pc-bios/s390-ccw/jump2ipl.c                   |   4 +-
pc-bios/s390-ccw/menu.c                       |   8 +-
pc-bios/s390-ccw/virtio.c                     |   2 +-
tests/docker/dockerfiles/alpine.docker        |   1 +
.../dockerfiles/fedora-i386-cross.docker      |   1 +
.../dockerfiles/fedora-win32-cross.docker     |   1 +
.../dockerfiles/fedora-win64-cross.docker     |   1 +
tests/docker/dockerfiles/opensuse-leap.docker |   1 +
tests/qtest/ahci-test.c                       |   4 +-
tests/qtest/ipmi-bt-test.c                    |   6 +-
tests/qtest/ipmi-kcs-test.c                   |   3 +-
tests/qtest/libqos/qgraph.c                   |   2 +-
tests/qtest/libqtest.c                        |   9 +-
tests/qtest/migration-test.c                  |  75 +++++--------
tests/qtest/npcm7xx_pwm-test.c                |   4 +-
tests/qtest/rtc-test.c                        |   6 +-
tests/qtest/tpm-util.c                        |   4 +-
util/compatfd.c                               |   8 +-
33 files changed, 270 insertions(+), 186 deletions(-)
create mode 100644 migration/target.c
[PULL 00/20] CI, qtest, and misc patches
Posted by Thomas Huth 2 years, 11 months ago
 Hi Peter!

The following changes since commit 2d3fc4e2b069494b1e9e2e4a1e3de24cbc036426:

  Merge remote-tracking branch 'remotes/armbru/tags/pull-misc-2021-05-12' into staging (2021-05-13 20:13:24 +0100)

are available in the Git repository at:

  https://gitlab.com/thuth/qemu.git tags/pull-request-2021-05-14

for you to fetch changes up to 568740bedf22cc8d0ec9ab1ce522a97baab5961c:

  cirrus.yml: Fix the MSYS2 task (2021-05-14 13:11:48 +0200)

----------------------------------------------------------------
* Replace YAML anchors by extends in the gitlab-CI yaml files
* Many small qtest fixes (e.g. to fix issues discovered by Coverity)
* Poison more config switches in common code
* Fix the failing Travis-CI and Cirrus-CI tasks
----------------------------------------------------------------

Note: This PR replaces my PR from May 3rd - the moxie removal has
      already been merged via Markus' tree, and I addes some other
      CI- and qtest related patches instead.

Dr. David Alan Gilbert (1):
  tests/migration-test: Fix "true" vs true

Mahmoud Mandour (1):
  util/compatfd.c: Replaced a malloc call with g_malloc.

Peter Maydell (6):
  tests/qtest/ahci-test.c: Calculate iso_size with 64-bit arithmetic
  tests/qtest/npcm7xx_pwm-test.c: Avoid g_assert_true() for non-test
    assertions
  tests/qtest/tpm-util.c: Free memory with correct free function
  tests/qtest/rtc-test: Remove pointless NULL check
  tests: Avoid side effects inside g_assert() arguments
  tests/qtest/migration-test: Use g_autofree to avoid leaks on error
    paths

Philippe Mathieu-Daudé (3):
  gitlab-ci: Replace YAML anchors by extends (container_job)
  gitlab-ci: Replace YAML anchors by extends (native_build_job)
  gitlab-ci: Replace YAML anchors by extends (native_test_job)

Stefan Hajnoczi (3):
  libqos/qgraph: fix "UNAVAILBLE" typo
  docs/devel/qgraph: add troubleshooting information
  libqtest: refuse QTEST_QEMU_BINARY=qemu-kvm

Thomas Huth (6):
  tests/docker/dockerfiles: Add ccache to containers where it was
    missing
  include/sysemu: Poison all accelerator CONFIG switches in common code
  migration: Move populate_vfio_info() into a separate file
  configure: Poison all current target-specific #defines
  pc-bios/s390-ccw: Fix inline assembly for older versions of Clang
  cirrus.yml: Fix the MSYS2 task

 .cirrus.yml                                   |   4 +-
 .gitlab-ci.d/containers.yml                   |  76 ++++++-------
 .gitlab-ci.yml                                | 104 +++++++++---------
 Makefile                                      |   2 +-
 configure                                     |   7 ++
 docs/devel/qgraph.rst                         |  58 ++++++++++
 include/exec/poison.h                         |   6 +
 include/sysemu/hax.h                          |   4 +
 include/sysemu/hvf.h                          |   4 +
 include/sysemu/whpx.h                         |   4 +
 migration/meson.build                         |   3 +-
 migration/migration.c                         |  15 ---
 migration/migration.h                         |   2 +
 migration/target.c                            |  25 +++++
 pc-bios/s390-ccw/helper.h                     |   2 +-
 pc-bios/s390-ccw/jump2ipl.c                   |   4 +-
 pc-bios/s390-ccw/menu.c                       |   8 +-
 pc-bios/s390-ccw/virtio.c                     |   2 +-
 tests/docker/dockerfiles/alpine.docker        |   1 +
 .../dockerfiles/fedora-i386-cross.docker      |   1 +
 .../dockerfiles/fedora-win32-cross.docker     |   1 +
 .../dockerfiles/fedora-win64-cross.docker     |   1 +
 tests/docker/dockerfiles/opensuse-leap.docker |   1 +
 tests/qtest/ahci-test.c                       |   4 +-
 tests/qtest/ipmi-bt-test.c                    |   6 +-
 tests/qtest/ipmi-kcs-test.c                   |   3 +-
 tests/qtest/libqos/qgraph.c                   |   2 +-
 tests/qtest/libqtest.c                        |   9 +-
 tests/qtest/migration-test.c                  |  75 +++++--------
 tests/qtest/npcm7xx_pwm-test.c                |   4 +-
 tests/qtest/rtc-test.c                        |   6 +-
 tests/qtest/tpm-util.c                        |   4 +-
 util/compatfd.c                               |   8 +-
 33 files changed, 270 insertions(+), 186 deletions(-)
 create mode 100644 migration/target.c


Re: [PULL 00/20] CI, qtest, and misc patches
Posted by Peter Maydell 2 years, 11 months ago
On Fri, 14 May 2021 at 13:15, Thomas Huth <thuth@redhat.com> wrote:
>
>  Hi Peter!
>
> The following changes since commit 2d3fc4e2b069494b1e9e2e4a1e3de24cbc036426:
>
>   Merge remote-tracking branch 'remotes/armbru/tags/pull-misc-2021-05-12' into staging (2021-05-13 20:13:24 +0100)
>
> are available in the Git repository at:
>
>   https://gitlab.com/thuth/qemu.git tags/pull-request-2021-05-14
>
> for you to fetch changes up to 568740bedf22cc8d0ec9ab1ce522a97baab5961c:
>
>   cirrus.yml: Fix the MSYS2 task (2021-05-14 13:11:48 +0200)
>
> ----------------------------------------------------------------
> * Replace YAML anchors by extends in the gitlab-CI yaml files
> * Many small qtest fixes (e.g. to fix issues discovered by Coverity)
> * Poison more config switches in common code
> * Fix the failing Travis-CI and Cirrus-CI tasks
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/6.1
for any user-visible changes.

-- PMM