From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Tue, 15 Apr 2008 11:10:38 +0900 From: KAMEZAWA Hiroyuki Subject: [PATCH] use vmalloc for mem_cgroup allocation. v2 Message-Id: <20080415111038.ffac0e12.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <20080415105434.3044afb6.kamezawa.hiroyu@jp.fujitsu.com> References: <20080415105434.3044afb6.kamezawa.hiroyu@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: KAMEZAWA Hiroyuki Cc: Andrew Morton , "balbir@linux.vnet.ibm.com" , "xemul@openvz.org" , lizf@cn.fujitsu.com, menage@google.com, "linux-mm@kvack.org" , LKML List-ID: On ia64, this kmalloc() requires order-4 pages. But this is not necessary to be phisically contiguous. (and x86-32, which has small vmalloc area, has small mem_cgroup struct.) For here, vmalloc is better. Changelog: v1->v2 - added memset(). Signed-off-by: KAMEZAWA Hiroyuki Index: mm-2.6.25-rc8-mm2/mm/memcontrol.c =================================================================== --- mm-2.6.25-rc8-mm2.orig/mm/memcontrol.c +++ mm-2.6.25-rc8-mm2/mm/memcontrol.c @@ -31,6 +31,7 @@ #include #include #include +#include #include @@ -992,8 +993,10 @@ mem_cgroup_create(struct cgroup_subsys * if (unlikely((cont->parent) == NULL)) { mem = &init_mem_cgroup; page_cgroup_cache = KMEM_CACHE(page_cgroup, SLAB_PANIC); - } else - mem = kzalloc(sizeof(struct mem_cgroup), GFP_KERNEL); + } else { + mem = vmalloc(sizeof(struct mem_cgroup)); + memset(mem, 0, sizeof(*mem)); + } if (mem == NULL) return ERR_PTR(-ENOMEM); @@ -1011,7 +1014,7 @@ free_out: for_each_node_state(node, N_POSSIBLE) free_mem_cgroup_per_zone_info(mem, node); if (cont->parent != NULL) - kfree(mem); + vfree(mem); return ERR_PTR(-ENOMEM); } @@ -1031,7 +1034,7 @@ static void mem_cgroup_destroy(struct cg for_each_node_state(node, N_POSSIBLE) free_mem_cgroup_per_zone_info(mem, node); - kfree(mem_cgroup_from_cont(cont)); + vfree(mem_cgroup_from_cont(cont)); } static int mem_cgroup_populate(struct cgroup_subsys *ss, -- 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