[Devel] Re: [PATCH 1/3] Signal semantics for /sbin/init

sukadev at us.ibm.com sukadev at us.ibm.com
Mon Sep 3 08:56:09 PDT 2007


Oleg Nesterov [oleg at tv-sign.ru] wrote:
| On 08/31, sukadev at us.ibm.com 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().

Sorry. I wasn't paying enough attenttion to this patch and including it
only for reference. Was planning to replace this with your final patch.
Or do you want me to fix the two bugs and resend ?
| 
| Oleg.
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list