On Wed, Mar 14, 2012 at 4:12 PM, Kautuk Consul <consul.kautuk@xxxxxxxxx> wrote: > On Wed, Mar 14, 2012 at 2:17 AM, David Rientjes <rientjes@xxxxxxxxxx> wrote: >> On Tue, 13 Mar 2012, Kautuk Consul wrote: >> >>> The find_zone_movable_pfns_for_nodes() function does not utiilize >>> the argument to it. >>> >> >> It could, though, if we made it to do so. >> >>> Removing this argument from the function prototype as well as its >>> caller, i.e. free_area_init_nodes(). >>> >> >> Not sure if we'd ever want it or not for other purposes, but >> find_zone_movable_pfns_for_nodes() could easily be made to use the passed >> in array rather than zone_movable_pfn in file scope directly. That seems >> to be why it took an argument in the first place. >> > > No function is calling this function and I just wanted to remove the > slight overhead of passing an > argument which does not get used. Sorry.. I meant : no other function oter than free_area_init_nodes() calls this function. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href