[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 0/9] Fix check-qtest-ppc64 sanitizer errors
From: |
Akihiko Odaki |
Subject: |
[PATCH v3 0/9] Fix check-qtest-ppc64 sanitizer errors |
Date: |
Mon, 08 Jul 2024 15:55:11 +0900 |
Based-on:
<3ad18bc590ef28e1526e8053568086b453e7ffde.1718211878.git.quic_mathbern@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.odaki@daynix.com>
---
Changes in v3:
- Added patch "memory: Clarify that we use owner's reference count".
- Added patch "memory: Refer to docs/devel/memory.rst for 'owner'".
- Fixed the message of patch
"memory: Do not create circular reference with subregion".
- Dropped patch "cpu: Free cpu_ases" in favor of:
https://lore.kernel.org/r/20240607115649.214622-7-salil.mehta@huawei.com/
("[PATCH V13 6/8] physmem: Add helper function to destroy CPU
AddressSpace")
- Dropped patches "hw/ide: Convert macio ide_irq into GPIO line" and
"hw/ide: Remove internal DMA qemu_irq" in favor of commit efb359346c7a
("hw/ide/macio: switch from using qemu_allocate_irq() to qdev input
GPIOs")
- Dropped patch "hw/isa/vt82c686: Define a GPIO line between vt82c686
and i8259" in favor of:
20240704205854.18537-1-shentey@gmail.com/">https://patchew.org/QEMU/20240704205854.18537-1-shentey@gmail.com/
("[PATCH 0/3] Resolve vt82c686 and piix4 qemu_irq memory leaks")
- Dropped pulled patches.
- Link to v2:
https://lore.kernel.org/r/20240627-san-v2-0-750bb0946dbd@daynix.com
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-f3cc42302189@daynix.com
---
Akihiko Odaki (9):
spapr: Free stdout path
ppc/vof: Fix unaligned FDT property access
migration: Free removed SaveStateEntry
memory: Do not refer to "memory region's reference count"
memory: Refer to docs/devel/memory.rst for "owner"
memory: Clarify that owner may be missing
memory: Clarify owner must not call memory_region_ref()
memory: Do not create circular reference with subregion
tests/qtest: Delete previous boot file
include/exec/memory.h | 22 +++++++---------------
hw/ppc/spapr_vof.c | 2 +-
hw/ppc/vof.c | 2 +-
migration/savevm.c | 2 ++
system/memory.c | 11 +++++++++--
tests/qtest/migration-test.c | 18 +++++++++++-------
6 files changed, 31 insertions(+), 26 deletions(-)
---
base-commit: f2cb4026fccfe073f84a4b440e41d3ed0c3134f6
change-id: 20240625-san-097afaf4f1c2
Best regards,
--
Akihiko Odaki <akihiko.odaki@daynix.com>
- [PATCH v3 0/9] Fix check-qtest-ppc64 sanitizer errors,
Akihiko Odaki <=
- [PATCH v3 1/9] spapr: Free stdout path, Akihiko Odaki, 2024/07/08
- [PATCH v3 2/9] ppc/vof: Fix unaligned FDT property access, Akihiko Odaki, 2024/07/08
- [PATCH v3 3/9] migration: Free removed SaveStateEntry, Akihiko Odaki, 2024/07/08
- [PATCH v3 4/9] memory: Do not refer to "memory region's reference count", Akihiko Odaki, 2024/07/08
- [PATCH v3 5/9] memory: Refer to docs/devel/memory.rst for "owner", Akihiko Odaki, 2024/07/08
- [PATCH v3 6/9] memory: Clarify that owner may be missing, Akihiko Odaki, 2024/07/08
- [PATCH v3 8/9] memory: Do not create circular reference with subregion, Akihiko Odaki, 2024/07/08
- [PATCH v3 7/9] memory: Clarify owner must not call memory_region_ref(), Akihiko Odaki, 2024/07/08
- [PATCH v3 9/9] tests/qtest: Delete previous boot file, Akihiko Odaki, 2024/07/08