Re: [PATCH 10/13] signal: Move set_notify_signal and clear_notify_signal into sched/signal.h

Kees Cook
2022-03-09T22:37:19Z

On Wed, Mar 09, 2022 at 10:24:51AM -0600, Eric W. Biederman wrote:
> The header tracehook.h is no place for code to live. The functions
> set_notify_signal and clear_notify_signal are not about signals. They
> are about interruptions that act like signals. The fundamental signal
> primitives wind up calling set_notify_signal and clear_notify_signal.
> Which means they need to be maintained with the signal code.
>
> Since set_notify_signal and clear_notify_signal must be maintained
> with the signal subsystem move them into sched/signal.h and claim them
> as part of the signal subsystem.
>
> Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>

Reviewed-by: Kees Cook <keescook@chromium.org>

--
Kees Cook