From: Matthias Kaehlcke <matthias.kaehlcke@gmail.com>
Date: Fri, 19 Oct 2007 06:39:58 +0000 (-0700)
Subject: mm/oom_kill.c: Use list_for_each_entry instead of list_for_each
X-Git-Tag: v2.6.24-rc1~228
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=7b1915a989ea4d426d0fd98974ab80f30ef1d779;p=~emulex%2Finfiniband.git

mm/oom_kill.c: Use list_for_each_entry instead of list_for_each

mm/oom_kill.c: Convert list_for_each to list_for_each_entry in
oom_kill_process()

Signed-off-by: Matthias Kaehlcke <matthias.kaehlcke@gmail.com>
Cc: Nick Piggin <nickpiggin@yahoo.com.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/mm/oom_kill.c b/mm/oom_kill.c
index b1c2d0f8622..a7cb4c43fd1 100644
--- a/mm/oom_kill.c
+++ b/mm/oom_kill.c
@@ -337,7 +337,6 @@ static int oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order,
 			    unsigned long points, const char *message)
 {
 	struct task_struct *c;
-	struct list_head *tsk;
 
 	if (printk_ratelimit()) {
 		printk(KERN_WARNING "%s invoked oom-killer: "
@@ -360,8 +359,7 @@ static int oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order,
 					message, p->pid, p->comm, points);
 
 	/* Try to kill a child first */
-	list_for_each(tsk, &p->children) {
-		c = list_entry(tsk, struct task_struct, sibling);
+	list_for_each_entry(c, &p->children, sibling) {
 		if (c->mm == p->mm)
 			continue;
 		if (!oom_kill_task(c))