On 01/18/2013 07:25 PM, Michal Hocko wrote: >> - spin_lock_init(&memcg->move_lock); >> > + memcg->swappiness = mem_cgroup_swappiness(parent); > Please move this up to oom_kill_disable and use_hierarchy > initialization. One thing: wouldn't moving it to inside use_hierarchy be a change of behavior here? -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html