Messages in this thread Patch in this message | | | From | KOSAKI Motohiro <> | Subject | [PATCH 06/10] oom: cleanup has_intersects_mems_allowed() | Date | Tue, 8 Jun 2010 20:59:06 +0900 (JST) |
| |
Now has_intersects_mems_allowed() has own thread iterate logic, but it should use while_each_thread().
It slightly improve the code readability.
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> --- mm/oom_kill.c | 8 +++----- 1 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/mm/oom_kill.c b/mm/oom_kill.c index b88172c..8376ad1 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -38,16 +38,14 @@ static DEFINE_SPINLOCK(zone_scan_lock); /* * Is all threads of the target process nodes overlap ours? */ -static int has_intersects_mems_allowed(struct task_struct *tsk) +static int has_intersects_mems_allowed(struct task_struct *p) { - struct task_struct *t; + struct task_struct *t = p; - t = tsk; do { if (cpuset_mems_allowed_intersects(current, t)) return 1; - t = next_thread(t); - } while (t != tsk); + } while_each_thread(p, t); return 0; } -- 1.6.5.2
| |