Hi Peter, On Fri, Jun 15, 2018 at 11:08:11AM +0100, Peter Maydell wrote: > On 4 June 2018 at 12:03, Peter Maydell <peter.mayd...@linaro.org> wrote: > > This patchset fixes a a bug in the MIPS boston and malta boards: > > they currently use memory_region_init_rom_nomigrate() to create > > memory regions for their BIOS/flash, and they don't manually > > register the MR with vmstate_register_ram() either. This currently > > means that its contents are migrated, but as a RAM block whose > > name is the empty string; in future (when a patch which is currently > > in the outstanding migration tree pull request hits master) it > > will mean they are not migrated at all. Switch to using > > memory_region_init_ram() instead, so the memory contents are > > migrated with a reasonable name. > > > > Note that this is a cross-version migration compatibility break > > for both machines, but: > > (a) migration will break anyway due to the "don't migrate > > unnamed RAM blocks" patch > > (b) neither machine is versioned, so we don't really care about > > maintaining cross-version migration compatibility AFAIK > > Any comment from the MIPS maintainers? > > thanks > -- PMM
FWIW, this looks fine to me for both Boston & Malta: Reviewed-by: Paul Burton <paul.bur...@mips.com> Thanks, Paul