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

[Xen-devel] RE: [PATCH 2/6] Change the boot allocator function to suppor

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH 2/6] Change the boot allocator function to support memory add
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Mon, 29 Jun 2009 10:04:39 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc:
Delivery-date: Sun, 28 Jun 2009 19:07:34 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C66D4CFD.E5A7%keir.fraser@xxxxxxxxxxxxx>
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: <E2263E4A5B2284449EEBD0AAB751098402CCF824F8@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C66D4CFD.E5A7%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acn30qSePbAyG89HR16wSczYgDMjIQAAzbqgAAIDDVAABiKE1wAAItCgAAPeiX4AFBqxUA==
Thread-topic: [PATCH 2/6] Change the boot allocator function to support memory add
 

>-----Original Message-----
>From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx] 
>Sent: 2009年6月28日 23:38
>To: Jiang, Yunhong; Jan Beulich; Tim Deegan; 
>xen-devel@xxxxxxxxxxxxxxxxxxx
>Subject: Re: [PATCH 2/6] Change the boot allocator function to 
>support memory add
>
>On 28/06/2009 14:59, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> wrote:
>
>>> I think the PGC_ flags could be extended to do this, yes.
>>> Maybe I will look
>>> into that.
>> 
>> Thanks. We try to do this way in the beginning , but in the 
>end gives up
>> because currently too many direct operation to count_info 
>without any macro
>> wrapper, and I'm not confident to take over ALL of them.
>
>This should already have been dealt with when 
>PGC_offlining/broken/offlined

Yes, I remember I did some cleanup on it before :$

>were added. Thus I think it should be straightforward to make 
>the change.
>Also this flag would only be set for pages which belong to the page
>allocator. Noone else would ever set the flag since they do 
>not own the bit.
>So I think the patch is straightforward to construct.
>
>Actually it'd be nice to get rid of the allocator bitmap 
>altogether, since

I checked the code and seems the main caller is map_pages_to_xen(). So do you 
mean you will setup the frametable earlier, so that all page allocator will use 
the main allocator, even in map_pages_to_xen()?

For memory add, it may be a bit different. The new pages should not be in main 
allocator before the m2p table is setup (in boot stage, the m2p will not be 
used till construct dom0, so it is ok) , also we may want to use the new memory 
as the page table when setup the m2p table.

Thanks
Yunhong Jiang

>hardly anyone uses it. That will just require some care in 
>bringing up the
>frametable and its mappings, which the main allocator is dependent on.
>
> -- Keir
>
>
>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel