From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Mon, 05 May 2008 01:17:08 +0900 From: KOSAKI Motohiro Subject: [-mm][PATCH] make inlining to __alloc_pages() Message-Id: <20080505011158.8F6E.KOSAKI.MOTOHIRO@jp.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Return-Path: To: Lee Schermerhorn , Mel Gorman , LKML , linux-mm , Andrew Morton Cc: kosaki.motohiro@jp.fujitsu.com List-ID: Two zonelist patch series rewriten __page_alloc() largely. Now, it is just wrapper function. thus, that change inline function is better. CC: Lee Schermerhorn CC: Mel Gorman Signed-off-by: KOSAKI Motohiro --- include/linux/gfp.h | 21 +++++++++++++++++---- mm/page_alloc.c | 18 +----------------- 2 files changed, 18 insertions(+), 21 deletions(-) Index: b/include/linux/gfp.h =================================================================== --- a/include/linux/gfp.h 2008-05-05 01:32:00.000000000 +0900 +++ b/include/linux/gfp.h 2008-05-05 01:38:35.000000000 +0900 @@ -173,11 +173,24 @@ static inline void arch_free_page(struct static inline void arch_alloc_page(struct page *page, int order) { } #endif -extern struct page *__alloc_pages(gfp_t, unsigned int, struct zonelist *); +struct page * +__alloc_pages_internal(gfp_t gfp_mask, unsigned int order, + struct zonelist *zonelist, nodemask_t *nodemask); + +static inline struct page * +__alloc_pages(gfp_t gfp_mask, unsigned int order, + struct zonelist *zonelist) +{ + return __alloc_pages_internal(gfp_mask, order, zonelist, NULL); +} + +static inline struct page * +__alloc_pages_nodemask(gfp_t gfp_mask, unsigned int order, + struct zonelist *zonelist, nodemask_t *nodemask) +{ + return __alloc_pages_internal(gfp_mask, order, zonelist, nodemask); +} -extern struct page * -__alloc_pages_nodemask(gfp_t, unsigned int, - struct zonelist *, nodemask_t *nodemask); static inline struct page *alloc_pages_node(int nid, gfp_t gfp_mask, unsigned int order) Index: b/mm/page_alloc.c =================================================================== --- a/mm/page_alloc.c 2008-05-05 01:32:28.000000000 +0900 +++ b/mm/page_alloc.c 2008-05-05 01:38:22.000000000 +0900 @@ -1528,7 +1528,7 @@ static void set_page_owner(struct page * /* * This is the 'heart' of the zoned buddy allocator. */ -static struct page * +struct page * __alloc_pages_internal(gfp_t gfp_mask, unsigned int order, struct zonelist *zonelist, nodemask_t *nodemask) { @@ -1736,22 +1736,6 @@ got_pg: return page; } -struct page * -__alloc_pages(gfp_t gfp_mask, unsigned int order, - struct zonelist *zonelist) -{ - return __alloc_pages_internal(gfp_mask, order, zonelist, NULL); -} - -struct page * -__alloc_pages_nodemask(gfp_t gfp_mask, unsigned int order, - struct zonelist *zonelist, nodemask_t *nodemask) -{ - return __alloc_pages_internal(gfp_mask, order, zonelist, nodemask); -} - -EXPORT_SYMBOL(__alloc_pages); - /* * Common helper functions. */ -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org