[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] RE: [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日 21:44
>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:24, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> wrote:
>
>>> The undelrying implementation ideas may be okay, but calling these
>>> extensions to the 'boot allocator' makes no sense. The boot
>>> allocator only
>>> exists during boot. The fact that the bitmap it uses persists
>>> beyond boot is
>>> an implementation detail. Perhaps you just need to change the
>> 
>> So can I understand the allocator bitmap (i.e. those for 
>map_free/map_call) as
>> something not just about boot allocator, but for generic 
>bitmap to remember
>> the page allocation status?
>> Maybe we can just using one bit in page_info to indicate if 
>page is allocated
>> in future?
>
>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.

Shall our patch wait for your changes and do rebase, or shall I still working 
on the re-name method (I suspect the previous re-name patch may not be so 
important if we have this changes)?

Thanks
Yunhong Jiang

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.