Messages in this thread |  | | | Date | Wed, 14 Mar 2012 16:14:04 +0530 | | Subject | Re: [PATCH 2/2] page_alloc: Remove argument to find_zone_movable_pfns_for_nodes | | From | Kautuk Consul <> |
| |
On Wed, Mar 14, 2012 at 4:12 PM, Kautuk Consul <consul.kautuk@gmail.com> wrote: > On Wed, Mar 14, 2012 at 2:17 AM, David Rientjes <rientjes@google.com> 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 from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |