diff options
author | Oleg Nesterov <oleg@redhat.com> | 2014-01-23 18:55:53 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-01-23 19:37:02 -0500 |
commit | 185ee40ee7fd1ecfc6575e8cefa2331218d1eca2 (patch) | |
tree | 093d2616894ebc7e31970b3040615f0e0dd8d283 /fs/proc | |
parent | 98611e4e6a2b4a03fd2d4750cce8e4455a995c8d (diff) |
fs/proc/array.c: change do_task_stat() to use while_each_thread()
Change the remaining next_thread (ab)users to use while_each_thread().
The last user which should be changed is next_tid(), but we can't do this
now.
__exit_signal() and complete_signal() are fine, they actually need
next_thread() logic.
This patch (of 3):
do_task_stat() can use while_each_thread(), no changes in
the compiled code.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Kees Cook <keescook@chromium.org>
Reviewed-by: Sameer Nanda <snanda@chromium.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/proc')
-rw-r--r-- | fs/proc/array.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/proc/array.c b/fs/proc/array.c index 554a0b229ac2..656e401794de 100644 --- a/fs/proc/array.c +++ b/fs/proc/array.c | |||
@@ -444,8 +444,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, | |||
444 | min_flt += t->min_flt; | 444 | min_flt += t->min_flt; |
445 | maj_flt += t->maj_flt; | 445 | maj_flt += t->maj_flt; |
446 | gtime += task_gtime(t); | 446 | gtime += task_gtime(t); |
447 | t = next_thread(t); | 447 | } while_each_thread(task, t); |
448 | } while (t != task); | ||
449 | 448 | ||
450 | min_flt += sig->min_flt; | 449 | min_flt += sig->min_flt; |
451 | maj_flt += sig->maj_flt; | 450 | maj_flt += sig->maj_flt; |