[PATCH 0/3] sparc: fix minor memory leaks

Peter Maydell posted 3 patches 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260307112931.3322532-1-peter.maydell@linaro.org
Maintainers: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>
hw/sparc/sun4m.c        | 34 +++++++++++++++++-----------------
hw/timer/slavio_timer.c | 25 ++++++++++++++++++-------
2 files changed, 35 insertions(+), 24 deletions(-)
[PATCH 0/3] sparc: fix minor memory leaks
Posted by Peter Maydell 1 month ago
This patchset fixes some minor memory leaks in sparc targets
that show up when running "make check" with the address sanitizer
enabled. None of the leaks are important in themselves, as they
are leaks that happen once at startup of memory that must in
any case stay around for the lifetime of the QEMU process. But
getting rid of them helps in moving toward the goal of a completely
error-free asan run, which would let us guard against more
interesting leaks being added in future. It also removes a couple
of uses of qemu_allocate_irqs(), which is an API I think ideally
we would get rid of altogether eventually.

These patches are enough for a clean pass of 'make check'
and 'make check-functional' and 'make check-tcg' for the sparc
targets with asan enabled.

thanks
-- PMM

Peter Maydell (3):
  hw/sparc/sun4m: Don't set up dummy cpu interrupts
  hw/sparc/sun4m: Use qdev GPIOs rather than qemu_allocate_irqs()
  hw/timer/slavio_timer: Free resources allocated in instance_init

 hw/sparc/sun4m.c        | 34 +++++++++++++++++-----------------
 hw/timer/slavio_timer.c | 25 ++++++++++++++++++-------
 2 files changed, 35 insertions(+), 24 deletions(-)

-- 
2.43.0
Re: [PATCH 0/3] sparc: fix minor memory leaks
Posted by Philippe Mathieu-Daudé 1 month ago
On 7/3/26 12:29, Peter Maydell wrote:

> Peter Maydell (3):
>    hw/sparc/sun4m: Don't set up dummy cpu interrupts
>    hw/sparc/sun4m: Use qdev GPIOs rather than qemu_allocate_irqs()
>    hw/timer/slavio_timer: Free resources allocated in instance_init

Series queued, thanks.