From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 4/4] xen: do not create the extra e820 region at an addr lower than 4G Date: Tue, 12 Apr 2011 12:48:06 -0400 Message-ID: <20110412164806.GC1067@dumpdata.com> References: <1302607192-21355-4-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1302607192-21355-4-git-send-email-stefano.stabellini@eu.citrix.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: stefano.stabellini@eu.citrix.com Cc: jeremy@goop.org, xen-devel@lists.xensource.com, linux-kernel@vger.kernel.org, mingo@elte.hu, hpa@linux.intel.com, yinghai@kernel.org List-Id: xen-devel@lists.xenproject.org On Tue, Apr 12, 2011 at 12:19:52PM +0100, stefano.stabellini@eu.citrix.com wrote: > From: Stefano Stabellini > > Do not add the extra e820 region at a physical address lower than 4G > because it breaks e820_end_of_low_ram_pfn(). > > It is OK for us to move the xen_extra_mem_start up and down because this > is the index of the memory that can be ballooned in/out - it is memory > not available to the kernel during bootup. OK. Stuck it on rc3 queue. > > Signed-off-by: Stefano Stabellini > --- > arch/x86/xen/setup.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c > index 9c38bd1..a51e010 100644 > --- a/arch/x86/xen/setup.c > +++ b/arch/x86/xen/setup.c > @@ -229,7 +229,7 @@ char * __init xen_memory_setup(void) > > memcpy(map_raw, map, sizeof(map)); > e820.nr_map = 0; > - xen_extra_mem_start = mem_end; > + xen_extra_mem_start = max((1ULL << 32), mem_end); > for (i = 0; i < memmap.nr_entries; i++) { > unsigned long long end; > > -- > 1.7.2.3