RFR: JDK-8009152 - OPEN - A number of jtreg tests need review/improvement

[Gary Collins]

Applied changes Vladimir and Alan suggested.

New webrev can be found here.
http://cr.openjdk.java.net/~collins/8009152/webrev-01/

Please let me know if you see any issues with changes.

Thanks
Gary
On Mar 6, 2013, at 5:59 PM, Vladimir Kozlov wrote:

> test_env.sh should have Copyright header since it in open. Also there is duplicated java -version commands. I think first one at line 12 should be removed and the check (lines 14-18) moved after second one at line 77.
> 
> Changes in tests scripts are fine.
> 
> Thanks,
> Vladimir
> 
> On 3/6/13 4:43 PM, Gary Collins wrote:
>> Request for Review
>> 
>> Changes to allow better test utilization of jtreg hotspot regression tests
>> 	1: REPLACED JAVA_HOME with TESTJAVA
>> 	2: REPLACED TESTJAVA with COMPILEJAVA Some tests needed tools like  (javac,jar,path to native libraries) to run.
>> 		NOTE: THIS IS NEEDED WHEN TESTJAVA is a JRE. If TESTJAVA is a JDK then COMPILEJAVA is not needed.
>> 		NOTE: FALLBACK when COMPILEJAVA is not defined for JTREG is TESTJAVA (with implied JDK being tested)
>> 	3: Replaced most of the BIT_FLAG with TESTVMOPTS
>> 		NOTE: This allows the tests where possible to run with multiple compilers and switches. Asked for by Quality team
>> 	4: Added TESTJAVACOPTS for all javac compile lines. Suggestion from tools folks.
>> 	5: Added a new shell env template.  Phase 2 of hotspot test cleanup will utilize this file
>> 		NOTE: This test_env.sh file will allow engineers to remove allot of the copy/paste of test setup from each shell testcase.
>>  	6: Fixed a gc testcase that was timing out on a 24 core box.. Simple fix , add a timeout.
>> 	7: I filed many bugs around native case issues.  This changeset it not to address broken tests. Only to conform them in
>> 		a unified way of running.
>> 	8: Fixing tests as defined above allows us to run the tests from a JRE.  embedded use cases this is a stop gap.
>> 	9: In order to run jtreg with a JRE you need to define TESTJAVA (JRE - jdk/testjdk) & COMPILEJAVA (JDK - compilejdk)
>> 		If TESTJAVA is a JDK no need to set COMPILEJAVA.
>> 		NOTE: Most shell scripts from the command line may fail with the following issues:
>> 			TESTSRC not defined, TESTCLASSES not defined, COMPILEJAVA path not found because we don't
>> 			check for them at runtime.  #5 above will fix this in the next phase of improving these tests to be more
>> 			robust.
>> 
>> 	Fully tested on linux-i586 builds with server/client jvm.
>> 
>> Webrev:
>> http://cr.openjdk.java.net/~collins/8009152/webrev-rev0/
>> 
>> Please let me know if you have any suggestions.
>> Thanks
>> Gary
>> 	
>> 
>>