All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss
@ 2026-11-06 17:18 Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 1/4] hw/misc: move vmlaunchupdate.c from specific_ss " Daniel P. Berrangé
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Daniel P. Berrangé @ 2026-11-06 17:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Philippe Mathieu-Daudé,
	Alex Bennée, Daniel P. Berrangé

If building with IGVM enabled and both arm-softmmu and
aarch64-softmmu, test-multi-target breaks because the
vm-launch-update QOM class is linked in twice and thus
registered twice.

Daniel P. Berrangé (4):
  hw/misc: move vmlaunchupdate.c from specific_ss to system_ss
  tests/lcitool: add mips32r6el toolchain to refresh script
  tests/docker: add igvm to Fedora container images
  MAINTAINERS: fix typo in test-multi-target.c filename

 MAINTAINERS                                         | 2 +-
 hw/misc/meson.build                                 | 2 +-
 tests/docker/dockerfiles/fedora-rust-nightly.docker | 1 +
 tests/docker/dockerfiles/fedora.docker              | 1 +
 tests/lcitool/projects/qemu.yml                     | 1 +
 tests/lcitool/refresh                               | 2 ++
 6 files changed, 7 insertions(+), 2 deletions(-)

-- 
2.56.0



^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/4] hw/misc: move vmlaunchupdate.c from specific_ss to system_ss
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
@ 2026-11-06 17:18 ` Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script Daniel P. Berrangé
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Daniel P. Berrangé @ 2026-11-06 17:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Philippe Mathieu-Daudé,
	Alex Bennée, Daniel P. Berrangé

test-multi-target fails when igvm is enabled if both arm and
aarch64 targets are enabled:

  $ tests/unit/test-multi-target
  TAP version 14
  # random seed: R02S0c27cf3526927aa4a7f822f0a8315b3e
  1..2
  # Start of multi-targets tests
  # Start of qom-types-are-unique tests
  Registering `vm-launch-update' which already exists
  # child process (/multi-targets/qom-types-are-unique/aarch64 [1982534]) killed by signal 6 (Aborted)
  # child process (/multi-targets/qom-types-are-unique/aarch64 [1982534]) stdout: ""
  # child process (/multi-targets/qom-types-are-unique/aarch64 [1982534]) stderr: "Registering `vm-launch-update' which already exists\n"
  **
  ERROR:../tests/unit/test-multi-target.c:41:test_qom_types_are_unique: child process (/multi-targets/qom-types-are-unique/aarch64 [1982534]) failed unexpectedly
  not ok /multi-targets/qom-types-are-unique/aarch64 - ERROR:../tests/unit/test-multi-target.c:41:test_qom_types_are_unique: child process (/multi-targets/qom-types-are-unique/aarch64 [1982534]) failed unexpectedly
  Bail out!
  Aborted                    (core dumped) tests/unit/test-multi-target

This is because it is linked twice into test-multi-target
and thus the same device class object code is duplicated and
thus the class name gets registered twice.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 hw/misc/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/misc/meson.build b/hw/misc/meson.build
index fc3800baec0..7a071af82b0 100644
--- a/hw/misc/meson.build
+++ b/hw/misc/meson.build
@@ -170,7 +170,7 @@ specific_ss.add(when: 'CONFIG_MIPS_ITU', if_true: files('mips_itu.c'))
 specific_ss.add(when: 'CONFIG_RISCV_MIPS_CMGCR', if_true: files('riscv_cmgcr.c'))
 specific_ss.add(when: 'CONFIG_RISCV_MIPS_CPC', if_true: files('riscv_cpc.c'))
 if igvm.found()
-   specific_ss.add(when: 'CONFIG_FW_CFG_DMA', if_true: files('vmlaunchupdate.c'))
+   system_ss.add(when: 'CONFIG_FW_CFG_DMA', if_true: files('vmlaunchupdate.c'))
 endif
 
 system_ss.add(when: 'CONFIG_SBSA_REF', if_true: files('sbsa_ec.c'))
-- 
2.56.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 1/4] hw/misc: move vmlaunchupdate.c from specific_ss " Daniel P. Berrangé
@ 2026-11-06 17:18 ` Daniel P. Berrangé
  2026-11-09 11:04   ` Philippe Mathieu-Daudé
  2026-11-06 17:18 ` [PATCH 3/4] tests/docker: add igvm to Fedora container images Daniel P. Berrangé
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 8+ messages in thread
From: Daniel P. Berrangé @ 2026-11-06 17:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Philippe Mathieu-Daudé,
	Alex Bennée, Daniel P. Berrangé

The debian-all-test-cross.docker file is auto-generated so
the extra toolchains need to be listed in the prologue
in the refresh script.

Fixes: ace0b0b3d104 ("tests/docker: Add the mips32r6el toolchain on Debian cross container")
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 tests/lcitool/refresh | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/lcitool/refresh b/tests/lcitool/refresh
index 94cab24bbdb..075acb82e1d 100755
--- a/tests/lcitool/refresh
+++ b/tests/lcitool/refresh
@@ -200,6 +200,8 @@ RUN apt update && \
   libc6-dev-mips64el-cross \
   gcc-mipsel-linux-gnu \
   libc6-dev-mipsel-cross \
+  gcc-mipsisa32r6el-linux-gnu \
+  libc6-dev-mipsr6el-cross \
   gcc-mipsisa64r6-linux-gnuabi64 \
   libc6-dev-mips64r6-cross \
   gcc-mipsisa64r6el-linux-gnuabi64 \
-- 
2.56.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 3/4] tests/docker: add igvm to Fedora container images
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 1/4] hw/misc: move vmlaunchupdate.c from specific_ss " Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script Daniel P. Berrangé
@ 2026-11-06 17:18 ` Daniel P. Berrangé
  2026-11-06 17:18 ` [PATCH 4/4] MAINTAINERS: fix typo in test-multi-target.c filename Daniel P. Berrangé
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Daniel P. Berrangé @ 2026-11-06 17:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Philippe Mathieu-Daudé,
	Alex Bennée, Daniel P. Berrangé

The igvm-devel RPM is used for certain features on the
x86 and arm platforms, so should be included to give
build and test coverage in CI.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 tests/docker/dockerfiles/fedora-rust-nightly.docker | 1 +
 tests/docker/dockerfiles/fedora.docker              | 1 +
 tests/lcitool/projects/qemu.yml                     | 1 +
 3 files changed, 3 insertions(+)

diff --git a/tests/docker/dockerfiles/fedora-rust-nightly.docker b/tests/docker/dockerfiles/fedora-rust-nightly.docker
index 144325ae1e7..7eae32f126a 100644
--- a/tests/docker/dockerfiles/fedora-rust-nightly.docker
+++ b/tests/docker/dockerfiles/fedora-rust-nightly.docker
@@ -54,6 +54,7 @@ exec "$@"\n' > /usr/bin/nosync && \
                        gtk-vnc2-devel \
                        gtk3-devel \
                        hostname \
+                       igvm-devel \
                        jemalloc-devel \
                        json-c-devel \
                        libaio-devel \
diff --git a/tests/docker/dockerfiles/fedora.docker b/tests/docker/dockerfiles/fedora.docker
index 8f7e9efc899..871b1ba3f20 100644
--- a/tests/docker/dockerfiles/fedora.docker
+++ b/tests/docker/dockerfiles/fedora.docker
@@ -54,6 +54,7 @@ exec "$@"\n' > /usr/bin/nosync && \
                        gtk-vnc2-devel \
                        gtk3-devel \
                        hostname \
+                       igvm-devel \
                        jemalloc-devel \
                        json-c-devel \
                        libaio-devel \
diff --git a/tests/lcitool/projects/qemu.yml b/tests/lcitool/projects/qemu.yml
index abbd3e3f0b8..37a0c931fb2 100644
--- a/tests/lcitool/projects/qemu.yml
+++ b/tests/lcitool/projects/qemu.yml
@@ -35,6 +35,7 @@ packages:
  - gtk3
  - gtk-vnc
  - hostname
+ - igvm
  - json-c
  - libaio
  - libasan
-- 
2.56.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 4/4] MAINTAINERS: fix typo in test-multi-target.c filename
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
                   ` (2 preceding siblings ...)
  2026-11-06 17:18 ` [PATCH 3/4] tests/docker: add igvm to Fedora container images Daniel P. Berrangé
@ 2026-11-06 17:18 ` Daniel P. Berrangé
  2026-11-06 17:47 ` [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Pierrick Bouvier
  2026-11-06 23:18 ` Pierrick Bouvier
  5 siblings, 0 replies; 8+ messages in thread
From: Daniel P. Berrangé @ 2026-11-06 17:18 UTC (permalink / raw)
  To: qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Philippe Mathieu-Daudé,
	Alex Bennée, Daniel P. Berrangé

Fixes: 8b395deb9475 ("target-info: add multi-target test binary")
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 55463103ff1..fa0c6d0c9bc 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2186,7 +2186,7 @@ M: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>
 S: Supported
 F: include/qemu/target-info*.h
 F: target-info*.c
-F: tests/unit/test-multi-targets.c
+F: tests/unit/test-multi-target.c
 
 Xtensa Machines
 ---------------
-- 
2.56.0



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
                   ` (3 preceding siblings ...)
  2026-11-06 17:18 ` [PATCH 4/4] MAINTAINERS: fix typo in test-multi-target.c filename Daniel P. Berrangé
@ 2026-11-06 17:47 ` Pierrick Bouvier
  2026-11-06 23:18 ` Pierrick Bouvier
  5 siblings, 0 replies; 8+ messages in thread
From: Pierrick Bouvier @ 2026-11-06 17:47 UTC (permalink / raw)
  To: Daniel P. Berrangé, qemu-devel
  Cc: Jiaxun Yang, Philippe Mathieu-Daudé, Alex Bennée

On 11/6/2026 9:18 AM, Daniel P. Berrangé wrote:
> If building with IGVM enabled and both arm-softmmu and
> aarch64-softmmu, test-multi-target breaks because the
> vm-launch-update QOM class is linked in twice and thus
> registered twice.
> 
> Daniel P. Berrangé (4):
>   hw/misc: move vmlaunchupdate.c from specific_ss to system_ss
>   tests/lcitool: add mips32r6el toolchain to refresh script
>   tests/docker: add igvm to Fedora container images
>   MAINTAINERS: fix typo in test-multi-target.c filename
> 
>  MAINTAINERS                                         | 2 +-
>  hw/misc/meson.build                                 | 2 +-
>  tests/docker/dockerfiles/fedora-rust-nightly.docker | 1 +
>  tests/docker/dockerfiles/fedora.docker              | 1 +
>  tests/lcitool/projects/qemu.yml                     | 1 +
>  tests/lcitool/refresh                               | 2 ++
>  6 files changed, 7 insertions(+), 2 deletions(-)
> 

Good catch, thanks.

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>

Sending it for 11.2 as a multi-target fix, with the additional whpx fix.

Regards,
Pierrick


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss
  2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
                   ` (4 preceding siblings ...)
  2026-11-06 17:47 ` [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Pierrick Bouvier
@ 2026-11-06 23:18 ` Pierrick Bouvier
  5 siblings, 0 replies; 8+ messages in thread
From: Pierrick Bouvier @ 2026-11-06 23:18 UTC (permalink / raw)
  To: Daniel P. Berrangé, qemu-devel
  Cc: Jiaxun Yang, Philippe Mathieu-Daudé, Alex Bennée

On 11/6/2026 9:18 AM, Daniel P. Berrangé wrote:
> If building with IGVM enabled and both arm-softmmu and
> aarch64-softmmu, test-multi-target breaks because the
> vm-launch-update QOM class is linked in twice and thus
> registered twice.
> 
> Daniel P. Berrangé (4):
>   hw/misc: move vmlaunchupdate.c from specific_ss to system_ss
>   tests/lcitool: add mips32r6el toolchain to refresh script
>   tests/docker: add igvm to Fedora container images
>   MAINTAINERS: fix typo in test-multi-target.c filename
> 
>  MAINTAINERS                                         | 2 +-
>  hw/misc/meson.build                                 | 2 +-
>  tests/docker/dockerfiles/fedora-rust-nightly.docker | 1 +
>  tests/docker/dockerfiles/fedora.docker              | 1 +
>  tests/lcitool/projects/qemu.yml                     | 1 +
>  tests/lcitool/refresh                               | 2 ++
>  6 files changed, 7 insertions(+), 2 deletions(-)
> 

This was merged into master (71f25c75417d31c214ba2cd26b273c114fc99987).
Thank you for your contribution!

Regards,
Pierrick


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script
  2026-11-06 17:18 ` [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script Daniel P. Berrangé
@ 2026-11-09 11:04   ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2026-11-09 11:04 UTC (permalink / raw)
  To: Daniel P. Berrangé, qemu-devel
  Cc: Pierrick Bouvier, Jiaxun Yang, Alex Bennée

On 6/11/26 18:18, Daniel P. Berrangé wrote:
> The debian-all-test-cross.docker file is auto-generated so
> the extra toolchains need to be listed in the prologue
> in the refresh script.
> 
> Fixes: ace0b0b3d104 ("tests/docker: Add the mips32r6el toolchain on Debian cross container")
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>   tests/lcitool/refresh | 2 ++
>   1 file changed, 2 insertions(+)

Oops thanks.

Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>



^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-11-09 11:04 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-06 17:18 [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Daniel P. Berrangé
2026-11-06 17:18 ` [PATCH 1/4] hw/misc: move vmlaunchupdate.c from specific_ss " Daniel P. Berrangé
2026-11-06 17:18 ` [PATCH 2/4] tests/lcitool: add mips32r6el toolchain to refresh script Daniel P. Berrangé
2026-11-09 11:04   ` Philippe Mathieu-Daudé
2026-11-06 17:18 ` [PATCH 3/4] tests/docker: add igvm to Fedora container images Daniel P. Berrangé
2026-11-06 17:18 ` [PATCH 4/4] MAINTAINERS: fix typo in test-multi-target.c filename Daniel P. Berrangé
2026-11-06 17:47 ` [PATCH 0/4] hw/misc: move vmlaunchupdate.c to system_ss Pierrick Bouvier
2026-11-06 23:18 ` Pierrick Bouvier

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.