On Tue, 2 Mar 2010 16:01:41 -0800 (PST) David Rientjes <rientjes@xxxxxxxxxx> wrote: > On Tue, 2 Mar 2010, KAMEZAWA Hiroyuki wrote: > > > > Kame said earlier it would be no problem to rebase his memcg oom work on > > > mmotm if my patches were merged. > > > > > > > But I also said this patch cause regression. > > This patch causes a regression??? You never said that in any of your > reviews and I have no idea what you're talking about, this patch simply > cleans up the code and closes a race where VM_FAULT_OOM could needlessly > kill tasks in parallel oom conditions. > try_set_system_oom() is not called in memory_cgroup_out_of_memory() path. Then, oom kill twice. Thanks, -Kame -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>