qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 135e6a: tests/Makefile.include: clean-up old


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 135e6a: tests/Makefile.include: clean-up old code
Date: Sat, 12 Feb 2022 14:03:53 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 135e6a09a949e9dbf69003482561abe098c354e5
      
https://github.com/qemu/qemu/commit/135e6a09a949e9dbf69003482561abe098c354e5
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/Makefile.include

  Log Message:
  -----------
  tests/Makefile.include: clean-up old code

This is no longer needed since a2ce7dbd91 ("meson: convert tests/qtest
to meson", 2020-08-21)

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


  Commit: 8dcb404bff6d9147765d7dd3e9c8493372186420
      
https://github.com/qemu/qemu/commit/8dcb404bff6d9147765d7dd3e9c8493372186420
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/qtest/vhost-user-test.c

  Log Message:
  -----------
  tests/qtest: enable more vhost-user tests by default

If this starts causing failures again we should probably fix that.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20220204204335.1689602-3-alex.bennee@linaro.org>


  Commit: 029e2da880f323d97c644a9ee70150c15e91ae2a
      
https://github.com/qemu/qemu/commit/029e2da880f323d97c644a9ee70150c15e91ae2a
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M Makefile

  Log Message:
  -----------
  Makefile: also remove .gcno files when cleaning

Left over .gcno files from old builds can really confuse gcov and the
user expects a clean slate after "make clean". Make clean mean clean.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-4-alex.bennee@linaro.org>


  Commit: 5934ebe99025e12dac39cd1ba91f4116aec32774
      
https://github.com/qemu/qemu/commit/5934ebe99025e12dac39cd1ba91f4116aec32774
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .gitignore

  Log Message:
  -----------
  .gitignore: add .gcov pattern

The gcovr tool is very messy and can leave a lot of crap in the source
tree even when using build directories.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220204204335.1689602-5-alex.bennee@linaro.org>


  Commit: 3260f4e6f1d211fb521dd6ee154f9ed650341cf7
      
https://github.com/qemu/qemu/commit/3260f4e6f1d211fb521dd6ee154f9ed650341cf7
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Cover lcitool submodule with build test / automation

lcitool is used by build test / automation, we want maintainers
to get notified if the submodule is updated.

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-2-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-6-alex.bennee@linaro.org>


  Commit: 9641ba6b58d2ecc82e87545069b223c4ac46476b
      
https://github.com/qemu/qemu/commit/9641ba6b58d2ecc82e87545069b223c4ac46476b
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .gitmodules

  Log Message:
  -----------
  gitmodules: Correct libvirt-ci submodule URL

Correct the libvirt-ci repository URL to avoid this warning when
cloning / refreshing the submodule:

  warning: redirecting to https://gitlab.com/libvirt/libvirt-ci.git/

Fixes: 4ebb040f1fd ("tests: integrate lcitool for generating build env 
manifests")
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-3-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-7-alex.bennee@linaro.org>


  Commit: 5d3539d561d028263ea21e577fcadd12ff8abcc6
      
https://github.com/qemu/qemu/commit/5d3539d561d028263ea21e577fcadd12ff8abcc6
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests/lcitool: Include local qemu.yml when refreshing cirrus-ci files

The script only include the local qemu.yml for Dockerfiles.
Since we want to keep the Cirrus-CI generated files in sync,
also use the --data-dir option in generate_cirrus().

Fixes: c45a540f4bd (".gitlab-ci.d/cirrus: auto-generate variables with lcitool")
Reported-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-4-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-8-alex.bennee@linaro.org>


  Commit: 9e8be4c546ce8469ca9702715bf8f198d604b685
      
https://github.com/qemu/qemu/commit/9e8be4c546ce8469ca9702715bf8f198d604b685
  Author: Michael Tokarev <mjt@tls.msk.ru>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M block/meson.build
    M meson.build
    M meson_options.txt
    M scripts/checkpatch.pl
    M scripts/ci/org.centos/stream/8/x86_64/configure
    M scripts/coverity-scan/coverity-scan.docker
    M scripts/coverity-scan/run-coverity-scan
    M scripts/meson-buildoptions.sh

  Log Message:
  -----------
  drop libxml2 checks since libxml is not actually used (for parallels)

For a long time, we assumed that libxml2 is necessary for parallels
block format support (block/parallels*). However, this format actually
does not use libxml [*]. Since this is the only user of libxml2 in
whole QEMU tree, we can drop all libxml2 checks and dependencies too.

It is even more: --enable-parallels configure option was the only
option which was silently ignored when it's (fake) dependency
(libxml2) isn't installed.

Drop all mentions of libxml2.

[*] Actually the basis for libxml use were introduced in commit
    ed279a06c53 ("configure: add dependency") but the implementation
    was never merged:
    
https://lore.kernel.org/qemu-devel/70227bbd-a517-70e9-714f-e6e0ec431be9@openvz.org/

Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20220119090423.149315-1-mjt@msgid.tls.msk.ru>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
[PMD: Updated description and adapted to use lcitool]
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-5-f4bug@amsat.org>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20220204204335.1689602-9-alex.bennee@linaro.org>


  Commit: dabee8381a2c3b838f9398bb9f05a012df8768c9
      
https://github.com/qemu/qemu/commit/dabee8381a2c3b838f9398bb9f05a012df8768c9
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .gitlab-ci.d/cirrus/freebsd-12.vars
    M .gitlab-ci.d/cirrus/freebsd-13.vars
    M .gitlab-ci.d/cirrus/macos-11.vars
    M tests/docker/dockerfiles/alpine.docker
    M tests/docker/dockerfiles/centos8.docker
    M tests/docker/dockerfiles/fedora.docker
    M tests/docker/dockerfiles/opensuse-leap.docker
    M tests/docker/dockerfiles/ubuntu1804.docker
    M tests/docker/dockerfiles/ubuntu2004.docker
    M tests/lcitool/libvirt-ci
    M tests/lcitool/projects/qemu.yml

  Log Message:
  -----------
  tests/lcitool: Refresh submodule and remove libxml2

The previous commit removed all uses of libxml2.

Refresh lcitool submodule, update qemu.yml and refresh the generated
files by running:

  $ make lcitool-refresh

Note: This refreshment also removes libudev dependency on Fedora
and CentOS due to libvirt-ci commit 18bfaee ("mappings: Improve
mapping for libudev"), since "The udev project has been absorbed
by the systemd project", and lttng-ust on FreeBSD runners due to
libvirt-ci commit 6dd9b6f ("guests: drop lttng-ust from FreeBSD
platform").

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-6-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-10-alex.bennee@linaro.org>


  Commit: d2b7bb099fb1f8827f9b5e648f5b6ddd01191719
      
https://github.com/qemu/qemu/commit/d2b7bb099fb1f8827f9b5e648f5b6ddd01191719
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .cirrus.yml
    M .gitlab-ci.d/windows.yml

  Log Message:
  -----------
  tests: Manually remove libxml2 on MSYS2 runners

lcitool doesn't support MSYS2 targets, so manually remove
this now unnecessary library.

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-7-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-11-alex.bennee@linaro.org>


  Commit: 4491c46879a10c256f73a767822039bd3ec23c76
      
https://github.com/qemu/qemu/commit/4491c46879a10c256f73a767822039bd3ec23c76
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/docker/dockerfiles/ubuntu1804.docker
    M tests/docker/dockerfiles/ubuntu2004.docker
    M tests/lcitool/projects/qemu.yml

  Log Message:
  -----------
  tests/lcitool: Install libibumad to cover RDMA on Debian based distros

On Debian we also need libibumad to enable RDMA:

  $ ../configure --enable-rdma

  ERROR:  OpenFabrics librdmacm/libibverbs/libibumad not present.
          Your options:
           (1) Fast: Install infiniband packages (devel) from your distro.
           (2) Cleanest: Install libraries from www.openfabrics.org
           (3) Also: Install softiwarp if you don't have RDMA hardware

Add the dependency to lcitool's qemu.yml (where librdmacm and
libibverbs are already listed) and refresh the generated files
by running:

      $ make lcitool-refresh

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220121154134.315047-8-f4bug@amsat.org>
Message-Id: <20220204204335.1689602-12-alex.bennee@linaro.org>


  Commit: bda8bebad0c073e69ef16eb3efb44aa4306a69ac
      
https://github.com/qemu/qemu/commit/bda8bebad0c073e69ef16eb3efb44aa4306a69ac
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M docs/devel/style.rst

  Log Message:
  -----------
  docs/devel: mention our .editorconfig

Ideally we should keep all our automatic formatting gubins in here.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-13-alex.bennee@linaro.org>


  Commit: 6340af7a949589cf07b5194edbbfb352734d8a3b
      
https://github.com/qemu/qemu/commit/6340af7a949589cf07b5194edbbfb352734d8a3b
  Author: Stefan Hajnoczi <stefanha@redhat.com>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .gitlab-ci.d/crossbuild-template.yml

  Log Message:
  -----------
  gitlab: fall back to commit hash in qemu-setup filename

Personal repos may not have release tags (v6.0.0, v6.1.0, etc) and this
causes cross_system_build_job to fail when pretty-printing a unique
qemu-setup-*.exe name:

  version="$(git describe --match v[0-9]*)";
             ^^^^^^^^^^ fails ^^^^^^^^^^^
  mv -v qemu-setup*.exe qemu-setup-${version}.exe;

Fall back to the short commit hash if necessary. This fixes CI failures
that Greg Kurz and I experienced in our personal repos.

Cc: Greg Kurz <groug@kaod.org>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220125173454.10381-1-stefanha@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220204204335.1689602-14-alex.bennee@linaro.org>


  Commit: ab4f987c4c56a55977922595f95236890db88edf
      
https://github.com/qemu/qemu/commit/ab4f987c4c56a55977922595f95236890db88edf
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/lcitool/Makefile.include

  Log Message:
  -----------
  tests/lcitool: Allow lcitool-refresh in out-of-tree builds, too

When running "make lcitool-refresh" in an out-of-tree build, it
currently fails with an error message from git like this:

 fatal: not a git repository (or any parent up to mount point /)
 Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Fix it by changing to the source directory first before updating
the submodule.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220201085554.85733-1-thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220204204335.1689602-15-alex.bennee@linaro.org>


  Commit: 8b11f4c37a1691bbca36dc2756b02acf4ec5977e
      
https://github.com/qemu/qemu/commit/8b11f4c37a1691bbca36dc2756b02acf4ec5977e
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/docker/dockerfiles/centos8.docker
    M tests/lcitool/refresh

  Log Message:
  -----------
  tests: Update CentOS 8 container to CentOS Stream 8

Support for CentOS 8 has stopped at the end of 2021, so let's
switch to the Stream variant instead.

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


  Commit: 3bdc19af00937191623fb0a2cacbc4ad54882072
      
https://github.com/qemu/qemu/commit/3bdc19af00937191623fb0a2cacbc4ad54882072
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

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

  Log Message:
  -----------
  tests/tcg/sh4: disable another unreliable test

Given the other failures it looks like general thread handling on sh4
is sketchy. It fails more often on CI than on my developer machine
though. See https://gitlab.com/qemu-project/qemu/-/issues/856 for more
details.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Laurent Vivier <laurent@vivier.eu>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-17-alex.bennee@linaro.org>


  Commit: d9a6bad542cd796c785ff19793fa7ae24da6a97d
      
https://github.com/qemu/qemu/commit/d9a6bad542cd796c785ff19793fa7ae24da6a97d
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M docs/about/removed-features.rst
    M docs/devel/tracing.rst

  Log Message:
  -----------
  docs: remove references to TCG tracing

Users wanting this sort of functionality should turn to TCG plugins
instead.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Cc: Luis Vilanova <vilanova@imperial.ac.uk>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20220204204335.1689602-18-alex.bennee@linaro.org>


  Commit: c51e51005b74162df7a9c568213acca85e254efa
      
https://github.com/qemu/qemu/commit/c51e51005b74162df7a9c568213acca85e254efa
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M accel/tcg/atomic_common.c.inc
    M accel/tcg/atomic_template.h
    M accel/tcg/cputlb.c
    M accel/tcg/user-exec.c
    M tcg/tcg-op.c
    M trace-events

  Log Message:
  -----------
  tracing: remove TCG memory access tracing

If you really want to trace all memory operations TCG plugins gives
you a more flexible interface for doing so.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Cc: Luis Vilanova <vilanova@imperial.ac.uk>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20220204204335.1689602-19-alex.bennee@linaro.org>


  Commit: d201cf7a7323521a2abe4771298579673371a3cc
      
https://github.com/qemu/qemu/commit/d201cf7a7323521a2abe4771298579673371a3cc
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M include/exec/helper-proto.h
    M include/exec/helper-tcg.h
    R include/trace-tcg.h
    M trace/meson.build

  Log Message:
  -----------
  tracing: remove the trace-tcg includes from the build

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Cc: Luis Vilanova <vilanova@imperial.ac.uk>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-20-alex.bennee@linaro.org>


  Commit: 126d4123c50a78a99e04196126d42627911ef5b8
      
https://github.com/qemu/qemu/commit/126d4123c50a78a99e04196126d42627911ef5b8
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M include/exec/helper-gen.h
    M meson.build
    M scripts/tracetool/__init__.py
    R scripts/tracetool/format/tcg_h.py
    R scripts/tracetool/format/tcg_helper_c.py
    R scripts/tracetool/format/tcg_helper_h.py
    R scripts/tracetool/format/tcg_helper_wrapper_h.py
    M scripts/tracetool/vcpu.py
    M trace/meson.build

  Log Message:
  -----------
  tracing: excise the tcg related from tracetool

Now we have no TCG trace events and no longer handle them in the code
we can remove the handling from the tracetool to generate them. vcpu
tracing is still available although the existing syscall event is an
exercise in redundancy (plugins and -strace can also get the
information).

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Cc: Luis Vilanova <vilanova@imperial.ac.uk>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20220204204335.1689602-21-alex.bennee@linaro.org>


  Commit: 91d40327106c372760d09ffae93f91c24fcbbb6c
      
https://github.com/qemu/qemu/commit/91d40327106c372760d09ffae93f91c24fcbbb6c
  Author: Ivanov Arkady <arkadiy.ivanov@ispras.ru>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M include/qemu/qemu-plugin.h
    M plugins/api.c
    M plugins/qemu-plugins.symbols

  Log Message:
  -----------
  plugins: add helper functions for coverage plugins

Which provide information about:
- start_code.
- end_code.
- entry.
- path to the executable binary.

Signed-off-by: Ivanov Arkady <arkadiy.ivanov@ispras.ru>
Message-Id: 
<163491883461.304355.8210754161847179432.stgit@pc-System-Product-Name>
[AJB: reword title, better descriptions, defaults, rm export, fix include]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220204204335.1689602-22-alex.bennee@linaro.org>


  Commit: 227b45dc4218516262cc892b661e5438e134cf2e
      
https://github.com/qemu/qemu/commit/227b45dc4218516262cc892b661e5438e134cf2e
  Author: Ivanov Arkady <arkadiy.ivanov@ispras.ru>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M contrib/plugins/Makefile
    A contrib/plugins/drcov.c

  Log Message:
  -----------
  contrib/plugins: add a drcov plugin

This patch adds the ability to generate files in drcov format. Primary
goal this script is to have coverage logfiles thatwork in Lighthouse.

Signed-off-by: Ivanov Arkady <arkadiy.ivanov@ispras.ru>
Message-Id: 
<163491884553.304355.13246023070235438959.stgit@pc-System-Product-Name>
[AJB: use g_ptr_array instead of slist]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220204204335.1689602-23-alex.bennee@linaro.org>


  Commit: 39be9dd30fff4bae81ba048d93a195eabe2635ef
      
https://github.com/qemu/qemu/commit/39be9dd30fff4bae81ba048d93a195eabe2635ef
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/plugin/insn.c

  Log Message:
  -----------
  tests/plugin: allow libinsn.so per-CPU counts

We won't go fully flexible but for most system emulation 8 vCPUs
resolution should be enough for anybody ;-)

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


  Commit: f6d1cd4d48e84fc8047f9094760e42b0980eccec
      
https://github.com/qemu/qemu/commit/f6d1cd4d48e84fc8047f9094760e42b0980eccec
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M tests/plugin/insn.c

  Log Message:
  -----------
  tests/plugins: add instruction matching to libinsn.so

This adds simple instruction matching to the libinsn.so plugin which
is useful for examining the execution distance between instructions.
For example to track how often we flush in ARM due to TLB updates:

  -plugin ./tests/plugin/libinsn.so,match=tlbi

which leads to output like this:

  0xffffffc01019a918, 'tlbi vale1is, x1', 5702 hits, 31825 match hits, Δ+8112 
since last match, 68859 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5703 hits, 56593 match hits, 
Δ+17712125 since last match, 33455 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5704 hits, 56594 match hits, Δ+12689 
since last match, 33454 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5705 hits, 56595 match hits, Δ+12585 
since last match, 33454 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5706 hits, 56596 match hits, Δ+10491 
since last match, 33454 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5707 hits, 56597 match hits, Δ+4721 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5708 hits, 56598 match hits, Δ+10733 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5709 hits, 56599 match hits, Δ+61959 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5710 hits, 56600 match hits, Δ+55235 
since last match, 33454 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5711 hits, 56601 match hits, Δ+54373 
since last match, 33454 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5712 hits, 56602 match hits, Δ+2705 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5713 hits, 56603 match hits, Δ+17262 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5714 hits, 56604 match hits, Δ+17206 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5715 hits, 56605 match hits, Δ+28940 
since last match, 33453 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5716 hits, 56606 match hits, Δ+7370 
since last match, 33452 avg insns/match
  0xffffffc01019a918, 'tlbi vale1is, x1', 5717 hits, 56607 match hits, Δ+7066 
since last match, 33452 avg insns/match

showing we do some sort of TLBI invalidation every 33 thousand
instructions.

Cc: Vasilev Oleg <vasilev.oleg@huawei.com>
Cc: Richard Henderson <richard.henderson@linaro.org>
Cc: Emilio Cota <cota@braap.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-25-alex.bennee@linaro.org>


  Commit: 346cd004f65edf2e55dfd8333cb0f63e08423791
      
https://github.com/qemu/qemu/commit/346cd004f65edf2e55dfd8333cb0f63e08423791
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M target/i386/tcg/sysemu/seg_helper.c

  Log Message:
  -----------
  target/i386: use CPU_LOG_INT for IRQ servicing

I think these have been wrong since f193c7979c (do not depend on
thunk.h - more log items). Fix them so as not to confuse other
debugging.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220204204335.1689602-26-alex.bennee@linaro.org>


  Commit: 6f15c076dae07c2438eb488d71fa332e2b8d0963
      
https://github.com/qemu/qemu/commit/6f15c076dae07c2438eb488d71fa332e2b8d0963
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M accel/tcg/plugin-gen.c

  Log Message:
  -----------
  plugins: move reset of plugin data to tb_start

We can't always guarantee we get to the end of a translator loop.
Although this can happen for a variety of reasons it does happen more
often on x86 system emulation when an instruction spans across to an
un-faulted page. This caused confusion of the instruction tracking
data resulting in apparent reverse execution (at least from the
plugins point of view).

Fix this by moving the reset code to plugin_gen_tb_start so we always
start with a clean slate.

We unconditionally reset tcg_ctx->plugin_insn as the
plugin_insn_append code uses this as a proxy for knowing if plugins
are enabled for the current instruction. Otherwise we can hit a race
where a previously instrumented thread leaves a stale value after the
main thread exits and disables instrumentation.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/824
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220204204335.1689602-27-alex.bennee@linaro.org>


  Commit: 74154d7e4a9a693313ad7639a92ff443c6258741
      
https://github.com/qemu/qemu/commit/74154d7e4a9a693313ad7639a92ff443c6258741
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M .gitlab-ci.d/buildtest.yml
    R configs/targets/ppc64abi32-linux-user.mak
    M configure
    M docs/about/deprecated.rst
    M docs/about/removed-features.rst
    M docs/user/main.rst
    M linux-user/elfload.c
    M linux-user/ppc/signal.c
    M linux-user/ppc/target_syscall.h
    M linux-user/syscall_defs.h
    M tests/docker/dockerfiles/debian-ppc64el-cross.docker
    M tests/tcg/configure.sh

  Log Message:
  -----------
  linux-user: Remove the deprecated ppc64abi32 target

It's likely broken, and nobody cared for picking it up again
during the deprecation phase, so let's remove this now.

Since this is the last entry in deprecated_targets_list, remove
the related code in the configure script, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Cédric Le Goater <clg@kaod.org>
Acked-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20211215084958.185214-1-thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220112112722.3641051-32-alex.bennee@linaro.org>


  Commit: 514f9f8eb64bfd5d6c15024db93f83bd81998de5
      
https://github.com/qemu/qemu/commit/514f9f8eb64bfd5d6c15024db93f83bd81998de5
  Author: Alex Bennée <alex.bennee@linaro.org>
  Date:   2022-02-09 (Wed, 09 Feb 2022)

  Changed paths:
    M include/exec/cpu_ldst.h

  Log Message:
  -----------
  include/exec: fix softmmu version of TARGET_ABI_FMT_lx

TARGET_ABI_FMT_lx isn't available for softmmu which causes confusion
when trying to print. As abi_ptr == target_ulong use its format string
instead.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>


  Commit: da36afa2d8dc9c778292ff172083caba9558b4fa
      
https://github.com/qemu/qemu/commit/da36afa2d8dc9c778292ff172083caba9558b4fa
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2022-02-11 (Fri, 11 Feb 2022)

  Changed paths:
    M .cirrus.yml
    M .gitignore
    M .gitlab-ci.d/buildtest.yml
    M .gitlab-ci.d/cirrus/freebsd-12.vars
    M .gitlab-ci.d/cirrus/freebsd-13.vars
    M .gitlab-ci.d/cirrus/macos-11.vars
    M .gitlab-ci.d/crossbuild-template.yml
    M .gitlab-ci.d/windows.yml
    M .gitmodules
    M MAINTAINERS
    M Makefile
    M accel/tcg/atomic_common.c.inc
    M accel/tcg/atomic_template.h
    M accel/tcg/cputlb.c
    M accel/tcg/plugin-gen.c
    M accel/tcg/user-exec.c
    M block/meson.build
    R configs/targets/ppc64abi32-linux-user.mak
    M configure
    M contrib/plugins/Makefile
    A contrib/plugins/drcov.c
    M docs/about/deprecated.rst
    M docs/about/removed-features.rst
    M docs/devel/style.rst
    M docs/devel/tracing.rst
    M docs/user/main.rst
    M include/exec/cpu_ldst.h
    M include/exec/helper-gen.h
    M include/exec/helper-proto.h
    M include/exec/helper-tcg.h
    M include/qemu/qemu-plugin.h
    R include/trace-tcg.h
    M linux-user/elfload.c
    M linux-user/ppc/signal.c
    M linux-user/ppc/target_syscall.h
    M linux-user/syscall_defs.h
    M meson.build
    M meson_options.txt
    M plugins/api.c
    M plugins/qemu-plugins.symbols
    M scripts/checkpatch.pl
    M scripts/ci/org.centos/stream/8/x86_64/configure
    M scripts/coverity-scan/coverity-scan.docker
    M scripts/coverity-scan/run-coverity-scan
    M scripts/meson-buildoptions.sh
    M scripts/tracetool/__init__.py
    R scripts/tracetool/format/tcg_h.py
    R scripts/tracetool/format/tcg_helper_c.py
    R scripts/tracetool/format/tcg_helper_h.py
    R scripts/tracetool/format/tcg_helper_wrapper_h.py
    M scripts/tracetool/vcpu.py
    M target/i386/tcg/sysemu/seg_helper.c
    M tcg/tcg-op.c
    M tests/Makefile.include
    M tests/docker/dockerfiles/alpine.docker
    M tests/docker/dockerfiles/centos8.docker
    M tests/docker/dockerfiles/debian-ppc64el-cross.docker
    M tests/docker/dockerfiles/fedora.docker
    M tests/docker/dockerfiles/opensuse-leap.docker
    M tests/docker/dockerfiles/ubuntu1804.docker
    M tests/docker/dockerfiles/ubuntu2004.docker
    M tests/lcitool/Makefile.include
    M tests/lcitool/libvirt-ci
    M tests/lcitool/projects/qemu.yml
    M tests/lcitool/refresh
    M tests/plugin/insn.c
    M tests/qtest/vhost-user-test.c
    M tests/tcg/configure.sh
    M tests/tcg/sh4/Makefile.target
    M trace-events
    M trace/meson.build

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/stsquad/tags/pull-testing-and-plugins-090222-1' into staging

Testing and plugin updates:

  - include vhost tests in qtest
  - clean-up gcov ephemera in clean/.gitignore
  - lcitool and docker updates
  - mention .editorconfig in devel notes
  - switch Centos8 to Centos Stream 8
  - remove TCG tracing support
  - add coverage plugin using drcov format
  - expand abilities of libinsn.so plugin
  - use correct logging for i386 int cases
  - move reset of plugin data to start of block
  - deprecate ppc6432abi
  - fix TARGET_ABI_FMT_ptr for softmmu builds

# gpg: Signature made Wed 09 Feb 2022 14:13:14 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-and-plugins-090222-1: (28 commits)
  include/exec: fix softmmu version of TARGET_ABI_FMT_lx
  linux-user: Remove the deprecated ppc64abi32 target
  plugins: move reset of plugin data to tb_start
  target/i386: use CPU_LOG_INT for IRQ servicing
  tests/plugins: add instruction matching to libinsn.so
  tests/plugin: allow libinsn.so per-CPU counts
  contrib/plugins: add a drcov plugin
  plugins: add helper functions for coverage plugins
  tracing: excise the tcg related from tracetool
  tracing: remove the trace-tcg includes from the build
  tracing: remove TCG memory access tracing
  docs: remove references to TCG tracing
  tests/tcg/sh4: disable another unreliable test
  tests: Update CentOS 8 container to CentOS Stream 8
  tests/lcitool: Allow lcitool-refresh in out-of-tree builds, too
  gitlab: fall back to commit hash in qemu-setup filename
  docs/devel: mention our .editorconfig
  tests/lcitool: Install libibumad to cover RDMA on Debian based distros
  tests: Manually remove libxml2 on MSYS2 runners
  tests/lcitool: Refresh submodule and remove libxml2
  ...

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


Compare: https://github.com/qemu/qemu/compare/0a301624c2f4...da36afa2d8dc



reply via email to

[Prev in Thread] Current Thread [Next in Thread]