On 08/31, sukadev@xxxxxxxxxx wrote: > > -static int sig_ignored(struct task_struct *t, int sig) > + // Currently this check is a bit racy with exec(), > + // we can _simplify_ de_thread and close the race. > + if (likely(!is_container_init(tsk->group_leader))) > + return 0; > + > + if (!in_interrupt()) > + return 0; I don't understand why you are trying to mix this patch with pid_ns changes. We don't need in_interrupt() check unless we use current do decide if the signal goes from the parent namespace. And in fact, I'd personally prefer to use "is_global_init()" for this patch, because it hopefully can fix the problems we have even without namespaces. This also matches the current check in get_signal_to_deliver(). Oleg. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers
