[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] mips boston/malta: don't have nomigrate RAM
From: |
Paul Burton |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] mips boston/malta: don't have nomigrate RAM regions |
Date: |
Fri, 15 Jun 2018 10:14:38 -0700 |
User-agent: |
NeoMutt/20180512 |
Hi Peter,
On Fri, Jun 15, 2018 at 11:08:11AM +0100, Peter Maydell wrote:
> On 4 June 2018 at 12:03, Peter Maydell <address@hidden> 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 <address@hidden>
Thanks,
Paul