diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2023-09-09 19:26:29 +0200 |
|---|---|---|
| committer | Sasha Levin <sashal@kernel.org> | 2024-03-15 10:48:18 -0400 |
| commit | e24772adaaf4b81ac0855cceb17080352526f765 (patch) | |
| tree | 5e6d72cc9786961b64a6d48f54848d59f13fa212 /kernel | |
| parent | 33ec341e3e9588962ff3cf49f642da140d3ecfc0 (diff) | |
| download | linux-stable-e24772adaaf4b81ac0855cceb17080352526f765.tar.gz linux-stable-e24772adaaf4b81ac0855cceb17080352526f765.tar.bz2 linux-stable-e24772adaaf4b81ac0855cceb17080352526f765.zip | |
getrusage: use __for_each_thread()
[ Upstream commit 13b7bc60b5353371460a203df6c38ccd38ad7a3a ]
do/while_each_thread should be avoided when possible.
Plus this change allows to avoid lock_task_sighand(), we can use rcu
and/or sig->stats_lock instead.
Link: https://lkml.kernel.org/r/20230909172629.GA20454@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>
Stable-dep-of: f7ec1cd5cc7e ("getrusage: use sig->stats_lock rather than lock_task_sighand()")
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/sys.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/kernel/sys.c b/kernel/sys.c index ab621d7837c9..521a76d4adbb 100644 --- a/kernel/sys.c +++ b/kernel/sys.c @@ -1760,10 +1760,8 @@ void getrusage(struct task_struct *p, int who, struct rusage *r) r->ru_oublock += sig->oublock; if (maxrss < sig->maxrss) maxrss = sig->maxrss; - t = p; - do { + __for_each_thread(sig, t) accumulate_thread_rusage(t, r); - } while_each_thread(p, t); break; default: |