On Thu, Jun 27, 2024 at 10:37:43PM +0900, Akihiko Odaki wrote:
> Based-on:
> <3ad18bc590ef28e1526e8053568086b453e7ffde.1718211878.git.quic_mathb...@quicinc.com>
> ("[PATCH] cpu: fix memleak of 'halt_cond' and 'thread'")
>
> I saw various sanitizer errors when running check-qtest-ppc64. While
> I could just turn off sanitizers, I decided to tackle them this time.
>
> Unfortunately, GLib does not free test data in some cases so some
> sanitizer errors remain. All sanitizer errors will be gone with this
> patch series combined with the following change for GLib:
> https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4120
>
> Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com>
Reviewed-by: Michael S. Tsirkin <m...@redhat.com>
who's merging all this?
> ---
> Changes in v2:
> - Rebased to "[PATCH] cpu: fix memleak of 'halt_cond' and 'thread'".
> (Philippe Mathieu-Daudé)
> - Converted IRQs into GPIO lines and removed one qemu_irq usage.
> (Peter Maydell)
> - s/suppresses/fixes/ (Michael S. Tsirkin)
> - Corrected title of patch "hw/virtio: Free vqs after vhost_dev_cleanup()"
> (was "hw/virtio: Free vqs before vhost_dev_cleanup()")
> - Link to v1:
> https://lore.kernel.org/r/20240626-san-v1-0-f3cc42302...@daynix.com
>
> ---
> Akihiko Odaki (15):
> cpu: Free cpu_ases
> hw/ide: Convert macio ide_irq into GPIO line
> hw/ide: Remove internal DMA qemu_irq
> hw/isa/vt82c686: Define a GPIO line between vt82c686 and i8259
> spapr: Free stdout path
> ppc/vof: Fix unaligned FDT property access
> hw/virtio: Free vqs after vhost_dev_cleanup()
> migration: Free removed SaveStateEntry
> memory: Do not create circular reference with subregion
> tests/qtest: Use qtest_add_data_func_full()
> tests/qtest: Free unused QMP response
> tests/qtest: Free old machine variable name
> tests/qtest: Delete previous boot file
> tests/qtest: Free paths
> tests/qtest: Free GThread
>
> include/hw/ppc/mac_dbdma.h | 5 +++--
> hw/core/cpu-common.c | 1 +
> hw/ide/macio.c | 18 +++++++++++++-----
> hw/isa/vt82c686.c | 7 ++++---
> hw/misc/macio/mac_dbdma.c | 10 +++++-----
> hw/ppc/spapr_vof.c | 2 +-
> hw/ppc/vof.c | 2 +-
> hw/virtio/vhost-user-base.c | 2 ++
> migration/savevm.c | 2 ++
> system/memory.c | 11 +++++++++--
> tests/qtest/device-introspect-test.c | 7 +++----
> tests/qtest/libqtest.c | 3 +++
> tests/qtest/migration-test.c | 18 +++++++++++-------
> tests/qtest/qos-test.c | 16 ++++++++++++----
> tests/qtest/vhost-user-test.c | 6 +++---
> 15 files changed, 73 insertions(+), 37 deletions(-)
> ---
> base-commit: af799a2337c3e39994411f90631905d809a41da4
> change-id: 20240625-san-097afaf4f1c2
>
> Best regards,
> --
> Akihiko Odaki <akihiko.od...@daynix.com>
