<html>
<head>
<base href="https://llvm.org/bugs/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:todd.fiala@gmail.com" title="Todd Fiala <todd.fiala@gmail.com>"> <span class="fn">Todd Fiala</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - multiple-debuggers test: testprog is left running when test times out"
href="https://llvm.org/bugs/show_bug.cgi?id=25063">bug 25063</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>ASSIGNED
</td>
<td>RESOLVED
</td>
</tr>
<tr>
<td style="text-align:right;">Resolution</td>
<td>---
</td>
<td>FIXED
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - multiple-debuggers test: testprog is left running when test times out"
href="https://llvm.org/bugs/show_bug.cgi?id=25063#c2">Comment # 2</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - multiple-debuggers test: testprog is left running when test times out"
href="https://llvm.org/bugs/show_bug.cgi?id=25063">bug 25063</a>
from <span class="vcard"><a class="email" href="mailto:todd.fiala@gmail.com" title="Todd Fiala <todd.fiala@gmail.com>"> <span class="fn">Todd Fiala</span></a>
</span></b>
<pre>On the OS X side, the issue is that we start the processes suspended, which
puts them in a state that the kernel will absolutely refuse to service signals
for --- it is really not running yet.
I thought about adding a clean-up step in the test runner that scans for
process group members that are not yet dead at test exit, and if a hard kill
doesn't take care of them, then use the in-test-runner-process lldb module to
attach and then kill, since this is all that is really needed. But rather than
add that complication, I went with taking care of the core issue, which is
doing too much all at once in a parallel test runner environment. I could have
tried to calculate the number based on # cores, but that's not really good
enough. So for now I cut it from 50 to 20 after talking to Jason Molenda about
it. This is plenty to stress what was desired to be stressed at the time the
test was written.
Change to # concurrent debuggers made here:
svn commit
Sending test/api/multiple-debuggers/multi-process-driver.cpp
Transmitting file data .
Committed revision 249421.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>