Re: [PATCH] Include in balloon page tracking only PFNs which are marked as RAM in E820.

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

Gianluca,

I had not done any extensive testing. Would appreciate your feedback on this patch.

On Wed, Aug 19, 2009 at 03:12:29PM -0400, Konrad Rzeszutek Wilk wrote:
> From: Konrad Rzeszutek Wilk <kliw@darnok.org>
> 
> Signed-off-by: Konrad Rzeszutek Wilk <kliw@darnok.org>
> ---
>  drivers/xen/balloon.c |    8 ++++++--
>  1 files changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
> index 168e61f..ef2edd8 100644
> --- a/drivers/xen/balloon.c
> +++ b/drivers/xen/balloon.c
> @@ -416,8 +416,12 @@ static int __init balloon_init(void)
>  
>  	/* Initialise the balloon with excess memory space. */
>  	for (pfn = xen_start_info->nr_pages; pfn < max_pfn; pfn++) {
> -		page = pfn_to_page(pfn);
> -		balloon_append(page);
> +   		int is_ram = page_is_ram(pfn);
> +		if (is_ram) {
> +			page = pfn_to_page(pfn);
> +			if (PageReserved(page))
> +				balloon_append(page);
> +		}
>  	}
>  
>  	target_watch.callback = watch_target;
> -- 
> 1.6.2.5
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel