Re: [PULL 00/23] Final test, gdbstub, plugin and gitdm updates for 8.2

[Stefan Hajnoczi <stefanha@gmail.com>]

On Tue, 7 Nov 2023 at 22:25, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> The following changes since commit 462ad017ed76889d46696a3581e1b52343f9b683:
>
>   Merge tag 'pixman-pull-request' of https://gitlab.com/marcandre.lureau/qemu into staging (2023-11-07 19:00:03 +0800)
>
> are available in the Git repository at:
>
>   https://gitlab.com/stsquad/qemu.git tags/pull-halloween-omnibus-071123-1
>
> for you to fetch changes up to 00da668de6856d912cd75474ba759927e29d0e49:
>
>   Revert "tests/tcg/nios2: Re-enable linux-user tests" (2023-11-07 14:18:29 +0000)
>
> ----------------------------------------------------------------
> Final test, gdbstub, plugin and gitdm updates for 8.2
>
>   - fix duplicate register in arm xml
>   - hide various duplicate system registers from gdbstub
>   - add new gdb register test to the CI (skipping s390x/ppc64 for now)
>   - introduce GDBFeatureBuilder
>   - move plugin initialisation to after vCPU init completes
>   - enable building TCG plugins on Windows platform

Hi Alex,
Please take a look at the following CI failure:

i686-w64-mingw32-gcc -m32 -o tests/plugin/libempty.dll
plugins/qemu_plugin_api.lib tests/plugin/libempty.dll.p/empty.c.obj
tests/plugin/libempty.dll.p/.._.._contrib_plugins_win32_linker.c.obj
-Wl,--allow-shlib-undefined -shared -Wl,--start-group
-Wl,--out-implib=tests/plugin/libempty.dll.a -fstack-protector-strong
-Wl,--no-seh -Wl,--nxcompat -Wl,--dynamicbase -Wl,--warn-common
/usr/i686-w64-mingw32/sys-root/mingw/lib/libglib-2.0.dll.a
/usr/i686-w64-mingw32/sys-root/mingw/lib/libintl.dll.a
/usr/i686-w64-mingw32/sys-root/mingw/lib/libgmodule-2.0.dll.a
-lkernel32 -luser32 -lgdi32 -lwinspool -lshell32 -lole32 -loleaut32
-luuid -lcomdlg32 -ladvapi32 -Wl,--end-group
/usr/lib/gcc/i686-w64-mingw32/12.2.1/../../../../i686-w64-mingw32/bin/ld:
tests/plugin/libempty.dll.p/empty.c.obj: in function
`qemu_plugin_install':
/builds/qemu-project/qemu/build/../tests/plugin/empty.c:30: undefined
reference to `_imp__qemu_plugin_register_vcpu_tb_trans_cb'
collect2: error: ld returned 1 exit status

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

Thanks,
Stefan

>   - various gitdm updates
>   - some mailmap fixes
>   - disable testing for nios2 signals which have regressed
>
> ----------------------------------------------------------------
> Akihiko Odaki (5):
>       default-configs: Add TARGET_XML_FILES definition
>       gdbstub: Add num_regs member to GDBFeature
>       gdbstub: Introduce gdb_find_static_feature()
>       gdbstub: Introduce GDBFeatureBuilder
>       cpu: Call plugin hooks only when ready
>
> Alex Bennée (13):
>       gdb-xml: fix duplicate register in arm-neon.xml
>       target/arm: mark the 32bit alias of PAR when LPAE enabled
>       target/arm: hide all versions of DBGD[RS]AR from gdbstub
>       target/arm: hide aliased MIDR from gdbstub
>       tests/tcg: add an explicit gdbstub register tester
>       tests/avocado: update the tcg_plugins test
>       configure: tell meson and contrib_plugins about DLLTOOL
>       gitlab: add dlltool to Windows CI
>       contrib/gitdm: Add Rivos Inc to the domain map
>       contrib/gitdm: map HiSilicon to Huawei
>       contrib/gitdm: add Daynix to domain-map
>       mailmap: fixup some more corrupted author fields
>       Revert "tests/tcg/nios2: Re-enable linux-user tests"
>
> Greg Manning (4):
>       plugins: add dllexport and dllimport to api funcs
>       plugins: make test/example plugins work on windows
>       plugins: disable lockstep plugin on windows
>       plugins: allow plugins to be enabled on windows
>
> luzhipeng (1):
>       contrib/gitdm: add domain-map for Cestc
>
>  MAINTAINERS                                        |   2 +-
>  configure                                          |  13 +-
>  configs/targets/loongarch64-linux-user.mak         |   1 +
>  meson.build                                        |   5 +
>  include/exec/gdbstub.h                             |  59 ++++++
>  include/qemu/qemu-plugin.h                         |  50 +++++-
>  contrib/plugins/win32_linker.c                     |  34 ++++
>  cpu-target.c                                       |  11 --
>  gdbstub/gdbstub.c                                  |  78 ++++++++
>  hw/core/cpu-common.c                               |  10 ++
>  target/arm/debug_helper.c                          |  10 +-
>  target/arm/helper.c                                |  37 ++--
>  .gitlab-ci.d/windows.yml                           |   1 +
>  .mailmap                                           |   2 +
>  contrib/gitdm/domain-map                           |   4 +
>  contrib/plugins/Makefile                           |  26 ++-
>  gdb-xml/arm-neon.xml                               |   2 +-
>  plugins/meson.build                                |  19 ++
>  scripts/feature_to_c.py                            |  46 ++++-
>  tests/avocado/tcg_plugins.py                       |  28 +--
>  tests/plugin/meson.build                           |  14 +-
>  tests/tcg/multiarch/Makefile.target                |  11 +-
>  tests/tcg/multiarch/gdbstub/registers.py           | 197 +++++++++++++++++++++
>  tests/tcg/multiarch/system/Makefile.softmmu-target |  13 +-
>  tests/tcg/nios2/Makefile.target                    |  11 ++
>  tests/tcg/ppc64/Makefile.target                    |   7 +
>  tests/tcg/s390x/Makefile.target                    |   4 +
>  27 files changed, 637 insertions(+), 58 deletions(-)
>  create mode 100644 contrib/plugins/win32_linker.c
>  create mode 100644 tests/tcg/multiarch/gdbstub/registers.py
>  create mode 100644 tests/tcg/nios2/Makefile.target
>
> --
> 2.39.2
>
>