Re: [PATCH v12 04/13] task_isolation: add initial support

From: Peter Zijlstra
Date: Wed May 18 2016 - 09:34:46 EST


On Tue, Apr 05, 2016 at 01:38:33PM -0400, Chris Metcalf wrote:
> diff --git a/kernel/signal.c b/kernel/signal.c
> index aa9bf00749c1..53e4e62f2778 100644
> --- a/kernel/signal.c
> +++ b/kernel/signal.c
> @@ -34,6 +34,7 @@
> #include <linux/compat.h>
> #include <linux/cn_proc.h>
> #include <linux/compiler.h>
> +#include <linux/isolation.h>
>
> #define CREATE_TRACE_POINTS
> #include <trace/events/signal.h>
> @@ -2213,6 +2214,9 @@ relock:
> /* Trace actually delivered signals. */
> trace_signal_deliver(signr, &ksig->info, ka);
>
> + /* Disable task isolation when delivering a signal. */

Why !? Changelog is quiet on this.

> + task_isolation_set_flags(current, 0);
> +
> if (ka->sa.sa_handler == SIG_IGN) /* Do nothing. */
> continue;
> if (ka->sa.sa_handler != SIG_DFL) {