On Fri, Apr 16, 2010 at 11:10:01AM -0500, Christoph Lameter wrote: > On Wed, 14 Apr 2010, Pekka Enberg wrote: > > > Minchan, care to send a v2 with proper changelog and reviewed-by attributions? > > Still wondering what the big deal about alloc_pages_node_exact is. Its not > exact since we can fall back to another node. It is better to clarify the > API for alloc_pages_node and forbid / clarify the use of -1. > There should be a comment clarifing it now. I admit the naming fault is mine. At the time, the intended meaning was "allocate pages from any node in the fallback list and the caller knows exactly which node to start from". I did not take into account that the meaning of "exact" depends on context. With a comment clarifying the meaning, I do not think a rename is necessary. However, I'd rather not see a mass renaming of functions like alloc_pages() that have existed a long times. If nothing else, they are documented in books like "Linux Device Drivers" so why make life harder on device authors than it already is? -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>