Re: [Xen-devel] [PATCH 2 of 4] x86: make the pv-only e820 array be dynamic

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Tue, Apr 12, 2011 at 01:32:01PM +0100, Keir Fraser wrote:
> 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?

I was thinking about it, but the mm.c code did not have the
xen/xmalloc.h header, nor any references to xmalloc_array.

Is it OK to make an xmalloc_array during a hypercall?

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