Skip to content

cleanup/fix allow_signal/disallow_signal

Oleg Nesterov lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11signals: kill sigfindinword()arch/m68k/include/asm/signal.h arch/x86/include/asm/signal.h include/linux/signal.h+0 -202 replies
2/11signals: s/siginitset/sigemptyset/ in do_sigtimedwait()kernel/signal.c+1 -10 replies
3/11signals: kill rm_from_queue(), change prepare_signal() to use for_each_thread()kernel/signal.c+10 -330 replies
4/11signals: rename rm_from_queue_full() to flush_sigqueue_mask()kernel/signal.c+8 -110 replies
5/11signals: cleanup the usage of t/current in do_sigaction()kernel/signal.c+7 -80 replies
6/11signals: mv {dis,}allow_signal() from sched.h/exit.c to signal.[ch]include/linux/sched.h include/linux/signal.h kernel/exit.c +1 more+31 -420 replies
7/11signals: jffs2: fix the wrong usage of disallow_signal()fs/jffs2/background.c+7 -50 replies
8/11signals: kill the obsolete sigdelset() and recalc_sigpending() in allow_signal()kernel/signal.c+1 -40 replies
9/11signals: disallow_signal() should flush the potentially pending signalkernel/signal.c+7 -00 replies
10/11signals: introduce kernel_sigaction()include/linux/signal.h kernel/signal.c+28 -260 replies
11/11signals: change wait_for_helper() to use kernel_sigaction()kernel/kmod.c+1 -40 replies

Reviews trailers per patch

1 of 11 patches has a Reviewed-by or Acked-by.

PatchGeert Uytterhoeven
1/11A-b
2/11
3/11
4/11
5/11
6/11
7/11
8/11
9/11
10/11
11/11

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 00/11] cleanup/fix allow_signal/disallow_signal

Hello.

allow_signal() and disallow_signal() have the very strange (wrong actually) semantics. They mix/confuse "sigprocmask" and "sigaction" logic.

This series tries to cleanup and fix them, plus other (mostly related) cleanups.

Oleg.

 arch/m68k/include/asm/signal.h |    9 ----
 arch/x86/include/asm/signal.h  |    6 ---
 fs/jffs2/background.c          |   12 +++--
 include/linux/sched.h          |    3 -
 include/linux/signal.h         |   21 +++++++--
 kernel/exit.c                  |   39 -----------------
 kernel/kmod.c                  |    5 +--
 kernel/signal.c                |   90 ++++++++++++++++++---------------------
 8 files changed, 66 insertions(+), 119 deletions(-)

Read the cover letter in its thread