From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH] x86: make sure memory block is RAM before passing to the allocator Date: Mon, 04 Nov 2013 13:25:43 +0000 Message-ID: References: <5277842A02000078000FEF63@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1VdKAD-0006SB-QS for xen-devel@lists.xenproject.org; Mon, 04 Nov 2013 13:25:54 +0000 Received: by mail-wi0-f181.google.com with SMTP id ex4so488031wid.2 for ; Mon, 04 Nov 2013 05:25:52 -0800 (PST) In-Reply-To: <5277842A02000078000FEF63@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich , xen-devel List-Id: xen-devel@lists.xenproject.org On 04/11/2013 10:25, "Jan Beulich" wrote: > Memory blocks outside of the always visible 1:1 mapping range get > passed to the allocator separately (once enough other setup was done). > Skipping non-RAM regions, however, was forgotten in adc5afbf ("x86: > support up to 16Tb"). > > Signed-off-by: Jan Beulich Acked-by: Keir Fraser > --- a/xen/arch/x86/setup.c > +++ b/xen/arch/x86/setup.c > @@ -1155,6 +1155,8 @@ void __init __start_xen(unsigned long mb > { > uint64_t s, e; > > + if ( boot_e820.map[i].type != E820_RAM ) > + continue; > s = (boot_e820.map[i].addr + mask) & ~mask; > e = (boot_e820.map[i].addr + boot_e820.map[i].size) & ~mask; > if ( PFN_DOWN(e) <= limit ) > > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel