Kurosawa,

I like the idea of multiple controllers for a resource. Users will have
options to choose from. Thanks for doing it.

I have few questions:
 - how are shared pages handled ?
 - what is the plan to support "limit" ?
 - can you provide more information in stats ?
 - is it designed to work with cpumeter alone (i.e without ckrm) ?

comment/suggestion:
 - IMO, moving pages from a class at time of reclassification would be
   the right thing to do. May be we have to add a pointer to Chris patch
   and make sure it works as we expect.
 - instead of adding the pseudo zone related code to the core memory
   files, you can put them in a separate file.
 - Documentation on how to configure and use it would be good.
  
regards,

chandra
  
On Tue, 2006-01-31 at 11:30 +0900, KUROSAWA Takahiro wrote:
> I've split the patches into smaller pieces in order to increase
> readability.  The core part of the patchset is the fifth one with
> the subject "Add the pzone_create() function."
> 
> Changes since the last post:
> * Fixed a bug that pages allocated with __GFP_COLD are incorrectly handled.
> * Moved the PZONE bit in page flags next to the zone number bits in 
>   order to make changes by pzones smaller.
> * Moved the nr_zones locking functions outside of the CONFIG_PSEUDO_ZONE
>   because they are not directly related to pzones.
> 
> Thanks,
> 



-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to