Re: [Lldb-commits] [PATCH] D13555: Cap test threads on Windows to avoid open file limit

2015-10-12 Thread Adrian McCarthy via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL250044: Switch threading mode for tests on Windows when there are lots of cores. (authored by amccarth). Changed prior to commit: http://reviews.llvm.org/D13555?vs=36993&id=37111#toc Repository: rL L

Re: [Lldb-commits] [PATCH] D13555: Cap test threads on Windows to avoid open file limit

2015-10-10 Thread Todd Fiala via lldb-commits
tfiala added a comment. LGTM, Adrian! http://reviews.llvm.org/D13555 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D13555: Cap test threads on Windows to avoid open file limit

2015-10-09 Thread Adrian McCarthy via lldb-commits
amccarth updated this revision to Diff 36993. amccarth added a comment. Using the multiprocessing-pool mode seems more reliable than capping num_threads. I'm doing it only when the number of threads is high because of the loss of Ctrl+C support. http://reviews.llvm.org/D13555 Files: test/d

Re: [Lldb-commits] [PATCH] D13555: Cap test threads on Windows to avoid open file limit

2015-10-08 Thread Adrian McCarthy via lldb-commits
amccarth added a comment. Hold off. I just got a failure even at only 32 subprocesses. http://reviews.llvm.org/D13555 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D13555: Cap test threads on Windows to avoid open file limit

2015-10-08 Thread Adrian McCarthy via lldb-commits
amccarth created this revision. amccarth added a reviewer: tfiala. amccarth added a subscriber: lldb-commits. Python uses stdio from the C runtime for file descriptors and pipes. On Windows, by default, the CRT has a limit of 512 open file descriptors. https://msdn.microsoft.com/en-us/library/