[PATCH 2/3] drm/scheduler: Don't call wait_event_killable for signaled process.

Oleg Nesterov oleg at redhat.com
Wed Apr 25 13:22:28 UTC 2018


On 04/24, Daniel Vetter wrote:
>
> wait_event_killabel doesn't check for fatal_signal_pending before calling
> schedule, so definitely has a nice race there.

This is fine. See the signal_pending_state() check in __schedule().

And this doesn't differ from wait_event_interruptible(), it too doesn't
check signal_pending(), we rely on schedule() which must not block if the
caller is signalled/killed.

The problem is that it is not clear what should fatal_signal_pending() or
even signal_pending() mean after exit_signals().

Oleg.



More information about the amd-gfx mailing list