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

[Simon Marchi <simon.marchi@polymtl.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