All of lore.kernel.org
 help / color / mirror / Atom feed
From: BALATON Zoltan <balaton@eik.bme.hu>
To: qemu-devel@nongnu.org
Cc: "Peter Xu" <peterx@redhat.com>,
	"Akihiko Odaki" <odaki@rsg.ci.i.u-tokyo.ac.jp>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Mark Cave-Ayland" <mark.cave-ayland@ilande.co.uk>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Max Filippov" <jcmvbkbc@gmail.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>
Subject: [PATCH v3 0/6] Implement memory_region_new_* functions
Date: Tue, 10 Feb 2026 21:02:08 +0100 (CET)	[thread overview]
Message-ID: <cover.1770753117.git.balaton@eik.bme.hu> (raw)

Our documentation says that memory regions are automatically freed
when the owner dies and the reference counting to do this is also
implemented. However this relies on the QOM free funtion that can only
be set by creating objects with object_new but memory API only
provides constructors that call object_initialize which clears the
free function that prevents QOM to manage the memory region lifetime.
Implement corresponding memory_region_new_* functions that do the same
as the memory_region_init_* functions but create the memory region
with object_new so the lifetime can be automatically managed by QOM as
documented. The memory_region_init functions are kept because they are
useful for memory regions embedded in other object or managed
externally and not by QOM for some reason.

v3:
Based-on: <cover.1770753117.git.balaton@eik.bme.hu>
- convert xtfpga from nomigrate as this has no migration compatibility
issue

v2:
- rebase on master
- update documentation
- use these function to fix some leaks (there may be more, e.g. in
hw/pci-host/bonito but I leave that for later and/or others)

BALATON Zoltan (6):
  memory: Add memory_region_new* functions
  memory: Update documentation for memory_region_new*()
  hw/ide/sii3112: Use memory_region_new to avoid leaking regions
  hw/pci-host/articia: Map PCI memory windows in realize
  hw/pci-host/articia: Add variable for common type cast
  hw/xtensa/xtfpga: Fix leaking memory region

 docs/devel/memory.rst   |  21 ++---
 hw/ide/sii3112.c        |  30 +++----
 hw/pci-host/articia.c   |  22 +++--
 hw/ppc/amigaone.c       |  28 ++-----
 hw/ppc/pegasos.c        |  13 ---
 hw/xtensa/xtfpga.c      |   7 +-
 include/system/memory.h | 179 +++++++++++++++++++++++++++++++++++++++
 system/memory.c         | 181 ++++++++++++++++++++++++++++++++++++++++
 8 files changed, 407 insertions(+), 74 deletions(-)

-- 
2.41.3



             reply	other threads:[~2026-02-10 20:03 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-10 20:02 BALATON Zoltan [this message]
2026-02-10 20:02 ` [PATCH v3 1/6] memory: Add memory_region_new* functions BALATON Zoltan
2026-02-10 20:02 ` [PATCH v3 2/6] memory: Update documentation for memory_region_new*() BALATON Zoltan
2026-02-10 20:02 ` [PATCH v3 3/6] hw/ide/sii3112: Use memory_region_new to avoid leaking regions BALATON Zoltan
2026-02-10 20:02 ` [PATCH v3 4/6] hw/pci-host/articia: Map PCI memory windows in realize BALATON Zoltan
2026-02-10 20:02 ` [PATCH v3 5/6] hw/pci-host/articia: Add variable for common type cast BALATON Zoltan
2026-02-10 20:02 ` [PATCH v3 6/6] hw/xtensa/xtfpga: Fix leaking memory region BALATON Zoltan
2026-02-10 20:58 ` [PATCH v3 0/6] Implement memory_region_new_* functions BALATON Zoltan
2026-02-11  7:00 ` Paolo Bonzini
2026-02-11 11:47   ` BALATON Zoltan
2026-02-11 14:35     ` Peter Xu
2026-02-12 17:37       ` BALATON Zoltan
2026-02-14 13:44         ` BALATON Zoltan
2026-02-14 13:55           ` BALATON Zoltan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1770753117.git.balaton@eik.bme.hu \
    --to=balaton@eik.bme.hu \
    --cc=jcmvbkbc@gmail.com \
    --cc=kraxel@redhat.com \
    --cc=mark.cave-ayland@ilande.co.uk \
    --cc=odaki@rsg.ci.i.u-tokyo.ac.jp \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=peterx@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.