Adds an helper class to shard mspdbsrv.

pylib/gyp/win_tool.py

 #!/usr/bin/env python
 
 # Copyright (c) 2012 Google Inc. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Utility functions for Windows builds.
 
 These functions are executed via gyp-win-tool when using the ninja generator.
 """
 
 import os
+import re
 import shutil
 import subprocess
 import sys
 
 BASE_DIR = os.path.dirname(os.path.abspath(__file__))
 
+# A regex matching an argument corresponding to a PDB filename passed as an
+# argument to link.exe.
+_LINK_EXE_PDB_ARG = re.compile('/PDB:(?P<pdb>.+\.exe\.pdb)', re.IGNORECASE)
+_MSPDBSRV_ENDPOINT_ENV_VAR = '_MSPDBSRV_ENDPOINT_'
 
 def main(args):
   executor = WinTool()
   exit_code = executor.Dispatch(args)
   if exit_code is not None:
     sys.exit(exit_code)
 
 
 class WinTool(object):
   """This class performs all the Windows tooling steps. The methods can either
   be executed directly, or dispatched from an argument list."""
 
+  class MsPdbSrvHelper(object):
+    """This is an helper class to use a unique instance of mspdbsrv.exe for the
+    linkers linking an .exe target. This helps to reduce the memory pressure one
+    the shared instance of mspdbsrv.exe."""
+
+    def __init__(self, env, args):
+      self.env = env
+      self.args = args
+      self._MaybeStartMsPdbSrv()
+
+    def _MaybeStartMsPdbSrv(self):
+      """Starts an unique instance of mspdbsrv.exe if the given arguments are
+      those of a linker linking a .exe target."""
+      if not os.environ.get('GYP_USE_SEPARATE_MSPDBSRV'):
+        return
+
+      if len(self.args) < 1:
+        raise Exception("Not enough arguments")
+
+      if self.args[0] != 'link.exe':
+        return
+
+      # Checks if this linker produces a PDB for an .exe target. If so use the
+      # name of this PDB to generate an endpoint name for mspdbsrv.exe.
+      endpoint_name = None
+      for arg in self.args:
+        m = _LINK_EXE_PDB_ARG.match(arg)
+        if m:
+          endpoint_name = '%s_%d' % (m.group('pdb'), os.getpid())
+          break
+
+      if endpoint_name is None:
+        return
+
+      # Adds the appropriate environment variable. This will be read by link.exe
+      # to know which instance of mspdbsrv.exe it should connect to (if it's
+      # not set then the default endpoint is used).
+      self.env[_MSPDBSRV_ENDPOINT_ENV_VAR] = endpoint_name
+
   def Dispatch(self, args):
     """Dispatches a string command to a method."""
     if len(args) < 1:
       raise Exception("Not enough arguments")
 
     method = "Exec%s" % self._CommandifyName(args[0])
     return getattr(self, method)(*args[1:])
 
   def _CommandifyName(self, name_string):
     """Transforms a tool name like recursive-mirror to RecursiveMirror."""
@@ skipping 23 matching lines @@
       shutil.copytree(source, dest)
     else:
       shutil.copy2(source, dest)
 
   def ExecLinkWrapper(self, arch, *args):
     """Filter diagnostic output from link that looks like:
     '   Creating library ui.dll.lib and object ui.dll.exp'
     This happens when there are exports from the dll or exe.
     """
     env = self._GetEnv(arch)
-    popen = subprocess.Popen(args, shell=True, env=env,
-                              stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
-    out, _ = popen.communicate()
+    mspdbsrv_helper = WinTool.MsPdbSrvHelper(env, args)

[scottmg, 2013/11/27 20:33:41]

this can just be a function now (and it doesn't actually "Start" anything :)

[Sébastien Marchand, 2013/11/27 20:53:43]

Of course ! It's much simpler now that we know how this work :)

+    link_wrapper = subprocess.Popen(args,
+                                    shell=True,
+                                    env=env,
+                                    stdout=subprocess.PIPE,
+                                    stderr=subprocess.STDOUT)
+    out, _ = link_wrapper.communicate()
     for line in out.splitlines():
       if not line.startswith('   Creating library '):
         print line
-    return popen.returncode
+    return link_wrapper.returncode
 
   def ExecManifestWrapper(self, arch, *args):
     """Run manifest tool with environment set. Strip out undesirable warning
     (some XML blocks are recognized by the OS loader, but not the manifest
     tool)."""
     env = self._GetEnv(arch)
     popen = subprocess.Popen(args, shell=True, env=env,
                              stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
     out, _ = popen.communicate()
     for line in out.splitlines():
@@ skipping 70 matching lines @@
           line):
         print line
     return popen.returncode
 
   def ExecActionWrapper(self, arch, rspfile, *dir):
     """Runs an action command line from a response file using the environment
     for |arch|. If |dir| is supplied, use that as the working directory."""
     env = self._GetEnv(arch)
     args = open(rspfile).read()
     dir = dir[0] if dir else None
-    popen = subprocess.Popen(args, shell=True, env=env, cwd=dir)
-    popen.wait()
-    return popen.returncode
+    return subprocess.call(args, shell=True, env=env, cwd=dir)
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv[1:]))