Run more than 34 processes on Win32 if we have 32+ cores.
authorRui Ueyama <ruiu@google.com>
Fri, 17 Apr 2015 21:14:44 +0000 (14:14 -0700)
committerRui Ueyama <ruiu@google.com>
Fri, 17 Apr 2015 21:14:48 +0000 (14:14 -0700)
For compatiblity reason, dwNumberOfProcessors in Win32 is capped at 32.
So even if your machine has more than 32 cores, Ninja spawns at most 34
subprocesses. This patch fixes the issue by using GetNativeSystemInfo,
which returns the system info from Wow64 point of view, instead of
GetSystemInfo.

src/util.cc

index 720ccbd..aa47f2f 100644 (file)
@@ -500,7 +500,7 @@ string StripAnsiEscapeCodes(const string& in) {
 int GetProcessorCount() {
 #ifdef _WIN32
   SYSTEM_INFO info;
-  GetSystemInfo(&info);
+  GetNativeSystemInfo(&info);
   return info.dwNumberOfProcessors;
 #else
   return sysconf(_SC_NPROCESSORS_ONLN);