Request for review for JDK-8184045 - TestSystemGCWithG1.java times out on Solaris SPARC

[Daniel D. Daugherty]

On 7/20/17 9:31 AM, Alexander Harlap wrote:
> Please review changes forJDK-8184045 
> <https://bugs.openjdk.java.net/browse/JDK-8184045> - 
> TestSystemGCWithG1.java times out on Solaris SPARC (labeled  as 
> integration blocker)
>
> Proposed change will avoid timeout failure for new stress tests 
> TestSystemGCWithG1.java, TestSystemGCWithCMS.java and 
> TestSystemGCWithSerial.java regularly observed on Solaris Sparc
>
> Change is located at 
> http://cr.openjdk.java.net/~aharlap/8184045/webrev.00/

test/gc/stress/systemgc/TestSystemGC.java
     L32: import jdk.test.lib.Platform;
         Not sure why this was added.

     (old) L187:         runAllPhases();
         So your theory is that calling runAllPhases() twice
         takes too much time on the Solaris SPARC machines.

         What are the average times before and after making
         this change on Solaris SPARC?

Thumbs up! Don't need to see a new webrev if you decide
to delete the new import.

Dan

>
>
> Alex
>