<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Jan 9, 2016 at 1:26 AM, Dan Liew <span dir="ltr"><<a href="mailto:dan@su-root.co.uk" target="_blank">dan@su-root.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi JF,<br>
<span class=""><br>
On 9 January 2016 at 01:16, JF Bastien <<a href="mailto:jfb@chromium.org">jfb@chromium.org</a>> wrote:<br>
> Hi Dan,<br>
><br>
> I don't know if I'm the only one getting this, but it seems like your patch<br>
> causes timeouts for me and for some bots I'm testing with (they're all<br>
> timing out after your commit).<br>
<br>
</span>What is your timeout for your bots? If your system has the Python<br>
psutil module installed the lit timeout tests will run which are a<br>
little slow but **should not** hang indefinitely.<br></blockquote><div><br></div><div>The current timeout is 20 minutes per step, and the entire bots used to complete in less than 10 minutes (they do more than build and test LLVM).</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
```<br>
$ time ninja check-lit<br>
[2/2] Running lit's tests<br>
lit.py: lit.cfg:58: note: Found python psutil module<br>
Testing Time: 16.90s<br>
  Expected Passes    : 14<br>
<br>
real    0m17.096s<br>
user    0m13.610s<br>
sys     0m0.577s<br>
```<br>
<span class=""><br>
<br>
<br>
> I used the following to repro locally:<br>
><br>
> cmake ../llvm -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DLLVM_BUILD_TESTS=ON<br>
> -DCMAKE_BUILD_TYPE=RelWithDebInfo -DLLVM_ENABLE_ASSERTIONS=ON<br>
> -DLLVM_TARGETS_TO_BUILD="X86" && make check-all -j$(nproc)<br>
><br>
> Things are stuck at "running all regression tests".<br>
><br>
> I have the python processes at the bottom of this email hung (all disappear<br>
> when I kill make). I'm pretty suspicious of the infinite_loop.py test<br>
> considering things seem to loop forever :-)<br>
<br>
</span>That test is deliberately running a program that loops forever to test<br>
that the ``--timeout=`` feature of lit works. If it hangs it likely<br>
means there is a bug with that feature.<br>
<span class=""><br>
> The LLVM waterfall at <a href="http://lab.llvm.org:8011/console" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/console</a> seems fine, so maybe<br>
> it's a Python version difference? The bots I'm using are pretty simple x86<br>
> machines, whereas my machine is a beefy one, so I don't think it's a problem<br>
> with too-slow or too-big machine.<br>
<br>
</span>When I wrote the ``--timeout=`` tests I tested Python 2.7 and Python<br>
3.4 with the version 3.2 of the Python psutil module<br>
<br>
Could you tell me<br>
<br>
- What OS you are using (e.g. Ubuntu 14.04LTS)<br></blockquote><div><br></div><div>Ubuntu 14.04 LTS on 3.13.0-67-generic.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
- What Python version you have<br></blockquote><div><br></div><div>2.7.6<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
- What version of the Python psutil module you have<br></blockquote><div><br></div><div>1.2.1-1ubuntu2<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
With that I can have a go at reproducing your issue.<br>
<span class=""><br>
> Would you have an idea of what's going on? I haven't dug into the code yet<br>
> since I figure you may already have a hunch.<br>
<br>
</span>My hunch is that if you have an old version of psutil installed it<br>
might be behaving badly and failing to kill the processes (see<br>
``killProcessAndChildren()`` in ``utils/lit/lit/util.py``).<br>
If that is the case this might accelerate my plans to remove the<br>
dependency on the psutil module.<br>
<br>
Note I'm seeing Windows buildbot failures (not related to lit timeout feature)<br>
<br>
<a href="http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/8760" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/8760</a><br>
<a href="http://lab.llvm.org:8011/builders/clang-x86-win2008-selfhost/builds/5613" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-x86-win2008-selfhost/builds/5613</a><br>
<br>
so I need to revert my commit until I can work out why the tests are<br>
broken under Windows (eurgh...)<br>
<br>
Unfortunately it seems svn is down right now so I can't do anything<br>
until it's back up.<br>
<span class=""><font color="#888888"><br>
Dan.<br>
</font></span></blockquote></div><br></div></div>