WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] arch/x86/xen/setup: Fix for incorrect xen_extra_

To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] arch/x86/xen/setup: Fix for incorrect xen_extra_mem_start initialization ifdef CONFIG_X86_32
From: Daniel Kiper <dkiper@xxxxxxxxxxxx>
Date: Mon, 16 May 2011 09:46:54 +0200
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Daniel Kiper <dkiper@xxxxxxxxxxxx>, stefano.stabellini@xxxxxxxxxxxxx
Delivery-date: Mon, 16 May 2011 00:47:55 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20110512185808.GA26281@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20110511203438.GA21034@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20110512185808.GA26281@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.3.28i
On Thu, May 12, 2011 at 02:58:08PM -0400, Konrad Rzeszutek Wilk wrote:
> On Wed, May 11, 2011 at 10:34:38PM +0200, Daniel Kiper wrote:
> > git commit 24bdb0b62cc82120924762ae6bc85afc8c3f2b26 (xen: do not create
> > the extra e820 region at an addr lower than 4G) does not take into
> > account that ifdef CONFIG_X86_32 instead of e820_end_of_low_ram_pfn()
> > find_low_pfn_range() is called (both calls are from 
> > arch/x86/kernel/setup.c).
> > find_low_pfn_range() behaves correctly and does not require change in
> > xen_extra_mem_start initialization. Additionally, if xen_extra_mem_start
> > is initialized in the same way as ifdef CONFIG_X86_64 then memory hotplug
> > support for Xen balloon driver (under development) is broken.
>
> in linux-next.

Thanks. Additionally, I would like to mention that git commit 
279b706bf800b5967037f492dbe4fc5081ad5d0f
(x86,xen: introduce x86_init.mapping.pagetable_reserve) which fixes bug 
introduced by
git commit 4b239f458c229de044d6905c2b0f9fe16ed9e01e (x86-64, mm: Put early page 
table
high) works on my machine.

Daniel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>