Re: [Xen-devel] [PATCH 4/6] xc: XENMEM_claim_pages outstanding claims value

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-03-11 at 14:20 +0000, Konrad Rzeszutek Wilk wrote:
> From: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
> 
> This is patch provides the value of claimed pages but
> not yet accounted for in a domain (outstanding). This is a
> total global value that influences the hypervisors' MM system.
> This value - when the guest has finally been created and is
> running - ends up having the value zero. But during the
> memory populate calls the contents of this value will
> decrease.

I think you are trying to say that it jumps to some large value when a
claim is made, and then reduces as the domain's memory is populated and
eventually reaches zero (either through allocations or the claim being
released)?

> 
> Signed-off-by: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
> [v2: s/unclaimed_pages/outstanding_pages/ per Tim's suggestion]
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> ---
>  tools/libxc/xc_domain.c | 1 +
>  tools/libxc/xenctrl.h   | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/tools/libxc/xc_domain.c b/tools/libxc/xc_domain.c
> index b8a345c..ab6dde5 100644
> --- a/tools/libxc/xc_domain.c
> +++ b/tools/libxc/xc_domain.c
> @@ -234,6 +234,7 @@ int xc_domain_getinfo(xc_interface *xch,
>  
>          info->ssidref  = domctl.u.getdomaininfo.ssidref;
>          info->nr_pages = domctl.u.getdomaininfo.tot_pages;
> +        info->nr_outstanding_pages = 
> domctl.u.getdomaininfo.outstanding_pages;
>          info->nr_shared_pages = domctl.u.getdomaininfo.shr_pages;
>          info->nr_paged_pages = domctl.u.getdomaininfo.paged_pages;
>          info->max_memkb = domctl.u.getdomaininfo.max_pages << 
> (PAGE_SHIFT-10);
> diff --git a/tools/libxc/xenctrl.h b/tools/libxc/xenctrl.h
> index e695456..2a4d4df 100644
> --- a/tools/libxc/xenctrl.h
> +++ b/tools/libxc/xenctrl.h
> @@ -364,6 +364,7 @@ typedef struct xc_dominfo {
>                    hvm:1, debugged:1;
>      unsigned int  shutdown_reason; /* only meaningful if shutdown==1 */
>      unsigned long nr_pages; /* current number, not maximum */
> +    unsigned long nr_outstanding_pages;
>      unsigned long nr_shared_pages;
>      unsigned long nr_paged_pages;
>      unsigned long shared_info_frame;



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel