Re: strace lockup when tracing exec in go

2016-09-23 Thread Michal Hocko
On Fri 23-09-16 12:21:41, Oleg Nesterov wrote: > On 09/22, Michal Hocko wrote: > > > > --- a/kernel/signal.c > > +++ b/kernel/signal.c > > @@ -91,6 +91,10 @@ static int sig_ignored(struct task_struct *t, int sig, > > bool force) > > if (!sig_task_ignored(t, sig, force)) > > return

Re: strace lockup when tracing exec in go

2016-09-23 Thread Oleg Nesterov
On 09/22, Michal Hocko wrote: > > --- a/kernel/signal.c > +++ b/kernel/signal.c > @@ -91,6 +91,10 @@ static int sig_ignored(struct task_struct *t, int sig, > bool force) > if (!sig_task_ignored(t, sig, force)) > return 0; > > + /* Do not ignore signals sent from child to

Re: strace lockup when tracing exec in go

2016-09-23 Thread Oleg Nesterov
Sorry for delay, I was offline. I'll try to return to this problem next week, currently I can't even read this thread but at first glance the proposed patch(es) do not look right... On 09/21, Michal Hocko wrote: > > The further investigation shown that the tracer (strace) is stuck > waiting for

Re: strace lockup when tracing exec in go

2016-09-23 Thread Michal Hocko
On Fri 23-09-16 11:50:32, Oleg Nesterov wrote: > Sorry for delay, I was offline. I'll try to return to this problem next > week, currently I can't even read this thread but at first glance the > proposed patch(es) do not look right... > > On 09/21, Michal Hocko wrote: > > > > The further

Re: strace lockup when tracing exec in go

2016-09-23 Thread Oleg Nesterov
On 09/23, Michal Hocko wrote: > > On Fri 23-09-16 12:21:41, Oleg Nesterov wrote: > > On 09/22, Michal Hocko wrote: > > > > > > --- a/kernel/signal.c > > > +++ b/kernel/signal.c > > > @@ -91,6 +91,10 @@ static int sig_ignored(struct task_struct *t, int sig, > > > bool force) > > > if

Re: strace lockup when tracing exec in go

2016-09-23 Thread Michal Hocko
On Fri 23-09-16 15:21:02, Oleg Nesterov wrote: > On 09/23, Michal Hocko wrote: > > > > On Fri 23-09-16 12:21:41, Oleg Nesterov wrote: > > > On 09/22, Michal Hocko wrote: > > > > > > > > --- a/kernel/signal.c > > > > +++ b/kernel/signal.c > > > > @@ -91,6 +91,10 @@ static int sig_ignored(struct

Re: strace lockup when tracing exec in go

2016-09-23 Thread Oleg Nesterov
On 09/23, Michal Hocko wrote: > > On Fri 23-09-16 15:21:02, Oleg Nesterov wrote: > > > > This change is simply wrong no matter what. > > I've just tried to extend the existing > > /* >* Tracers may want to know about even ignored signals. >*/ > return !t->ptrace; > >

Re: strace lockup when tracing exec in go

2016-09-23 Thread Michal Hocko
On Fri 23-09-16 16:07:24, Oleg Nesterov wrote: > On 09/23, Michal Hocko wrote: > > > > On Fri 23-09-16 15:21:02, Oleg Nesterov wrote: > > > > > > This change is simply wrong no matter what. > > > > I've just tried to extend the existing > > > > /* > > * Tracers may want to know about even