<div dir="ltr">Hi Pavel,<div><br></div><div>I think this issue came back again.</div><div><br></div><div>I tried on Nexus Player and Nexus 9 and `process kill` gives the error: </div><div>```</div><div>Handle_k: failed to kill spawned pid <pid>, ignoring.<br></div><div>```</div><div>from lldb-server.</div><div><br></div><div>I think it might be a good idea if we archived stdout/stderr from lldb-server too. Hopefully it'll be empty, but if we get errors like these, we'll have a log.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 16, 2015 at 6:04 AM, Pavel Labath <span dir="ltr"><<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've tracked the problem down. The issue was that after this patch,<br>
some things were happening in a slightly different order, and we were<br>
deadlocking while trying to shut down the stdio forwarding thread. I<br>
did not detect this when running locally, because the read thread is<br>
not used there. I have reverted this for now. I will first get rid of<br>
the read thread and then resubmit this.<br>
<span class=""><br>
On 16 July 2015 at 07:20, Chaoren Lin <<a href="mailto:chaorenl@google.com">chaorenl@google.com</a>> wrote:<br>
> I wouldn't be surprised if this were Android specific (libc/kernel bug).<br>
> Since we did run into problems with pthread_sigmask before.<br>
><br>
</span><span class="">> On Wed, Jul 15, 2015 at 11:17 PM, Chaoren Lin <<a href="mailto:chaorenl@google.com">chaorenl@google.com</a>> wrote:<br>
>><br>
>> Yeah, it's definitely related. Those two tests launch a large number of<br>
>> processes. LLDB kinda hangs waiting for the inferior exit SIGCHLD. Multiply<br>
>> that by the number of processes and you have hanging tests.<br>
>><br>
</span>>> On Jul 15, 2015 23:14, "Oleksiy Vyalov" <<a href="mailto:ovyalov@google.com">ovyalov@google.com</a>> wrote:<br>
>>><br>
>>> Starting with this build  which includes the CL next 2 tests started to<br>
<span class="">>>> timeout - TestIntegerTypes & TestIntegerTypesExpr.<br>
>>> Could you check whether it's related or not?<br>
>>><br>
</span><span class="">>>> On Wed, Jul 15, 2015 at 7:06 PM, Chaoren Lin <<a href="mailto:chaorenl@google.com">chaorenl@google.com</a>> wrote:<br>
>>>><br>
>>>> Hi Pavel,<br>
>>>><br>
>>>> I think there's some problems with the signal handling. SIGCHLD is<br>
>>>> received 6 times, but SigchldHandler is only called 5 times (missing the<br>
>>>> exit signal).<br>
>>>><br>
>>>> The following error occurs when trying to `process kill` an Android<br>
>>>> inferior:<br>
>>>> ```<br>
>>>> Handle_k: failed to kill spawned pid ###, ignoring.<br>
>>>> ```<br>
>>>><br>
>>>> It doesn't seem to be directly causing any test failures, but it's<br>
>>>> significantly slowing down the test suite.<br>
>>>><br>
>>>> Could you please look into this?<br>
>>>><br>
</span>>>>> On Wed, Jul 15, 2015 at 10:20 AM, Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>><br>
<span class="">>>>> wrote:<br>
>>>>><br>
>>>>> This revision was automatically updated to reflect the committed<br>
>>>>> changes.<br>
>>>>> Closed by commit rL242305: [NativeProcessLinux] Integrate MainLoop<br>
>>>>> (authored by labath).<br>
>>>>><br>
>>>>> Changed prior to commit:<br>
>>>>>   <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11150-3Fvs-3D29660-26id-3D29789-23toc&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=WeA97KhaKJdoMlUmVrsznlk8i8q8AhtbsdT0iSFIzW8&s=lVcTNf6GShQaZ3AZkEoZ0oR2oCXIH1Z-X9qnwJ26SEk&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11150?vs=29660&id=29789#toc</a><br>
>>>>><br>
</span><span class="">>>>>> Repository:<br>
>>>>>   rL LLVM<br>
>>>>><br>
>>>>> <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11150&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=WeA97KhaKJdoMlUmVrsznlk8i8q8AhtbsdT0iSFIzW8&s=GUT8Kz6-eJy9wX2lNpvlCiltV10O50Sdi9fSwKPT6wE&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11150</a><br>
>>>>><br>
</span><span class="">>>>>> Files:<br>
>>>>>   lldb/trunk/include/lldb/Host/common/NativeProcessProtocol.h<br>
>>>>>   lldb/trunk/source/Host/common/NativeProcessProtocol.cpp<br>
>>>>>   lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp<br>
>>>>>   lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.h<br>
>>>>><br>
>>>>> lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux.cpp<br>
>>>>><br>
>>>>> lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm64.cpp<br>
>>>>><br>
>>>>> lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_mips64.cpp<br>
>>>>><br>
>>>>> lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp<br>
>>>>><br>
>>>>> lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.h<br>
>>>>><br>
>>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
</span>>>> Oleksiy Vyalov | Software Engineer | <a href="mailto:ovyalov@google.com">ovyalov@google.com</a><br>
><br>
><br>
</blockquote></div><br></div>