[PATCH v6 0/8] memory: Remove most _nomigrate variants

BALATON Zoltan posted 8 patches 1 month, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1772572714.git.balaton@eik.bme.hu
Maintainers: Pierrick Bouvier <pierrick.bouvier@linaro.org>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Peter Xu <peterx@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>
There is a newer version of this series
docs/devel/memory.rst                         |   9 +-
hw/display/cg3.c                              |   5 +-
hw/display/tcx.c                              |   8 +-
hw/sparc/sun4m.c                              |  19 +-
hw/sparc64/sun4u.c                            |  10 +-
include/system/memory.h                       |  26 --
.../memory-region-housekeeping.cocci          |  47 ----
system/memory.c                               | 259 ++++++------------
8 files changed, 104 insertions(+), 279 deletions(-)
[PATCH v6 0/8] memory: Remove most _nomigrate variants
Posted by BALATON Zoltan 1 month, 1 week ago
v6:
- keep the last two uses of memory_region_init_ram_nomigrate in vga and xtfpga for now
- added R-b tags

v5:
- convert Sun machines and their display devices from global vmstate

v4:
- separate patch converting Sun machines from memory_region_init_ram_nomigrate
- split helper to init ram into two functions: setup and error_propagate
- also use memory_region_init_io in memory_region_init_ram_device_ptr

v3:
- rebased on master after some patches were merged
- drop some more line from memory-region-housekeeping.cocci
- added comment to explain what factored out helper does
- some more clean ups included

BALATON Zoltan (8):
  hw/display/{cg3.tcx}: Do not use memory_region_init_rom_nomigrate()
  memory: Remove memory_region_init_rom_nomigrate()
  sun4m,sun4u,tcx: Do not use memory_region_init_ram_nomigrate()
  system/memory: Reduce memory_region_init_ram_nomigrate() usage
  memory: Factor out common ram region initialization
  memory: Add internal memory_region_register_ram function
  memory: Shorten memory_region_init_ram_device_ptr and
    memory_region_init_rom_device
  memory: Factor out more common ram region initialization

 docs/devel/memory.rst                         |   9 +-
 hw/display/cg3.c                              |   5 +-
 hw/display/tcx.c                              |   8 +-
 hw/sparc/sun4m.c                              |  19 +-
 hw/sparc64/sun4u.c                            |  10 +-
 include/system/memory.h                       |  26 --
 .../memory-region-housekeeping.cocci          |  47 ----
 system/memory.c                               | 259 ++++++------------
 8 files changed, 104 insertions(+), 279 deletions(-)

-- 
2.41.3
Re: [PATCH v6 0/8] memory: Remove most _nomigrate variants
Posted by Peter Xu 1 month, 1 week ago
On Tue, Mar 03, 2026 at 10:47:49PM +0100, BALATON Zoltan wrote:
> BALATON Zoltan (8):
>   hw/display/{cg3.tcx}: Do not use memory_region_init_rom_nomigrate()
>   memory: Remove memory_region_init_rom_nomigrate()
>   sun4m,sun4u,tcx: Do not use memory_region_init_ram_nomigrate()

If no objections, I can queue these three patch (1-3) for 11.0-rc0 for now.

Thanks,

-- 
Peter Xu
Re: [PATCH v6 0/8] memory: Remove most _nomigrate variants
Posted by BALATON Zoltan 1 month, 1 week ago
On Wed, 4 Mar 2026, Peter Xu wrote:
> On Tue, Mar 03, 2026 at 10:47:49PM +0100, BALATON Zoltan wrote:
>> BALATON Zoltan (8):
>>   hw/display/{cg3.tcx}: Do not use memory_region_init_rom_nomigrate()
>>   memory: Remove memory_region_init_rom_nomigrate()
>>   sun4m,sun4u,tcx: Do not use memory_region_init_ram_nomigrate()
>
> If no objections, I can queue these three patch (1-3) for 11.0-rc0 for now.

What about the rest of this series? These are all simple clean ups so 
should not be hard to review. Patches 4-5 and 7-8 need review. Anybody?

Regards,
BALATON Zoltan