[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [qemu-s390x] [Qemu-devel] [PATCH RFC 1/2] memory: make MemoryRegion

From: Dr. David Alan Gilbert
Subject: Re: [qemu-s390x] [Qemu-devel] [PATCH RFC 1/2] memory: make MemoryRegion alias migratable
Date: Tue, 30 Jul 2019 16:41:43 +0100
User-agent: Mutt/1.12.0 (2019-05-25)

* Igor Mammedov (address@hidden) wrote:
> On Tue, 30 Jul 2019 15:34:54 +0200
> Paolo Bonzini <address@hidden> wrote:
> > On 30/07/19 15:25, Igor Mammedov wrote:
> > > I'd guess you've meant RAMBlocks instead of memory regions, if that's it
> > > then yes, every alias pointing to RAM backed memory region will have
> > > RAMBlock that's points to aliased part of aliased memory region.
> > 
> > The question is just, does it break migration from old QEMU to new QEMU
> > on x86 (which has plenty of RAM-backed aliases)?  If not, explain that
> > in the commit message or in a code comment.
> pc/q35 old<->new new<->old ping pong tests worked fine (qemu with default 
> devices).
> My understanding was that one needs to call vmstate_register_ram()
> to make memory region migratable, which marks specific RAMBlock
> as migratable. So just assigning new RAMBlock to mr->ram_block shouldn't
> affect migration, unless vmstate_register_ram() is called on alias.

You're right; that's actually relatively new and I forget about it.
 From Cedric's b895de50271 in 3.0, before that we migrated every

So yes, I think you should be OK.


> I'll add it to commit message.
> > 
> > Paolo
> > 
Dr. David Alan Gilbert / address@hidden / Manchester, UK

reply via email to

[Prev in Thread] Current Thread [Next in Thread]