RFR: 8016572 : (xs) Pass CONCURRENCY=$(JOBS) to test/Makefile

Erik Joelsson erik.joelsson at oracle.com
Fri Jun 14 07:24:33 UTC 2013


Change looks good to me.

/Erik

On 2013-06-14 01:53, Mike Duigou wrote:
> Hello all;
>
> This is a small change to the make test target with big consequences. It enables concurrent running of tests. The number of simultaneous test jobs is the same as number of build jobs.
>
> This issue probably should have been combined with 8014819 as the two issues have been tested together. Thanks go to Amy Lu and Alan Bateman for their feedback and experience reports with concurrent testing.
>
> JPRT is not impacted by this change since it invokes the test/Makefile directly.
>
> Mike
>
> diff -r b40d24f793d2 common/makefiles/Main.gmk
> --- a/common/makefiles/Main.gmk Tue Jun 04 10:36:00 2013 +0200
> +++ b/common/makefiles/Main.gmk Thu Jun 13 16:47:45 2013 -0700
> @@ -183,7 +183,7 @@
>   test: images test-only
>   test-only: start-make
>          @$(call TargetEnter)
> -       @($(CD) $(SRC_ROOT)/test&&  $(BUILD_LOG_WRAPPER) $(MAKE) -j1 -k JT_HOME=$(JT_HOME) MAKEFLAGS= PRODUCT_HOME=$(JDK_IMAGE_DIR) JPRT_JAVA_HOME=$(JDK_IMAGE_DIR) ALT_OUTPUTDIR=$(OUTPUT_ROOT) $(TEST)) || true
> +       @($(CD) $(SRC_ROOT)/test&&  $(BUILD_LOG_WRAPPER) $(MAKE) -j1 -k MAKEFLAGS= JT_HOME=$(JT_HOME) PRODUCT_HOME=$(JDK_IMAGE_DIR) JPRT_JAVA_HOME=$(JDK_IMAGE_DIR) ALT_OUTPUTDIR=$(OUTPUT_ROOT) CONCURRENCY=$(JOBS) $(TEST)) || true
>          @$(call TargetExit)
>
>   # Stores the tips for each repository. This file is be used when constructing the jdk image and can be
>



More information about the build-dev mailing list