From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 2 of 5] x86: make the pv-only e820 array be dynamic Date: Fri, 8 Apr 2011 09:21:13 -0400 Message-ID: <20110408132113.GC6189@dumpdata.com> References: <01d0d338b97491a3aa81.1302207923@localhost6.localdomain6> <1302250925.27835.50.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1302250925.27835.50.camel@zakaz.uk.xensource.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: Ian Campbell Cc: "xen-devel@lists.xensource.com" , "keir.fraser@eu.citrix.com" List-Id: xen-devel@lists.xenproject.org On Fri, Apr 08, 2011 at 09:22:05AM +0100, Ian Campbell wrote: > On Thu, 2011-04-07 at 21:25 +0100, Konrad Rzeszutek Wilk wrote: > > # HG changeset patch > > # User Konrad Rzeszutek Wilk > > # Date 1302202697 14400 > > # Node ID 01d0d338b97491a3aa816dab43cc709a234214f7 > > # Parent decab6c21cc3d7ce4d4dad949d34ba35d4600490 > > x86: make the pv-only e820 array be dynamic. > > > > During creation of the PV domain we allocate the E820 structure to be > > the E820MAX. This will allow the tool stack to fill the E820 with more > > than three entries. > > Is it possible to defer this allocation to set_memory_map time so we can > allocate only the required size? I guess E820MAX (128) entries is only > 2.5k but a typical e820 is capped at more like 20 entries in practice > (<0.5k). The later patch does that. If I figure out how to "squash" the hg commits together I can post it as one patch..