qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v2 0/6] Memory API mutators


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH v2 0/6] Memory API mutators
Date: Wed, 07 Dec 2011 09:56:44 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.21) Gecko/20110831 Lightning/1.0b2 Thunderbird/3.1.13

On 12/07/2011 09:54 AM, Avi Kivity wrote:
On 12/07/2011 05:52 PM, Anthony Liguori wrote:
On 12/04/2011 12:09 PM, Avi Kivity wrote:
This patchset introduces memory_region_set_enabled() and
memory_region_set_address() to avoid the requirement on memory
routers to track the internal state of the memory API (so they know
whether they need to add or remove a region).  Instead, they can
simply copy the state of the region from the guest-exposed register
to the memory core, via the new mutator functions.

Based on previous discussions, any time these functions are used, the
caller more than likely needs to call them again in a post_load hook
during restore, correct?

It would be good to document this very clearly in the header docs for
each function.

It's a layering violation, but I'll add something.

Ok, let's add a docs/vmstate.txt and add a section that says "If you use the following functions, you probably need to call them again in post_load" and put these functions on the list.

Regards,

Anthony Liguori






reply via email to

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