Re: [PATCH 1/3] xen/tools: Remove the XENMEM_get_oustanding_pages and provide the data via xc_phys_info

[Keir Fraser <keir.xen@gmail.com>]

On 10/05/2013 09:38, "Jan Beulich" <JBeulich@suse.com> wrote:

>>>> On 09.05.13 at 09:44, Ian Campbell <Ian.Campbell@citrix.com> wrote:
>> On Wed, 2013-05-08 at 19:35 +0100, Konrad Rzeszutek Wilk wrote:
>>> During the review of the patches it was noticed that there exists
>>> a race wherein the 'free_memory' value consists of information from
>>> two hypercalls. That is the XEN_SYSCTL_physinfo and
>> XENMEM_get_outstanding_pages.
>>> 
>>> The free memory the host has available for guest is the difference between
>>> the 'free_pages' (from XEN_SYSCTL_physinfo) and 'outstanding_pages'. As they
>>> are two hypercalls many things can happen in between the execution of them.
>>> 
>>> This patch resolves this by eliminating the XENMEM_get_outstanding_pages
>>> hypercall and providing the free_pages and outstanding_pages information
>>> via the xc_phys_info structure.
>>> 
>>> It also removes the XSM hooks and adds locking as needed.
>>> 
>>> CC: dgdera@tycho.nsa.gov
>>> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
>> 
>> For the tools side:
>> Acked-by: Ian Campbell <ian.campbell@citrix.com>
>> 
>> Needs a hypervisor ack though, since contrary to the subject line this
>> isn't just a tools change. Adding Keir, Tim & Jan (not sure which of
>> them is the right one here).
> 
> Looks okay to me (albeit quite late in the game for a bump of
> XEN_SYSCTL_INTERFACE_VERSION), but from a formal
> perspective Keir needs to ack this for it to go in.

Acked-by: Keir Fraser <keir@xen.org>

> Jan
>