From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH] Include in balloon page tracking only PFNs which are marked as RAM in E820. Date: Wed, 19 Aug 2009 15:16:07 -0400 Message-ID: <20090819191607.GA4798@phenom.dumpdata.com> References: <1250709149-4730-1-git-send-email-konrad.wilk@oracle.com> <1250709149-4730-2-git-send-email-konrad.wilk@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1250709149-4730-2-git-send-email-konrad.wilk@oracle.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel@lists.xensource.com Cc: gianluca.guida@citrix.com, Konrad Rzeszutek Wilk List-Id: xen-devel@lists.xenproject.org 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 > > Signed-off-by: Konrad Rzeszutek Wilk > --- > 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