diff options
| author |   Oleg Nesterov <oleg@redhat.com> | 2023-09-09 23:49:51 +0200 |
|---|---|---|
| committer |   Andrew Morton <akpm@linux-foundation.org> | 2023-10-04 10:41:57 -0700 |
| commit | ed5378a387fd7c382497f2abcf4605e030b64044 (patch) | |
| tree | bfe39f3100e0e514c46fc189b11d56b6bdbde892 /drivers/dma-buf | |
| parent | getrusage: use __for_each_thread() (diff) | |
| download | wireguard-linux-ed5378a387fd7c382497f2abcf4605e030b64044.tar.xz wireguard-linux-ed5378a387fd7c382497f2abcf4605e030b64044.zip | |
taskstats: fill_stats_for_tgid: use for_each_thread()
do/while_each_thread should be avoided when possible.
Plus I _think_ this change allows to avoid lock_task_sighand() but I am
not sure, I forgot everything about taskstats. In any case, this code
does not look right in that the same thread can be accounted twice:
taskstats_exit() can account the exiting thread in signal->stats and drop
->siglock but this thread is still on the thread-group list, so
lock_task_sighand() can't help.
Link: https://lkml.kernel.org/r/20230909214951.GA24274@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 'drivers/dma-buf')
0 files changed, 0 insertions, 0 deletions