If the request is not to create root group and we fail to meet it, we'd leave the root unchanged. Signed-off-by: Hillf Danton <dhillf@xxxxxxxxx> --- --- a/mm/memcontrol.c Fri Dec 9 21:57:40 2011 +++ b/mm/memcontrol.c Sun Dec 11 09:04:48 2011 @@ -4849,8 +4849,10 @@ mem_cgroup_create(struct cgroup_subsys * enable_swap_cgroup(); parent = NULL; root_mem_cgroup = memcg; - if (mem_cgroup_soft_limit_tree_init()) + if (mem_cgroup_soft_limit_tree_init()) { + root_mem_cgroup = NULL; goto free_out; + } for_each_possible_cpu(cpu) { struct memcg_stock_pcp *stock = &per_cpu(memcg_stock, cpu); @@ -4888,7 +4890,6 @@ mem_cgroup_create(struct cgroup_subsys * return &memcg->css; free_out: __mem_cgroup_free(memcg); - root_mem_cgroup = NULL; return ERR_PTR(error); } -- 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=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>