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 2 of 4] x86: make the pv-only e820 array be dynam

To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>, <Ian.Campbell@xxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, <Tim.Deegan@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 2 of 4] x86: make the pv-only e820 array be dynamic
From: Keir Fraser <keir@xxxxxxx>
Date: Tue, 12 Apr 2011 13:32:01 +0100
Cc:
Delivery-date: Tue, 12 Apr 2011 05:33:33 -0700
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:sender:user-agent:date:subject:from:to :message-id:thread-topic:thread-index:in-reply-to:mime-version :content-type:content-transfer-encoding; bh=Pq+IIaXCxBwltyegZQD1NnPHgn/u3qJBD462SGFwooE=; b=qy3zS1gYzDZrjAwKsO4MDbTwtgMwhoLR9lEbXlZ623pfMVuvYBL20FhtAtFG2iQ8fP f3XY9HhMxYJ2IvX6KZQEoOelDIGoSH/rMLXYxHk9s4oVZ9xlYVb0zcTUNoXLHMIeQiuB ueeKP5jycbxrpvK1aw4SuaQhxVkX3GYknNKgY=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=eAKeGYUtWFJrzFpO534kmhE99yPIEty+kWXzLCffx6LYeFePs9I4ujWfT06pAohwXv yEuYskEQQh5eD0aIJEfFl2op+0EvQ41WRcXcgpZPMkHb+/mh+VmsmsCuw3aMkkLCow9q CTIe3vlNckhJeHpTexlFzk1mmMjEWp+96D3uM=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <e7057fec103ba69776d1.1302557732@xxxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acv5DZ8jHFBylcxFw0u/ochvvhjmwQ==
Thread-topic: [Xen-devel] [PATCH 2 of 4] x86: make the pv-only e820 array be dynamic
User-agent: Microsoft-Entourage/12.28.0.101117
On 11/04/2011 22:35, "Konrad Rzeszutek Wilk" <konrad.wilk@xxxxxxxxxx> wrote:

> # HG changeset patch
> # User Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> # Date 1302295108 14400
> # Node ID e7057fec103ba69776d157d470d630ce99dbc540
> # Parent  0f279f43f41ffa8549e076f2a30f499bd8d6cc1d
> x86: make the pv-only e820 array be dynamic.
> 
> During creation of the PV domain we allocate the E820 structure to
> have the amount of E820 entries on the machine, plus the number three.

How cunning.

Why wouldn't you just allocate exactly the right size of array in
XENMEM_set_memory_map?

 -- Keir



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

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