]> git.ipfire.org Git - thirdparty/linux.git/commit
signal/drbd: Use send_sig not force_sig
authorEric W. Biederman <ebiederm@xmission.com>
Tue, 5 Feb 2019 23:25:02 +0000 (17:25 -0600)
committerEric W. Biederman <ebiederm@xmission.com>
Mon, 27 May 2019 14:36:28 +0000 (09:36 -0500)
commitfee109901f392a96189bd84a41845535a52bd94d
tree88dda887810d94133bb5bce1d5cbc2e872621f1a
parent82e10af2248d2d09c99834613f1b47d5002dc379
signal/drbd: Use send_sig not force_sig

The drbd module exclusively sends signals to kernel threads it creates with
kthread_create.  These kernel threads do not block or ignore signals (only
flush signals after they have been delivered), nor can drbd threads
possibly be pid namespace init processes so the extra work that force_sig
performs that send_sig does not is unnecessary.

Further force_sig is for delivering synchronous signals (aka exceptions).
The locking in force_sig is not prepared to deal with running processes, as
tsk->sighand may change during exec for a running process.

In short it is not only unnecessary for drbd to use force_sig it is
semantically wrong.

With drbd using send_sig it becomes easier to maintain force_sig as only
synchronous signals need to be considered.

Cc: Philipp Reisner <philipp.reisner@linbit.com>
Cc: Lars Ellenberg <lars.ellenberg@linbit.com>
Cc: drbd-dev@lists.linbit.com
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
drivers/block/drbd/drbd_int.h
drivers/block/drbd/drbd_main.c
drivers/block/drbd/drbd_nl.c