diff options
| author |   Mateusz Guzik <mjguzik@gmail.com> | 2025-02-06 17:44:11 +0100 |
|---|---|---|
| committer |   Christian Brauner <brauner@kernel.org> | 2025-02-07 11:22:43 +0100 |
| commit | 6731cd97e60d6f3a30057c0fc513aed543187104 (patch) | |
| tree | 10e56d9ba6de1b363633fa2db2c6db843b43e410 | |
| parent | exit: perform add_device_randomness() without tasklist_lock (diff) | |
| download | wireguard-linux-6731cd97e60d6f3a30057c0fc513aed543187104.tar.xz wireguard-linux-6731cd97e60d6f3a30057c0fc513aed543187104.zip | |
exit: hoist get_pid() in release_task() outside of tasklist_lock
Reduces hold time as get_pid() contains an atomic.
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
Link: https://lore.kernel.org/r/20250206164415.450051-3-mjguzik@gmail.com
Acked-by: "Liam R. Howlett" <Liam.Howlett@Oracle.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | kernel/exit.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index a00273db024b..ebc1c5bec512 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -240,9 +240,10 @@ repeat: cgroup_release(p); + thread_pid = get_pid(p->thread_pid); + write_lock_irq(&tasklist_lock); ptrace_release_task(p); - thread_pid = get_pid(p->thread_pid); __exit_signal(p); /* |