pid: perform free_pid() calls outside of tasklist_lock

Browse source

As the clone side already executes pid allocation with only pidmap_lock
held, issuing free_pid() while still holding tasklist_lock exacerbates
total hold time of the latter.

More things may show up later which require initial clean up with the
lock held and allow finishing without it. For that reason a struct to
collect such work is added instead of merely passing the pid array.

Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
Link: https://lore.kernel.org/r/20250206164415.450051-5-mjguzik@gmail.com
Acked-by: "Liam R. Howlett" <Liam.Howlett@Oracle.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>

...

This commit is contained in:

Mateusz Guzik 2025-02-06 17:44:13 +01:00 • committed by Christian Brauner

parent 74198dc206

commit 7903f907a2

No known key found for this signature in database

GPG key ID: 91C61BC06578DCA2

4 changed files with 55 additions and 38 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

7

include/linux/pid.h

View file

28

kernel/exit.c

View file

44

kernel/pid.c

View file

14

kernel/sys.c

View file