signal: kill the obsolete SIGNAL_UNKILLABLE check in complete_signal()
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 3 patches 3 repliesIn review
Patches in v1 No applies recorded yet
None of the 3 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/3] signal: kill the obsolete SIGNAL_UNKILLABLE check in complete_signal()
Quoted 5 lines · Oleg Nesterov
Cough... and on the second thought this patch needs v2. Sorry Andrew, please drop signal-kill-the-obsolete-signal_unkillable-check-in-complete_signal.patch I'll send the updated version.
Quoted 8 lines
With this patch the parent namespace can use any fatal signal (not only SIGKILL) to kill the init process in container. I do not think this is actually bad, but in any case this should not silently come as a side effect. And this is not consistent with SIGNAL_UNKILLABLE/sig_kernel_only() check in
get_signal().Most probably I will just resend this patch as 2/2, while 1/2 will change
sig_task_ignored()because afaics it is not actually right too (albeit not really buggy).
So this replaces signal-kill-the-obsolete-signal_unkillable-check-in-complete_signal.patch 2/3 is actually the same change, just the changelog was updated.
I'll try to send the cleanups tomorrow. sig_task_ignored() doesn't look nice, sig_handler_ignored() must die, SIGNAL_UNKILLABLE checks in sig_task_ignored() and get_signal() should be unified.
And to remind, we need to fix more problems I found yesterday.
Oleg.