Re: [PATCH 1/2] tools: OVMF parallel build

[Ian Campbell <ian.campbell@citrix.com>]

On Mon, 2015-03-02 at 10:52 +0000, Wei Liu wrote:
> Though it doesn't work with make's "-j" option, the build system of OVMF
> has an option to specify parallel threads used to run the build.
> 
> Using 4 threads to build OVMF looks like a sensible default.

It's a bit arbitrary, but it'll do, I suppose.

It's worth noting that this won't interact with gnu makes token handling
which is used to make sure there are only N processes across the
recursive build. So this will consume 1 token but launch 4 threads. So
if -j4 is used you'll actually get 7 things happening.

I think we can live with that.

So I'm happy with the patch, but I'd like to give others a chance to
disagree before applying.

> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> Cc: Anthony Perard <anthony.perard@citrix.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> ---
>  tools/firmware/ovmf-makefile | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/firmware/ovmf-makefile b/tools/firmware/ovmf-makefile
> index 1ad041f..2838744 100644
> --- a/tools/firmware/ovmf-makefile
> +++ b/tools/firmware/ovmf-makefile
> @@ -1,6 +1,3 @@
> -# OVMF building system is not ready yet to run in parallel.
> -# Force it to be serial in order to exploit parallelism for neighbors.
> -
>  XEN_ROOT=$(CURDIR)/../../..
>  include $(XEN_ROOT)/tools/Rules.mk
>  
> @@ -10,6 +7,7 @@ else
>  TARGET=RELEASE
>  endif
>  
> +# OVMF build system has its own parallel building support.
>  .NOTPARALLEL:
>  MAKEFLAGS  += -j1

TBH I'm not sure what if anything these actually do...

>  
> @@ -18,7 +16,7 @@ all: build
>  
>  .PHONY: build
>  build:
> -	OvmfPkg/build.sh -a X64 -b $(TARGET)
> +	OvmfPkg/build.sh -a X64 -b $(TARGET) -n 4
>  	cp Build/OvmfX64/$(TARGET)_GCC*/FV/OVMF.fd ovmf.bin
>  
>  .PHONY: clean