Re: [patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-15 Thread Kamezawa Hiroyuki
(2012/11/14 18:15), David Rientjes wrote: out_of_memory() will already cause current to schedule if it has not been killed, so doing it again in pagefault_out_of_memory() is redundant. Remove it. Cc: KAMEZAWA Hiroyuki Cc: KOSAKI Motohiro Cc: Michal Hocko Signed-off-by: David Rientjes

Re: [patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-15 Thread Kamezawa Hiroyuki
(2012/11/14 18:15), David Rientjes wrote: out_of_memory() will already cause current to schedule if it has not been killed, so doing it again in pagefault_out_of_memory() is redundant. Remove it. Cc: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com Cc: KOSAKI Motohiro

Re: [patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-14 Thread Michal Hocko
On Wed 14-11-12 01:15:25, David Rientjes wrote: > out_of_memory() will already cause current to schedule if it has not been > killed, so doing it again in pagefault_out_of_memory() is redundant. > Remove it. > > Cc: KAMEZAWA Hiroyuki > Cc: KOSAKI Motohiro > Cc: Michal Hocko > Signed-off-by:

[patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-14 Thread David Rientjes
out_of_memory() will already cause current to schedule if it has not been killed, so doing it again in pagefault_out_of_memory() is redundant. Remove it. Cc: KAMEZAWA Hiroyuki Cc: KOSAKI Motohiro Cc: Michal Hocko Signed-off-by: David Rientjes --- mm/oom_kill.c |1 - 1 files changed, 0

[patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-14 Thread David Rientjes
out_of_memory() will already cause current to schedule if it has not been killed, so doing it again in pagefault_out_of_memory() is redundant. Remove it. Cc: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com Cc: KOSAKI Motohiro kosaki.motoh...@jp.fujitsu.com Cc: Michal Hocko mho...@suse.cz

Re: [patch 3/4] mm, oom: remove redundant sleep in pagefault oom handler

2012-11-14 Thread Michal Hocko
On Wed 14-11-12 01:15:25, David Rientjes wrote: out_of_memory() will already cause current to schedule if it has not been killed, so doing it again in pagefault_out_of_memory() is redundant. Remove it. Cc: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com Cc: KOSAKI Motohiro