Re: [PATCH v2] PR threads/20743: Don't attempt to suspend or resume exited threads.

[Simon Marchi <simon dot marchi at polymtl dot ca>]

On 2017-04-17 14:27, John Baldwin wrote:
> On Friday, April 14, 2017 08:01:25 PM Luis Machado wrote:
> > On 04/14/2017 05:40 PM, John Baldwin wrote:
> > >> Otherwise i have no further comments. I assume you ran gdb's testsuite
> > >> against this change and verified the results are sane?
> > >
> > > There were no regressions at least.  With the stock tree there are
> > > several unexpected failures already which I will get to at some point.
> > >
> >
> > Great. I have no further comments.
>
> To be clear (since I believe I messed this up before), I still need 
> approval
> from an approver?

I read both the v1 and v2 threads to get the context, the patch looks 
good to me.  Pedro was fine with the ALL_NON_EXITED_THREADS approach as 
well, so go ahead and push.  I think it should go in the 8.0 branch as 
well.

Btw, do you know why the double parenthesis in

  perror_with_name (("ptrace"));

?  Feel free to remove the extra parenthesis before pushing.

Thanks,

Simon