diff options
author | David Rientjes <rientjes@google.com> | 2010-08-09 17:18:57 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-08-09 20:44:57 -0700 |
commit | b940fd703572f7f9e5f894c682c91c3cbd84c11e (patch) | |
tree | cbb1f23073373ba1c425762f38f6f0bf9decf278 /Documentation | |
parent | 8e4228e1edb922afa83366803a1e5b3fa8e987c2 (diff) | |
download | linux-3.10-b940fd703572f7f9e5f894c682c91c3cbd84c11e.tar.gz linux-3.10-b940fd703572f7f9e5f894c682c91c3cbd84c11e.tar.bz2 linux-3.10-b940fd703572f7f9e5f894c682c91c3cbd84c11e.zip |
oom: remove unnecessary code and cleanup
Remove the redundancy in __oom_kill_task() since:
- init can never be passed to this function: it will never be PF_EXITING
or selectable from select_bad_process(), and
- it will never be passed a task from oom_kill_task() without an ->mm
and we're unconcerned about detachment from exiting tasks, there's no
reason to protect them against SIGKILL or access to memory reserves.
Also moves the kernel log message to a higher level since the verbosity is
not always emitted here; we need not print an error message if an exiting
task is given a longer timeslice.
__oom_kill_task() only has a single caller, so it can be merged into that
function at the same time.
Signed-off-by: David Rientjes <rientjes@google.com>
Reviewed-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions