public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Liming" <liming.gao@intel.com>
To: "Carsey, Jaben" <jaben.carsey@intel.com>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
	"Guo, Dongao" <dongao.guo@intel.com>
Subject: Re: [Patch] BaseTools: Support multi thread build Basetool on Windows
Date: Wed, 29 Aug 2018 14:47:33 +0000	[thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E2EACA8@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <DD2D08E1-463F-4E82-9B82-559BE525B828@intel.com>

Jaben:
  NmakeSubdirs.bat will be removed once NmakeSubdirs.py is stable. Now, bat is kept for a while.

Thanks
Liming
> -----Original Message-----
> From: Carsey, Jaben
> Sent: Wednesday, August 29, 2018 12:10 AM
> To: Gao, Liming <liming.gao@intel.com>
> Cc: edk2-devel@lists.01.org; Guo, Dongao <dongao.guo@intel.com>
> Subject: Re: [edk2] [Patch] BaseTools: Support multi thread build Basetool on Windows
> 
> This looks like a great change.  Why not delete the bat file?
> 
> Reviewed-by Jaben Carsey: <Jaben.casprsey@intel.com>
> 
> Jaben
> 
> > On Aug 28, 2018, at 8:28 AM, Liming Gao <liming.gao@intel.com> wrote:
> >
> > From: Dongao Guo <dongao.guo@intel.com>
> >
> > Add NmakeSubdirs.py to replace NmakeSubdirs.bat in VS Makefile. This script will
> > invoke nmake in multi thread mode. It can save more than half time of BaseTools
> > C clean build.
> > GCC make supports multiple thread in make phase. So, GNNmakefile doesn't need apply
> > this script.
> >
> > single task or job=1:
> >    just single thread and invoke subprocess,subprocess will use
> >    system.stdout to print output.
> > multi task:
> >    thread number is logic cpu count.All subprocess output will pass to
> >    python script by PIPE and then script print it to system.stdout.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Dongao Guo<dongao.guo@intel.com>
> > Reviewed-by: Liming Gao <liming.gao@intel.com>
> > Test-by: Liming Gao <liming.gao@intel.com>
> > ---
> > BaseTools/Makefile                           |  12 +-
> > BaseTools/Source/C/Makefile                  |  14 +--
> > BaseTools/Source/C/Makefiles/NmakeSubdirs.py | 169 +++++++++++++++++++++++++++
> > 3 files changed, 182 insertions(+), 13 deletions(-)
> > create mode 100644 BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> >
> > diff --git a/BaseTools/Makefile b/BaseTools/Makefile
> > index 3736d85..b98cd85 100644
> > --- a/BaseTools/Makefile
> > +++ b/BaseTools/Makefile
> > @@ -1,7 +1,7 @@
> > ## @file
> > # Windows makefile for Base Tools project build.
> > #
> > -# Copyright (c) 2007 - 2017, Intel Corporation. All rights reserved.<BR>
> > +# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
> > # This program and the accompanying materials
> > # are licensed and made available under the terms and conditions of the BSD License
> > # which accompanies this distribution.  The full text of the license may be found at
> > @@ -20,19 +20,19 @@ SUBDIRS = $(BASE_TOOLS_PATH)\Source\C $(BASE_TOOLS_PATH)\Source\Python
> > all: c python
> >
> > c :
> > -  @$(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.bat all $(BASE_TOOLS_PATH)\Source\C
> > +  @$(PYTHON_HOME)\python.exe $(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.py  all
> $(BASE_TOOLS_PATH)\Source\C
> >
> > python:
> > -  @$(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.bat all $(BASE_TOOLS_PATH)\Source\Python
> > +  @$(PYTHON_HOME)\python.exe $(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.py  all
> $(BASE_TOOLS_PATH)\Source\Python
> >
> > subdirs: $(SUBDIRS)
> > -  @$(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.bat all $**
> > +  @$(PYTHON_HOME)\python.exe $(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.py  all $**
> >
> > .PHONY: clean
> > clean:
> > -  @$(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.bat clean $(SUBDIRS)
> > +  $(PYTHON_HOME)\python.exe $(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.py clean $(SUBDIRS)
> >
> > .PHONY: cleanall
> > cleanall:
> > -  @$(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.bat cleanall $(SUBDIRS)
> > +  $(PYTHON_HOME)\python.exe $(BASE_TOOLS_PATH)\Source\C\Makefiles\NmakeSubdirs.py  cleanall $(SUBDIRS)
> >
> > diff --git a/BaseTools/Source/C/Makefile b/BaseTools/Source/C/Makefile
> > index 5428180..1246d23 100644
> > --- a/BaseTools/Source/C/Makefile
> > +++ b/BaseTools/Source/C/Makefile
> > @@ -1,7 +1,7 @@
> > ## @file
> > # Windows makefile for C tools build.
> > #
> > -# Copyright (c) 2009 - 2017, Intel Corporation. All rights reserved.<BR>
> > +# Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.<BR>
> > # This program and the accompanying materials
> > # are licensed and made available under the terms and conditions of the BSD License
> > # which accompanies this distribution.  The full text of the license may be found at
> > @@ -16,7 +16,7 @@ HOST_ARCH = IA32
> >
> > LIBRARIES = Common
> > APPLICATIONS = \
> > -  BootSectImage \
> > +  VfrCompile \
> >   BrotliCompress \
> >   EfiLdrImage \
> >   EfiRom \
> > @@ -32,7 +32,7 @@ APPLICATIONS = \
> >   Split \
> >   TianoCompress \
> >   VolInfo \
> > -  VfrCompile \
> > +  BootSectImage \
> >   DevicePath
> >
> > all: libs apps install
> > @@ -43,7 +43,7 @@ libs: $(LIBRARIES)
> >    @echo # Build libraries
> >    @echo ######################
> >    @if not exist $(LIB_PATH) mkdir $(LIB_PATH)
> > -    @Makefiles\NmakeSubdirs.bat all $**
> > +    @$(PYTHON_HOME)\python.exe Makefiles\NmakeSubdirs.py all $**
> >
> > apps: $(APPLICATIONS)
> >    @echo.
> > @@ -51,7 +51,7 @@ apps: $(APPLICATIONS)
> >    @echo # Build executables
> >    @echo ######################
> >    @if not exist $(BIN_PATH) mkdir $(BIN_PATH)
> > -    @Makefiles\NmakeSubdirs.bat all $**
> > +    @$(PYTHON_HOME)\python.exe Makefiles\NmakeSubdirs.py all $**
> >
> > install: $(LIB_PATH) $(BIN_PATH)
> >    @echo.
> > @@ -65,11 +65,11 @@ install: $(LIB_PATH) $(BIN_PATH)
> >
> > .PHONY: clean
> > clean:
> > -  @Makefiles\NmakeSubdirs.bat clean $(LIBRARIES) $(APPLICATIONS)
> > +  @$(PYTHON_HOME)\python.exe Makefiles\NmakeSubdirs.py clean $(LIBRARIES) $(APPLICATIONS)
> >
> > .PHONY: cleanall
> > cleanall:
> > -  @Makefiles\NmakeSubdirs.bat cleanall $(LIBRARIES) $(APPLICATIONS)
> > +  @$(PYTHON_HOME)\python.exe Makefiles\NmakeSubdirs.py cleanall $(LIBRARIES) $(APPLICATIONS)
> >
> > !INCLUDE Makefiles\ms.rule
> >
> > diff --git a/BaseTools/Source/C/Makefiles/NmakeSubdirs.py b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> > new file mode 100644
> > index 0000000..29bb5df
> > --- /dev/null
> > +++ b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> > @@ -0,0 +1,169 @@
> > +# @file NmakeSubdirs.py
> > +# This script support parallel build for nmake in windows environment.
> > +# It supports Python2.x and Python3.x both.
> > +#
> > +#  Copyright (c) 2018, Intel Corporation. All rights reserved.<BR>
> > +#
> > +#  This program and the accompanying materials
> > +#  are licensed and made available under the terms and conditions of the BSD License
> > +#  which accompanies this distribution.  The full text of the license may be found at
> > +#  http://opensource.org/licenses/bsd-license.php
> > +#
> > +#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> > +#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> > +#
> > +
> > +#
> > +# Import Modules
> > +#
> > +
> > +from __future__ import print_function
> > +import argparse
> > +import threading
> > +import time
> > +import os
> > +import subprocess
> > +import multiprocessing
> > +import copy
> > +import sys
> > +__prog__        = 'NmakeSubdirs'
> > +__version__     = '%s Version %s' % (__prog__, '0.10 ')
> > +__copyright__   = 'Copyright (c) 2018, Intel Corporation. All rights reserved.'
> > +__description__ = 'Replace for NmakeSubdirs.bat in windows ,support parallel build for nmake.\n'
> > +
> > +cpu_count = multiprocessing.cpu_count()
> > +output_lock = threading.Lock()
> > +def RunCommand(WorkDir=None, *Args, **kwargs):
> > +    if WorkDir is None:
> > +        WorkDir = os.curdir
> > +    if "stderr" not in kwargs:
> > +        kwargs["stderr"] = subprocess.STDOUT
> > +    if "stdout" not in kwargs:
> > +        kwargs["stdout"] = subprocess.PIPE
> > +    p = subprocess.Popen(Args, cwd=WorkDir, stderr=kwargs["stderr"], stdout=kwargs["stdout"])
> > +    stdout, stderr = p.communicate()
> > +    message = ""
> > +    if stdout is not None:
> > +        message = stdout.decode() #for compatibility in python 2 and 3
> > +
> > +    if p.returncode != 0:
> > +        raise RuntimeError("Error while execute command \'{0}\' in direcotry {1}\n{2}".format(" ".join(Args), WorkDir, message))
> > +
> > +    output_lock.acquire(True)
> > +    print("execute command \"{0}\" in directory {1}".format(" ".join(Args), WorkDir))
> > +    print(message)
> > +    output_lock.release()
> > +
> > +    return p.returncode, stdout
> > +
> > +class TaskUnit(object):
> > +    def __init__(self, func, args, kwargs):
> > +        self.func = func
> > +        self.args = args
> > +        self.kwargs = kwargs
> > +
> > +    def __eq__(self, other):
> > +        return id(self).__eq__(id(other))
> > +
> > +    def run(self):
> > +        return self.func(*self.args, **self.kwargs)
> > +
> > +    def __str__(self):
> > +        para = list(self.args)
> > +        para.extend("{0}={1}".format(k, v)for k, v in self.kwargs.items())
> > +
> > +        return "{0}({1})".format(self.func.__name__, ",".join(para))
> > +
> > +class ThreadControl(object):
> > +
> > +    def __init__(self, maxthread):
> > +        self._processNum = maxthread
> > +        self.pending = []
> > +        self.running = []
> > +        self.pendingLock = threading.Lock()
> > +        self.runningLock = threading.Lock()
> > +        self.error = False
> > +        self.errorLock = threading.Lock()
> > +        self.errorMsg = "errorMsg"
> > +
> > +    def addTask(self, func, *args, **kwargs):
> > +        self.pending.append(TaskUnit(func, args, kwargs))
> > +
> > +    def waitComplete(self):
> > +        self._schedule.join()
> > +
> > +    def startSchedule(self):
> > +        self._schedule = threading.Thread(target=self.Schedule)
> > +        self._schedule.start()
> > +
> > +    def Schedule(self):
> > +        for i in range(self._processNum):
> > +            task = threading.Thread(target=self.startTask)
> > +            task.daemon = False
> > +            self.running.append(task)
> > +
> > +        self.runningLock.acquire(True)
> > +        for thread in self.running:
> > +            thread.start()
> > +        self.runningLock.release()
> > +
> > +        while len(self.running) > 0:
> > +            time.sleep(0.1)
> > +        if self.error:
> > +            print("subprocess not exit sucessfully")
> > +            print(self.errorMsg)
> > +
> > +    def startTask(self):
> > +        while True:
> > +            if self.error:
> > +                break
> > +            self.pendingLock.acquire(True)
> > +            if len(self.pending) == 0:
> > +                self.pendingLock.release()
> > +                break
> > +            task = self.pending.pop(0)
> > +            self.pendingLock.release()
> > +            try:
> > +                task.run()
> > +            except RuntimeError as e:
> > +                if self.error: break
> > +                self.errorLock.acquire(True)
> > +                self.error = True
> > +                self.errorMsg = str(e)
> > +                time.sleep(0.1)
> > +                self.errorLock.release()
> > +                break
> > +
> > +        self.runningLock.acquire(True)
> > +        self.running.remove(threading.currentThread())
> > +        self.runningLock.release()
> > +
> > +def Run():
> > +    curdir = os.path.abspath(os.curdir)
> > +    if len(args.subdirs) == 1:
> > +        args.jobs = 1
> > +    if args.jobs == 1:
> > +        try:
> > +            for dir in args.subdirs:
> > +                RunCommand(os.path.join(curdir, dir), "nmake", args.target, stdout=sys.stdout, stderr=subprocess.STDOUT)
> > +        except RuntimeError:
> > +            exit(1)
> > +    else:
> > +        controller = ThreadControl(args.jobs)
> > +        for dir in args.subdirs:
> > +            controller.addTask(RunCommand, os.path.join(curdir, dir), "nmake", args.target)
> > +        controller.startSchedule()
> > +        controller.waitComplete()
> > +        if controller.error:
> > +            exit(1)
> > +
> > +if __name__ == "__main__":
> > +    parser = argparse.ArgumentParser(prog=__prog__, description=__description__ + __copyright__, conflict_handler='resolve')
> > +
> > +    parser.add_argument("target", help="the target for nmake")
> > +    parser.add_argument("subdirs", nargs="+", help="the relative dir path of makefile")
> > +    parser.add_argument("--jobs", type=int, dest="jobs", default=cpu_count, help="thread number")
> > +    parser.add_argument('--version', action='version', version=__version__)
> > +    args = parser.parse_args()
> > +    Run()
> > +
> > --
> > 2.10.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel


      reply	other threads:[~2018-08-29 14:48 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-28 15:26 [Patch] BaseTools: Support multi thread build Basetool on Windows Liming Gao
2018-08-28 16:09 ` Carsey, Jaben
2018-08-29 14:47   ` Gao, Liming [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14E2EACA8@SHSMSX104.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox