Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-02-06 Thread Alan Bateman
On 05/02/2013 20:10, Mike Duigou wrote: On Feb 4 2013, at 08:33 , Alan Bateman wrote: On 04/02/2013 16:02, Mike Duigou wrote: : If so then I think we have to careful because -concurrency means a lot of virtual memory and I'm not convinced that we limit it via -vmoption in jdk/test/Makefile

Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-02-05 Thread Mike Duigou
On Feb 4 2013, at 08:33 , Alan Bateman wrote: On 04/02/2013 16:02, Mike Duigou wrote: : If so then I think we have to careful because -concurrency means a lot of virtual memory and I'm not convinced that we limit it via -vmoption in jdk/test/Makefile I could see us wanting to dial this

Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-02-04 Thread Alan Bateman
On 02/02/2013 20:40, Mike Duigou wrote: I have updated both the jdk repo patch: 8006594: Add jdk_core target to jdk/test/Makefile http://cr.openjdk.java.net/~mduigou/JDK-8006594/1/webrev/ and the root repo patch: 8006595: Use jdk/test/Makefile targets in preference to local definitions

Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-02-04 Thread Mike Duigou
On Feb 4 2013, at 05:54 , Alan Bateman wrote: On 02/02/2013 20:40, Mike Duigou wrote: I have updated both the jdk repo patch: 8006594: Add jdk_core target to jdk/test/Makefile http://cr.openjdk.java.net/~mduigou/JDK-8006594/1/webrev/ and the root repo patch: 8006595: Use

Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-02-04 Thread Alan Bateman
On 04/02/2013 16:02, Mike Duigou wrote: : If so then I think we have to careful because -concurrency means a lot of virtual memory and I'm not convinced that we limit it via -vmoption in jdk/test/Makefile I could see us wanting to dial this down on 32-bit Windows for example. We do limit the

RFR : 8006594/8006595 : Define jdk_core test set

2013-02-02 Thread Mike Duigou
I have updated both the jdk repo patch: 8006594: Add jdk_core target to jdk/test/Makefile http://cr.openjdk.java.net/~mduigou/JDK-8006594/1/webrev/ and the root repo patch: 8006595: Use jdk/test/Makefile targets in preference to local definitions

Re: RFR : 8006594/8006595 : Define jdk_core test set

2013-01-23 Thread Alan Bateman
On 22/01/2013 21:15, Mike Duigou wrote: Hello all; This is a two piece patch. 8006594: Add jdk_core target to jdk/test/Makefile 8006595: Use jdk/test/Makefile targets in preference to local definitions I chose to do it as two patches because the changes are in different repos and the changes

RFR : 8006594/8006595 : Define jdk_core test set

2013-01-22 Thread Mike Duigou
Hello all; This is a two piece patch. 8006594: Add jdk_core target to jdk/test/Makefile 8006595: Use jdk/test/Makefile targets in preference to local definitions I chose to do it as two patches because the changes are in different repos and the changes in 8006594 could be committed without