On Sun, 1 Mar 2026 at 21:03, BALATON Zoltan <[email protected]> wrote: > > This is rarely needed and we can use > memory_region_init_ram_flags_nomigrate() instead which is now the only > nomigrate variant left. > > Signed-off-by: BALATON Zoltan <[email protected]>
I don't think this one is worth doing. We might as well wait until we can get rid of the VGA case. > diff --git a/hw/xtensa/xtfpga.c b/hw/xtensa/xtfpga.c > index d427d68e50..b025cc53a8 100644 > --- a/hw/xtensa/xtfpga.c > +++ b/hw/xtensa/xtfpga.c > @@ -163,8 +163,8 @@ static void xtfpga_net_init(MemoryRegion *address_space, > sysbus_mmio_get_region(s, 1)); > > ram = g_malloc(sizeof(*ram)); > - memory_region_init_ram_nomigrate(ram, OBJECT(s), "open_eth.ram", 16 * > KiB, > - &error_fatal); > + memory_region_init_ram_flags_nomigrate(ram, OBJECT(s), "open_eth.ram", > + 16 * KiB, 0, &error_fatal); > vmstate_register_ram_global(ram); > memory_region_add_subregion(address_space, buffers, ram); > } We should change this to memory_region_init_ram(), taking the compat break (which is fine for xtensa). thanks -- PMM
