Branch: refs/heads/staging
Home: https://github.com/qemu/qemu
Commit: 96bd50435e7e5ab4b6f9af5deebd4d71fd026673
https://github.com/qemu/qemu/commit/96bd50435e7e5ab4b6f9af5deebd4d71fd026673
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M docs/about/deprecated.rst
M docs/about/removed-features.rst
M hw/sd/sd.c
M include/hw/sd/sd.h
Log Message:
-----------
hw/sd/sdcard: Remove support for spec v1.10
Support for spec v1.10 was deprecated in QEMU v9.1.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Pierrick Bouvier <[email protected]>
Message-ID: <[email protected]>
Commit: 98739a7d1044d0bc481fc14579670192eec917bb
https://github.com/qemu/qemu/commit/98739a7d1044d0bc481fc14579670192eec917bb
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M target/ppc/kvm.c
Log Message:
-----------
target/ppc/kvm: Avoid using alloca()
kvmppc_load_htab_chunk() is used for migration, thus is not
a hot path. Use the heap instead of the stack, removing the
alloca() call.
Reported-by: Peter Maydell <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Peter Maydell <[email protected]>
Reviewed-by: Manos Pitsidianakis <[email protected]>
Reviewed-by: Harsh Prateek Bora <[email protected]>
Message-Id: <[email protected]>
Commit: 362bcd4f7443228bacd4459cecb6a5e9a2a66f56
https://github.com/qemu/qemu/commit/362bcd4f7443228bacd4459cecb6a5e9a2a66f56
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M docs/devel/style.rst
Log Message:
-----------
docs/devel/style: Mention alloca() family API is forbidden
Suggested-by: Alex Bennée <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Manos Pitsidianakis <[email protected]>
Message-Id: <[email protected]>
Commit: 76a2d0542e2ca7783729a03ad882cebe50ee9447
https://github.com/qemu/qemu/commit/76a2d0542e2ca7783729a03ad882cebe50ee9447
Author: Djordje Todorovic <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/pci/pci.c
Log Message:
-----------
hw/pci: Allow explicit function numbers in pci
Since there is no pch_gbe emulation, we could be using func other
than 0 when adding new devices to specific boards.
Signed-off-by: Chao-ying Fu <[email protected]>
Signed-off-by: Djordje Todorovic <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
[PMD: Compare with null character ('\0'), not '0']
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: f3d9dfee29cef4ab43481fa5ddb093bc4900b741
https://github.com/qemu/qemu/commit/f3d9dfee29cef4ab43481fa5ddb093bc4900b741
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M migration/meson.build
Log Message:
-----------
migration: compile migration/ram.c once
Acked-by: Fabiano Rosas <[email protected]>
Reviewed-by: Peter Xu <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 13a2c53b606838d564e071df37f9984a31cd994d
https://github.com/qemu/qemu/commit/13a2c53b606838d564e071df37f9984a31cd994d
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M MAINTAINERS
M migration/meson.build
A migration/vfio-stub.c
M migration/vfio.c
Log Message:
-----------
migration/vfio: compile only once
Acked-by: Fabiano Rosas <[email protected]>
Reviewed-by: Peter Xu <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Message-ID: <[email protected]>
[PMD: Cover vfio-stub.c in MAINTAINERS]
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: d92187c552b008e8b6b5d9fbf7a2f904aea22011
https://github.com/qemu/qemu/commit/d92187c552b008e8b6b5d9fbf7a2f904aea22011
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M cpu-target.c
M meson.build
M target-info-stub.c
Log Message:
-----------
cpu-target: build compilation unit once for user/system
Reviewed-by: Richard Henderson <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 07dfb0ee1c98f9e14ff7b3d86e84f35ea9344d71
https://github.com/qemu/qemu/commit/07dfb0ee1c98f9e14ff7b3d86e84f35ea9344d71
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M include/exec/target_page.h
M meson.build
R page-target.c
Log Message:
-----------
include/exec/target_page.h: move page-target.c to header
Reviewed-by: Richard Henderson <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 1a6939aaf1069d30700442e7fdf8e317f50d2a66
https://github.com/qemu/qemu/commit/1a6939aaf1069d30700442e7fdf8e317f50d2a66
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/meson.build
Log Message:
-----------
hw/meson: enter target hw first
We can reuse target source sets for "generic" devices that are related
to a single architecture (like interrupt controllers).
Signed-off-by: Pierrick Bouvier <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Tested-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: a9d20589f35367e1a2a22f41e77498f4e2faea35
https://github.com/qemu/qemu/commit/a9d20589f35367e1a2a22f41e77498f4e2faea35
Author: Pierrick Bouvier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/intc/meson.build
Log Message:
-----------
hw/intc: compile some arm related source once
Let kvm related gic file out for now, as they are compiled only on
aarch64 hosts.
Signed-off-by: Pierrick Bouvier <[email protected]>
Tested-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 9a487619954c678bdf6dee596b120f31ebd2e47c
https://github.com/qemu/qemu/commit/9a487619954c678bdf6dee596b120f31ebd2e47c
Author: Laurent Vivier <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/net/e1000e_core.c
Log Message:
-----------
e1000e: Prevent crash from legacy interrupt firing after MSI-X enable
A race condition between guest driver actions and QEMU timers can lead
to an assertion failure when the guest switches the e1000e from legacy
interrupt mode to MSI-X. If a legacy interrupt delay timer (TIDV or
RDTR) is active, but the guest enables MSI-X before the timer fires,
the pending interrupt cause can trigger an assert in
e1000e_intmgr_collect_delayed_causes().
This patch removes the assertion and executes the code that clears the
pending legacy causes. This change is safe and introduces no unintended
behavioral side effects, as it only alters a state that previously led
to termination.
- when core->delayed_causes == 0 the function was already a no-op and
remains so.
- when core->delayed_causes != 0 the function would previously
crash due to the assertion failure. The patch now defines a safe
outcome by clearing the cause and returning. Since behavior after
the assertion never existed, this simply corrects the crash.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1863
Suggested-by: Akihiko Odaki <[email protected]>
Signed-off-by: Laurent Vivier <[email protected]>
Acked-by: Jason Wang <[email protected]>
Reviewed-by: Akihiko Odaki <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: f3593be02000ad377fb679066b2083c71d150307
https://github.com/qemu/qemu/commit/f3593be02000ad377fb679066b2083c71d150307
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M scripts/coverity-scan/COMPONENTS.md
Log Message:
-----------
scripts/coverity-scan/COMPONENTS.md: Add a 'plugins' category
Cover the TCG plugins files under their own Coverity category.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Peter Maydell <[email protected]>
Acked-by: Alex Bennée <[email protected]>
Message-Id: <[email protected]>
Commit: 6b08daa1ffc8327a344793c7dc64e1418c993fae
https://github.com/qemu/qemu/commit/6b08daa1ffc8327a344793c7dc64e1418c993fae
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/scsi/mptsas.c
Log Message:
-----------
hw/scsi/mptsas: Avoid silent integer truncation in MPI_FUNC_IOC_INIT
For the MaxDevices 8-bit field of the request / response structures
of the MPI_FUNCTION_IOC_INIT command, the 0x00 value means "max 256
devices". This is not a problem because when max_devices=256, its
value (0x100), being casted to a uint8_t, is truncated to 0x00.
However Coverity complains for an "Overflowed constant". Fix by
re-using the request fields in the response, since they are not
modified and use the same types.
Fix: Coverity 1547736 (Overflowed constant)
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Peter Maydell <[email protected]>
Message-Id: <[email protected]>
Commit: ec5f1e4839d2abeb1c3397d411543f47d8a9a6e7
https://github.com/qemu/qemu/commit/ec5f1e4839d2abeb1c3397d411543f47d8a9a6e7
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M include/hw/ssi/ssi.h
Log Message:
-----------
hw/ssi: Document ssi_transfer() method
A SPI transaction consists of shifting bit in sync with the CLK
line, writing on the MOSI (output) line / and reading MISO (input)
line.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Gustavo Romero <[email protected]>
Reviewed-by: Alex Bennée <[email protected]>
Commit: 1c463c23f793b21b36e87ea0407c5a37fd67a93c
https://github.com/qemu/qemu/commit/1c463c23f793b21b36e87ea0407c5a37fd67a93c
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M include/elf.h
Log Message:
-----------
elf: Add EF_MIPS_ARCH_ASE definitions
Include MIPS ASE ELF definitions from binutils:
https://sourceware.org/git/?p=binutils-gdb.git;a=blob;f=include/elf/mips.h;h=4fc190f404d828ded84e621bfcece5fa9f9c23c8;hb=HEAD#l210
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Message-Id: <[email protected]>
Commit: c4a143aa401531b1f8d9e32c396a0953ae359405
https://github.com/qemu/qemu/commit/c4a143aa401531b1f8d9e32c396a0953ae359405
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M linux-user/mips/elfload.c
Log Message:
-----------
linux-user/mips: Select 74Kf CPU to run MIPS16e binaries
The 74Kf is our latest CPU supporting MIPS16e ASE.
Note, currently QEMU doesn't have 64-bit CPU supporting MIPS16e ASE.
Cc: [email protected]
Fixes: 6ea219d0196..d19954f46df ("target-mips: MIPS16 support")
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/3054
Reported-by: Justin Applegate <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Message-Id: <[email protected]>
Commit: affa2c11561cec0cb3b4eda4f0a4835591fd5d25
https://github.com/qemu/qemu/commit/affa2c11561cec0cb3b4eda4f0a4835591fd5d25
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M linux-user/mips/elfload.c
Log Message:
-----------
linux-user/mips: Select M14Kc CPU to run microMIPS binaries
The M14Kc is our latest CPU supporting the microMIPS ASE.
Note, currently QEMU doesn't have 64-bit CPU supporting microMIPS ASE.
Cc: [email protected]
Fixes: 3c824109da0 ("target-mips: microMIPS ASE support")
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/3054
Reported-by: Justin Applegate <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Message-Id: <[email protected]>
Commit: fbc6bde7988663cae7f3207ad03217eaf3c634c8
https://github.com/qemu/qemu/commit/fbc6bde7988663cae7f3207ad03217eaf3c634c8
Author: Denis Rastyogin <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M target/mips/tcg/system/tlb_helper.c
Log Message:
-----------
target/mips: fix TLB huge page check to use 64-bit shift
Use extract64(entry, psn, 1) instead of (entry & (1 << psn)) to avoid
undefined behavior for shifts by 32–63 and to make bit extraction intent
explicit.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Signed-off-by: Denis Rastyogin <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 9bb42942cc241712866a626329db74ccb44f19da
https://github.com/qemu/qemu/commit/9bb42942cc241712866a626329db74ccb44f19da
Author: Aditya Gupta <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/ppc/pnv.c
M target/ppc/cpu.h
M target/ppc/misc_helper.c
Log Message:
-----------
hw/ppc: Fix build error with CONFIG_POWERNV disabled
Currently when CONFIG_POWERNV is not enabled, the build fails, such as
with --without-default-devices:
$ ./configure --without-default-devices
$ make
[281/283] Linking target qemu-system-ppc64
FAILED: qemu-system-ppc64
cc -m64 @qemu-system-ppc64.rsp
/usr/bin/ld: libqemu-ppc64-softmmu.a.p/target_ppc_misc_helper.c.o: in
function `helper_load_sprd':
.../target/ppc/misc_helper.c:335:(.text+0xcdc): undefined reference to
`pnv_chip_find_core'
/usr/bin/ld: libqemu-ppc64-softmmu.a.p/target_ppc_misc_helper.c.o: in
function `helper_store_sprd':
.../target/ppc/misc_helper.c:375:(.text+0xdf4): undefined reference to
`pnv_chip_find_core'
collect2: error: ld returned 1 exit status
...
This is since target/ppc/misc_helper.c references PowerNV specific
'pnv_chip_find_core' call.
Split the PowerNV specific SPRD code out of the generic PowerPC code, by
moving the SPRD code to pnv.c
Fixes: 9808ce6d5cb ("target/ppc: Big-core scratch register fix")
Cc: Philippe Mathieu-Daudé <[email protected]>
Reported-by: Thomas Huth <[email protected]>
Suggested-by: Cédric Le Goater <[email protected]>
Signed-off-by: Aditya Gupta <[email protected]>
Acked-by: Cédric Le Goater <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 2b77888d1c5da3767f43a23c46f83abcd7046d24
https://github.com/qemu/qemu/commit/2b77888d1c5da3767f43a23c46f83abcd7046d24
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/core/irq.c
M include/hw/irq.h
Log Message:
-----------
hw/irq: New qemu_init_irq_child() function
The qemu_init_irq() function initializes a TYPE_IRQ QOM object. The
caller is therefore responsible for eventually calling
qemu_free_irq() to unref (and thus free) it.
In many places where we want to initialize an IRQ we are in
the init/realize of some other QOM object; if we have a variant
of this function that calls object_initialize_child() then the
IRQ will be automatically cleaned up when its parent object is
destroyed, and we don't need to remember to manually free it.
Implement qemu_init_irq_child(), which is to qemu_init_irq()
what object_initialize_child() is to object_initialize().
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: f7caead7231ed38df32894b77a6755655c077152
https://github.com/qemu/qemu/commit/f7caead7231ed38df32894b77a6755655c077152
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/char/serial-pci-multi.c
Log Message:
-----------
hw/char/serial-pci-multi: Use qemu_init_irq_child() to avoid leak
The serial-pci-multi device initializes an IRQ with qemu_init_irq()
in its instance_init function; however it never calls qemu_free_irq(),
so the init/deinit cycle has a memory leak, which ASAN catches
in the device-introspect-test:
Direct leak of 576 byte(s) in 6 object(s) allocated from:
#0 0x626306ddade3 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qem
u-system-arm+0x21f1de3) (BuildId: 52ece17287eba2d68e5be980e1856cd1f6be932f)
#1 0x7756ade79b09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId: 1
eb6131419edb83b2178b682829a6913cf682d75)
#2 0x7756ade5b45a in g_hash_table_new_full
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4445a
) (BuildId: 1eb6131419edb83b2178b682829a6913cf682d75)
#3 0x62630965da37 in object_initialize_with_type
/mnt/nvmedisk/linaro/qemu-from-laptop/qem
u/build/arm-asan/../../qom/object.c:568:23
#4 0x62630965d440 in object_initialize
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/ar
m-asan/../../qom/object.c:578:5
#5 0x626309653eeb in qemu_init_irq
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-as
an/../../hw/core/irq.c:48:5
#6 0x6263072370bb in multi_serial_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/char/serial-pci-multi.c:183:9
Use the new qemu_init_irq_child() function instead, so that the
IRQ object is automatically unreffed when the serial-pci
device is deinited.
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
[PMD: Use "irq[*]" as child property name]
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: df2ec63709432f94201b3a286718edeba62e9b16
https://github.com/qemu/qemu/commit/df2ec63709432f94201b3a286718edeba62e9b16
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/ide/ich.c
Log Message:
-----------
hw/ide/ich.c: Use qemu_init_irq_child() to avoid memory leak
The ICH9 PCI device uses qemu_init_irq() in its instance_init method,
but fails to clean it up in its uninit. This results in a leak,
detected by ASAN when running the device-introspect-test:
Direct leak of 96 byte(s) in 1 object(s) allocated from:
#0 0x58f3b53ecde3 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qem
u-system-arm+0x21f1de3) (BuildId: 8dcd38b1d76bd7bd44f905c38200f4cceafd7ca4)
#1 0x72e446dd5b09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId: 1
eb6131419edb83b2178b682829a6913cf682d75)
#2 0x72e446db745a in g_hash_table_new_full
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4445a
) (BuildId: 1eb6131419edb83b2178b682829a6913cf682d75)
#3 0x58f3b7c6fc67 in object_initialize_with_type
/mnt/nvmedisk/linaro/qemu-from-laptop/qem
u/build/arm-asan/../../qom/object.c:568:23
#4 0x58f3b7c6f670 in object_initialize
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/ar
m-asan/../../qom/object.c:578:5
#5 0x58f3b7c6611b in qemu_init_irq
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/core/irq.c:48:5
#6 0x58f3b5c6e931 in pci_ich9_ahci_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/ide/ich.c:117:5
We could call qemu_free_irq() in pci_ich9_uninit(), but
since we have a method of initializing the IRQ that doesn't
need manual freeing, use that instead: qemu_init_irq_child().
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 9ffce135d28e9accdfea7327232f4ca34e8bacec
https://github.com/qemu/qemu/commit/9ffce135d28e9accdfea7327232f4ca34e8bacec
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/gpio/pca9554.c
Log Message:
-----------
hw/gpio/pca9554: Avoid leak in pca9554_set_pin()
In pca9554_set_pin() we have a string property which we parse in
order to set some non-string fields in the device state. So we call
visit_type_str(), passing it the address of the local variable
state_str.
visit_type_str() will allocate a new copy of the string; we
never free this string, so the result is a memory leak, detected
by ASAN during a "make check" run:
Direct leak of 5 byte(s) in 1 object(s) allocated from:
#0 0x5d605212ede3 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-arm+0x21f1de3)
(
BuildId: 3d5373c89317f58bfcd191a33988c7347714be14)
#1 0x7f7edea57b09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b68282
9a6913cf682d75)
#2 0x7f7edea6d4d8 in g_strdup
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x784d8) (BuildId:
1eb6131419edb83b2178b68282
9a6913cf682d75)
#3 0x5d6055289a91 in g_strdup_inline
/usr/include/glib-2.0/glib/gstrfuncs.h:321:10
#4 0x5d6055289a91 in qobject_input_type_str
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qapi/qo
bject-input-visitor.c:542:12
#5 0x5d605528479c in visit_type_str
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qapi/qapi-visit
-core.c:349:10
#6 0x5d60528bdd87 in pca9554_set_pin
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/gpio/pca9554.c:179:10
#7 0x5d60549bcbbb in object_property_set
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:1450:5
#8 0x5d60549d2055 in object_property_set_qobject
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/qom-qobject.c:28:10
#9 0x5d60549bcdf1 in object_property_set_str
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:1458:15
#10 0x5d605439d077 in gb200nvl_bmc_i2c_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/arm/aspeed.c:1267:5
#11 0x5d60543a3bbc in aspeed_machine_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/arm/aspeed.c:493:9
Make the state_str g_autofree, so that we will always free
it, on both error-exit and success codepaths.
Cc: [email protected]
Fixes: de0c7d543bca ("misc: Add a pca9554 GPIO device model")
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Glenn Miles <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: a275256595de5c80ee0cffe20c8792dfbef95abd
https://github.com/qemu/qemu/commit/a275256595de5c80ee0cffe20c8792dfbef95abd
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/char/max78000_uart.c
Log Message:
-----------
hw/char/max78000_uart: Destroy FIFO on deinit
In the max78000_uart we create a FIFO in the instance_init function,
but we don't destroy it on deinit, so ASAN reports a leak in the
device-introspect-test:
#0 0x561cc92d5de3 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-arm+0x21f1de3)
(BuildId: 98fdf9fc85c3beaeca8eda0be8412f1e11b9c6ad)
#1 0x70cbf2afab09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x561ccc4c884d in fifo8_create
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../util/fifo8.c:27:18
#3 0x561cc9744ec9 in max78000_uart_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/char/max78000_uart.c:241:5
Add an instance_finalize method to destroy the FIFO.
Cc: [email protected]
Fixes: d447e4b70295 ("MAX78000: UART Implementation")
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: fc6d9d0baa4bfcb3a461ec7b7dc3cc84d4f5ed9d
https://github.com/qemu/qemu/commit/fc6d9d0baa4bfcb3a461ec7b7dc3cc84d4f5ed9d
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/misc/xlnx-versal-cframe-reg.c
Log Message:
-----------
hw/misc/xlnx-versal-cframe-reg: Free FIFO, g_tree on deinit
In the xlnx-versal-cframe-reg device we create a FIFO in
instance_init but don't destroy it on deinit, causing ASAN
to report a leak in the device-introspect-test:
Direct leak of 400 byte(s) in 1 object(s) allocated from:
#0 0x5aded4d54e23 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x24ffe23)
(BuildId: 9f1e6c53fecd904ba5fc1f521d7da080a0e4103b)
#1 0x71fbfac9bb09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x5aded850059d in fifo8_create
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../util/fifo8.c:27:18
#3 0x5aded582b9e4 in fifo32_create
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/include/qemu/fifo32.h:35:5
#4 0x5aded582b326 in cframe_reg_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/misc/xlnx-versal-cframe-reg.c:693:5
Similarly, we don't clean up the g_tree we create:
Direct leak of 48 byte(s) in 1 object(s) allocated from:
#0 0x5aded4d54e23 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x24ffe23)
(BuildId: 9f1e6c5
3fecd904ba5fc1f521d7da080a0e4103b)
#1 0x71fbfac9bb09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x71fbfaccc799 in g_tree_new_full
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x93799) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d7
5)
#3 0x5aded582b21a in cframe_reg_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/misc/xlnx-versal-cframe-reg.c:691:18
Add an instance_finalize method to clean up what we
allocated in instance_init.
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Edgar E. Iglesias <[email protected]>
Reviewed-by: Manos Pitsidianakis <[email protected]>
Reviewed-by: Francisco Iglesias <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: c93638c10dfb6b3e8934b398c7ddb02480abe9f0
https://github.com/qemu/qemu/commit/c93638c10dfb6b3e8934b398c7ddb02480abe9f0
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/display/xlnx_dp.c
Log Message:
-----------
hw/display/xlnx_dp: Don't leak dpcd and edid objects
In the xnlx_dp_init() function we create the s->dpcd and
s->edid objects with qdev_new(); then in xlnx_dp_realize()
we realize the dpcd with qdev_realize() and the edid with
qdev_realize_and_unref().
This is inconsistent, and both ways result in a memory
leak for the instance_init -> deinit lifecycle tested
by device-introspect-test:
Indirect leak of 1968 byte(s) in 1 object(s) allocated from:
#0 0x5aded4d54e23 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x24ffe23)
(BuildId: 9f1e6c5
3fecd904ba5fc1f521d7da080a0e4103b)
#1 0x71fbfac9bb09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x5aded7b9211c in object_new_with_type
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:767:15
#3 0x5aded7b92240 in object_new
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:789:12
#4 0x5aded7b773e4 in qdev_new
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/core/qdev.c:149:19
#5 0x5aded54458be in xlnx_dp_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/display/xlnx_dp.c:1272:20
Direct leak of 344 byte(s) in 1 object(s) allocated from:
#0 0x5aded4d54e23 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x24ffe23)
(BuildId: 9f1e6c53fecd904ba5fc1f521d7da080a0e4103b)
#1 0x71fbfac9bb09 in g_malloc
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x62b09) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x5aded7b9211c in object_new_with_type
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:767:15
#3 0x5aded7b92240 in object_new
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:789:12
#4 0x5aded7b773e4 in qdev_new
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/core/qdev.c:149:19
#5 0x5aded5445a56 in xlnx_dp_init
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/display/xlnx_dp.c:1275:22
Instead, explicitly object_unref() after we have added the objects as
child properties of the device. This means they will automatically
be freed when this device is deinited. When we do this,
qdev_realize() is the correct way to realize them in
xlnx_dp_realize().
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Francisco Iglesias <[email protected]>
Reviewed-by: Manos Pitsidianakis <[email protected]>
Reviewed-by: Edgar E. Iglesias <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 63de7c6249144dd490343cff87c498cd4fd49966
https://github.com/qemu/qemu/commit/63de7c6249144dd490343cff87c498cd4fd49966
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M MAINTAINERS
M configs/devices/mips-softmmu/common.mak
M docs/about/deprecated.rst
M docs/about/removed-features.rst
M docs/system/target-mips.rst
M hw/mips/Kconfig
M hw/mips/meson.build
R hw/mips/mipssim.c
Log Message:
-----------
hw/mips: Remove mipssim machine
The "mipssim" machine is deprecated since commit facfc943cb9
("hw/mips: Mark the "mipssim" machine as deprecated"), released
in v10.0; time to remove.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Ján Tomko <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Reviewed-by: Jiaxun Yang <[email protected]>
Message-Id: <[email protected]>
Commit: 8d0352d4ca81c9b88652c5e2e88676b62bade426
https://github.com/qemu/qemu/commit/8d0352d4ca81c9b88652c5e2e88676b62bade426
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M MAINTAINERS
M hw/net/Kconfig
M hw/net/meson.build
R hw/net/mipsnet.c
M hw/net/trace-events
Log Message:
-----------
hw/net: Remove mipsnet device model
The mipsnet device model was only used by the mipssim machine,
which just got removed. Remove as now dead code.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Ján Tomko <[email protected]>
Reviewed-by: Richard Henderson <[email protected]>
Reviewed-by: Jiaxun Yang <[email protected]>
Message-Id: <[email protected]>
Commit: 8c81d38ea5ae2845d293d904d87a52b9ab6d26a2
https://github.com/qemu/qemu/commit/8c81d38ea5ae2845d293d904d87a52b9ab6d26a2
Author: Jan Kiszka <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/sd/sd.c
Log Message:
-----------
hw/sd/sdcard: Fix size check for backing block image
The power-of-2 rule applies to the user data area, not the complete
block image. The latter can be concatenation of boot partition images
and the user data.
Signed-off-by: Jan Kiszka <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID:
<0d95cffa6131a06404f3cd6ef5951720ce01c37e.1756019920.git.jan.kis...@siemens.com>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: d188a8edf4404539a6cce7fec057f30dbe57a928
https://github.com/qemu/qemu/commit/d188a8edf4404539a6cce7fec057f30dbe57a928
Author: Jan Kiszka <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/sd/sd.c
Log Message:
-----------
hw/sd/sdcard: Add validation for boot-partition-size
Make sure we are not silently rounding down or even wrapping around,
causing inconsistencies with the provided image.
Signed-off-by: Jan Kiszka <[email protected]>
Reviewed-by: Alex Bennée <[email protected]>
[PMD: Use g_autofree, suggested by Alex]
Message-ID:
<1fff448da042bdf8cff7733ce67cadff4c540f1d.1756706188.git.jan.kis...@siemens.com>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 039ba0aa1f758b25d345a141174af4fdd094ebb3
https://github.com/qemu/qemu/commit/039ba0aa1f758b25d345a141174af4fdd094ebb3
Author: Jan Kiszka <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/sd/sd.c
M hw/sd/sdmmc-internal.h
Log Message:
-----------
hw/sd/sdcard: Refactor sd_bootpart_offset
This function provides the offset for any partition in the block image,
not only the boot partitions, therefore rename it. Align the constant
names with the numbering scheme in the standard and use constants for
both boot partitions for consistency reasons. There is also no reason to
return early if boot_part_size is zero because the existing code will
provide the right value in that case as well.
Signed-off-by: Jan Kiszka <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID:
<66e9b07476aad61820c4f42f4f984cc90752ba5e.1756706188.git.jan.kis...@siemens.com>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: afaad1c59cc26b500460b8771c2f6241aeeb5c35
https://github.com/qemu/qemu/commit/afaad1c59cc26b500460b8771c2f6241aeeb5c35
Author: Jan Kiszka <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M crypto/hmac-gcrypt.c
M crypto/hmac-glib.c
M crypto/hmac-gnutls.c
M crypto/hmac-nettle.c
M include/crypto/hmac.h
Log Message:
-----------
crypto/hmac: Allow to build hmac over multiple qcrypto_gnutls_hmac_bytes[v]
calls
If the buffers that should be considered for building the hmac are not
available at the same time, the current API is unsuitable. Extend it so
that passing a NULL pointer as result_len is used as indicator that
further buffers will be passed in succeeding calls to
qcrypto_gnutls_hmac_bytes[v].
Signed-off-by: Jan Kiszka <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID:
<2d3539c247a6c323491a3821f0e5b6fc382a4686.1756706188.git.jan.kis...@siemens.com>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 190dd135670c1c06c3e394cd157dcaf1db8c9857
https://github.com/qemu/qemu/commit/190dd135670c1c06c3e394cd157dcaf1db8c9857
Author: Cédric Le Goater <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/arm/virt.c
Log Message:
-----------
hw/arm/virt: Include 'system/system.h'
hw/arm/virt.c should include 'system/system.h' for :
serial_hd()
qemu_add_machine_init_done_notifier()
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Peter Maydell <[email protected]>
Link: https://lore.kernel.org/qemu-devel/[email protected]
Signed-off-by: Cédric Le Goater <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 76f225f778327fe473f3e806b7e879a524bc6c2e
https://github.com/qemu/qemu/commit/76f225f778327fe473f3e806b7e879a524bc6c2e
Author: Cédric Le Goater <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/isa/isa-superio.c
Log Message:
-----------
hw/isa/superio: Include 'system/system.h'
Files using serial_hd() should include 'system/system.h'. Fix that.
Cc: Michael S. Tsirkin <[email protected]>
Cc: Paolo Bonzini <[email protected]>
Signed-off-by: Cédric Le Goater <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 60f1b05132a2549306006adba8076123a7dc821b
https://github.com/qemu/qemu/commit/60f1b05132a2549306006adba8076123a7dc821b
Author: Cédric Le Goater <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/mips/loongson3_virt.c
Log Message:
-----------
hw/mips/loongson3_virt: Include 'system/system.h'
Files using serial_hd() should include 'system/system.h'. Fix that.
Cc: Philippe Mathieu-Daudé <[email protected]>
Cc: Huacai Chen <[email protected]>
Cc: Jiaxun Yang <[email protected]>
Signed-off-by: Cédric Le Goater <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 0593f258e9a1974f9562bea9075d224b7f99174c
https://github.com/qemu/qemu/commit/0593f258e9a1974f9562bea9075d224b7f99174c
Author: Cédric Le Goater <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/mips/malta.c
Log Message:
-----------
hw/mips/malta: Include 'system/system.h'
Files using serial_hd() should include 'system/system.h'. Fix that.
Cc: Philippe Mathieu-Daudé <[email protected]>
Cc: Aurelien Jarno <[email protected]>
Cc: Jiaxun Yang <[email protected]>
Signed-off-by: Cédric Le Goater <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: af37c16c59a465254f92f065a666643f49a324de
https://github.com/qemu/qemu/commit/af37c16c59a465254f92f065a666643f49a324de
Author: Philippe Mathieu-Daudé <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M docs/about/removed-features.rst
Log Message:
-----------
docs/about/removed-features: Clarify 'device_add' is removed
All other titles in removed-features.rst mention when
the feature was removed using "removed in". Use that
instead of "since" which we use for when a feature is
deprecated.
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Daniel P. Berrangé <[email protected]>
Reviewed-by: Markus Armbruster <[email protected]>
Message-Id: <[email protected]>
Commit: c8f3f7757b09d641bbb2e6580619ad49d82d2201
https://github.com/qemu/qemu/commit/c8f3f7757b09d641bbb2e6580619ad49d82d2201
Author: Peter Maydell <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/arm/boot.c
Log Message:
-----------
hw/arm/boot: Correctly free the MemoryDeviceInfoList
When running the bios-tables-test under ASAN we see leaks like this:
Direct leak of 16 byte(s) in 1 object(s) allocated from:
#0 0x5bc58579b00d in calloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x250400d)
(BuildId: 2e27b63dc9ac45f522ced40a17c2a60cc32f1d38)
#1 0x7b4ad90337b1 in g_malloc0
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x637b1) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#2 0x5bc5861826db in qmp_memory_device_list
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/mem/memory-device.c:307:34
#3 0x5bc587a9edb6 in arm_load_dtb
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/arm/boot.c:656:15
Indirect leak of 28 byte(s) in 2 object(s) allocated from:
#0 0x5bc58579ae23 in malloc
(/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/qemu-system-aarch64+0x2503e23)
(BuildId: 2e27b63dc9ac45f522ced40a17c2a60cc32f1d38)
#1 0x7b4ad6c8f947 in __vasprintf_internal libio/vasprintf.c:116:16
#2 0x7b4ad9080a52 in g_vasprintf
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0xb0a52) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#3 0x7b4ad90515e4 in g_strdup_vprintf
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x815e4) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#4 0x7b4ad9051940 in g_strdup_printf
(/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x81940) (BuildId:
1eb6131419edb83b2178b682829a6913cf682d75)
#5 0x5bc5885eb739 in object_get_canonical_path
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../qom/object.c:2123:19
#6 0x5bc58618dca8 in pc_dimm_md_fill_device_info
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/mem/pc-dimm.c:268:18
#7 0x5bc586182792 in qmp_memory_device_list
/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/arm-asan/../../hw/mem/memory-device.c:310:9
This happens because we declared the MemoryDeviceInfoList *md_list
with g_autofree, which will free the direct memory with g_free() but
doesn't free all the other data structures referenced by it. Instead
what we want is to declare the pointer with g_autoptr(), which will
automatically call the qapi_free_MemoryDeviceInfoList() cleanup
function when the variable goes out of scope.
Fixes: 36bc78aca83cfd ("hw/arm: add static NVDIMMs in device tree")
Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Manos Pitsidianakis <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 3071c0ce5f55aeb9ec65fb20be1e762290371102
https://github.com/qemu/qemu/commit/3071c0ce5f55aeb9ec65fb20be1e762290371102
Author: Mark Cave-Ayland <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M hw/i386/pc_piix.c
Log Message:
-----------
hw/i386/pc_piix.c: remove unnecessary if() from pc_init1()
Now that the isapc logic has been split out of pc_piix.c, the PCI Host Bridge
(phb) object is now always set in pc_init1().
Since phb is now guaranteed not to be NULL, Coverity reports that the if()
statement surrounding ioapic_init_gsi() is now unnecessary and can be removed
along with the phb NULL initialiser.
Coverity: CID 1620557
Signed-off-by: Mark Cave-Ayland <[email protected]>
Fixes: 99d0630a45 ("hw/i386/pc_piix.c: assume pcmc->pci_enabled is always true
in pc_init1()")
Reviewed-by: Peter Maydell <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
Commit: 5537e253160c2fb977dea33defea987a752cc8ef
https://github.com/qemu/qemu/commit/5537e253160c2fb977dea33defea987a752cc8ef
Author: Richard Henderson <[email protected]>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M MAINTAINERS
M configs/devices/mips-softmmu/common.mak
M cpu-target.c
M crypto/hmac-gcrypt.c
M crypto/hmac-glib.c
M crypto/hmac-gnutls.c
M crypto/hmac-nettle.c
M docs/about/deprecated.rst
M docs/about/removed-features.rst
M docs/devel/style.rst
M docs/system/target-mips.rst
M hw/arm/boot.c
M hw/arm/virt.c
M hw/char/max78000_uart.c
M hw/char/serial-pci-multi.c
M hw/core/irq.c
M hw/display/xlnx_dp.c
M hw/gpio/pca9554.c
M hw/i386/pc_piix.c
M hw/ide/ich.c
M hw/intc/meson.build
M hw/isa/isa-superio.c
M hw/meson.build
M hw/mips/Kconfig
M hw/mips/loongson3_virt.c
M hw/mips/malta.c
M hw/mips/meson.build
R hw/mips/mipssim.c
M hw/misc/xlnx-versal-cframe-reg.c
M hw/net/Kconfig
M hw/net/e1000e_core.c
M hw/net/meson.build
R hw/net/mipsnet.c
M hw/net/trace-events
M hw/pci/pci.c
M hw/ppc/pnv.c
M hw/scsi/mptsas.c
M hw/sd/sd.c
M hw/sd/sdmmc-internal.h
M include/crypto/hmac.h
M include/elf.h
M include/exec/target_page.h
M include/hw/irq.h
M include/hw/sd/sd.h
M include/hw/ssi/ssi.h
M linux-user/mips/elfload.c
M meson.build
M migration/meson.build
A migration/vfio-stub.c
M migration/vfio.c
R page-target.c
M scripts/coverity-scan/COMPONENTS.md
M target-info-stub.c
M target/mips/tcg/system/tlb_helper.c
M target/ppc/cpu.h
M target/ppc/kvm.c
M target/ppc/misc_helper.c
Log Message:
-----------
Merge tag 'hw-misc-20250902' of https://github.com/philmd/qemu into staging
Misc HW patches
- Compile various system files once
- Remove SDCard spec v1.10
- Remove mipssim machine and mipsnet device model
- Prevent crash in e1000e when legacy interrupt fires after enabling MSI-X
- Introduce qemu_init_irq_child()
- Remove various memory leaks reported by ASan
- Few Coverity fixes
- Use 74Kf CPU to run MIPS16e binaries and M14Kc for microMIPS ones
# -----BEGIN PGP SIGNATURE-----
#
# iQIzBAABCAAdFiEE+qvnXhKRciHc/Wuy4+MsLN6twN4FAmi27GgACgkQ4+MsLN6t
# wN6SAxAAv1sdKCQ5ePK0XcXAc1EkrLzGBMVh1QB6bFGHfLngITAAdBv6tkG0r0rG
# pSmR2P81XrxOSY6DFzqVs7Mx42T30nN5paAt312xrhlGl1sbwdRkK/M17fcSr29B
# 1TBvIFJD8nxtBD5U0Okkf11iXUxKbfmg+j+it6HKhxK6uJ4kP7nb5U3vy52GO9h3
# o/1pdx6z+xqgtk1oWAIFav/TT86wliqFAZdhuxH0vWi0rDYv9ZCCKBcnljKqlesV
# u8qUJ6RcvbOH3S84SY9hxjafP0/kDcL9sZ22Ya+8uN3lyG6oDpRi9vFpRNf6tTlE
# Ys/uIBV5WKoEf8a2W2lsedFtU+Vp3oriqVhLd5tcOYgTvED+yCrgy8sY8T1hRb70
# wexvaOQ0lgG1GYS+Nvm0p+DK4p82lUPbf8yac99cQsssWBmnAW0cpsUnQ5Hfoio8
# WZP9k9r3Bqcjp7aV7CqNSmvMzT87qc/lwAAPErrmKhqNWlhMYT2OMoROnzWr6F9x
# wppTm/Dg2B+e9726wDODx7flrJZtoUN8IoWwA1DPuTUSyjLNobZM93V50mTK316O
# cIJ6CrCyYoau86MkvdrhJ5iGIyG2NPXdUnHug4wPbjYANx0X39jv8czy1lwzGoFt
# zOux6AIf0fAVFEPQXhZDOUtUgUU6dzrM4q/SKaFk3upPEuX7eQk=
# =4O2G
# -----END PGP SIGNATURE-----
# gpg: Signature made Tue 02 Sep 2025 03:08:56 PM CEST
# gpg: using RSA key FAABE75E12917221DCFD6BB2E3E32C2CDEADC0DE
# gpg: Good signature from "Philippe Mathieu-Daudé (F4BUG) <[email protected]>"
[unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg: There is no indication that the signature belongs to the owner.
# Primary key fingerprint: FAAB E75E 1291 7221 DCFD 6BB2 E3E3 2C2C DEAD C0DE
* tag 'hw-misc-20250902' of https://github.com/philmd/qemu: (39 commits)
hw/i386/pc_piix.c: remove unnecessary if() from pc_init1()
hw/arm/boot: Correctly free the MemoryDeviceInfoList
docs/about/removed-features: Clarify 'device_add' is removed
hw/mips/malta: Include 'system/system.h'
hw/mips/loongson3_virt: Include 'system/system.h'
hw/isa/superio: Include 'system/system.h'
hw/arm/virt: Include 'system/system.h'
crypto/hmac: Allow to build hmac over multiple qcrypto_gnutls_hmac_bytes[v]
calls
hw/sd/sdcard: Refactor sd_bootpart_offset
hw/sd/sdcard: Add validation for boot-partition-size
hw/sd/sdcard: Fix size check for backing block image
hw/net: Remove mipsnet device model
hw/mips: Remove mipssim machine
hw/display/xlnx_dp: Don't leak dpcd and edid objects
hw/misc/xlnx-versal-cframe-reg: Free FIFO, g_tree on deinit
hw/char/max78000_uart: Destroy FIFO on deinit
hw/gpio/pca9554: Avoid leak in pca9554_set_pin()
hw/ide/ich.c: Use qemu_init_irq_child() to avoid memory leak
hw/char/serial-pci-multi: Use qemu_init_irq_child() to avoid leak
hw/irq: New qemu_init_irq_child() function
...
Signed-off-by: Richard Henderson <[email protected]>
Compare: https://github.com/qemu/qemu/compare/8415b0619f65...5537e253160c
To unsubscribe from these emails, change your notification settings at
https://github.com/qemu/qemu/settings/notifications
