[llvm] r256291 - lit: Limit number of processes on Windows to 32.

[Sean Silva via llvm-commits]

Hi Nico,

Can you give more backstory for this change? Was it just a single machine
failing?

Takumi, do you know any more details about this? Do you override it on your
bots?

-- Sean Silva



On Tue, Dec 22, 2015 at 3:12 PM, Nico Weber via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: nico
> Date: Tue Dec 22 17:12:00 2015
> New Revision: 256291
>
> URL: http://llvm.org/viewvc/llvm-project?rev=256291&view=rev
> Log:
> lit: Limit number of processes on Windows to 32.
>
> Modified:
>     llvm/trunk/utils/lit/lit/util.py
>
> Modified: llvm/trunk/utils/lit/lit/util.py
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/util.py?rev=256291&r1=256290&r2=256291&view=diff
>
> ==============================================================================
> --- llvm/trunk/utils/lit/lit/util.py (original)
> +++ llvm/trunk/utils/lit/lit/util.py Tue Dec 22 17:12:00 2015
> @@ -39,7 +39,9 @@ def detectCPUs():
>      if "NUMBER_OF_PROCESSORS" in os.environ:
>          ncpus = int(os.environ["NUMBER_OF_PROCESSORS"])
>          if ncpus > 0:
> -            return ncpus
> +            # With more than 32 processes, process creation often fails
> with
> +            # "Too many open files".  FIXME: Check if there's a better
> fix.
> +            return min(ncpus, 32)
>      return 1 # Default
>
>  def mkdir_p(path):
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160219/36896675/attachment.html>