diff options
author | Oleg Nesterov <oleg@redhat.com> | 2023-09-09 18:45:01 +0200 |
---|---|---|
committer | Andrew Morton <akpm@linux-foundation.org> | 2023-10-04 19:41:57 +0200 |
commit | 7904e53ed5a20fc678c01d5d1b07ec486425bb6a (patch) | |
tree | daa45f2a8cdbf8b2697ba0291e93a3bb634d5420 /fs/proc | |
parent | scsi: qla2xxx: use DEFINE_SHOW_STORE_ATTRIBUTE() helper for debugfs (diff) | |
download | linux-7904e53ed5a20fc678c01d5d1b07ec486425bb6a.tar.xz linux-7904e53ed5a20fc678c01d5d1b07ec486425bb6a.zip |
fs/proc: do_task_stat: use __for_each_thread()
do/while_each_thread should be avoided when possible.
Link: https://lkml.kernel.org/r/20230909164501.GA11581@redhat.com
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'fs/proc')
-rw-r--r-- | fs/proc/array.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/fs/proc/array.c b/fs/proc/array.c index 2c2efbe685d8..ff08a8957552 100644 --- a/fs/proc/array.c +++ b/fs/proc/array.c @@ -536,12 +536,13 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, /* add up live thread stats at the group level */ if (whole) { - struct task_struct *t = task; - do { + struct task_struct *t; + + __for_each_thread(sig, t) { min_flt += t->min_flt; maj_flt += t->maj_flt; gtime += task_gtime(t); - } while_each_thread(task, t); + } min_flt += sig->min_flt; maj_flt += sig->maj_flt; |