Request for review for JDK-8184045 - TestSystemGCWithG1.java times out on Solaris SPARC
alexander.harlap at oracle.com
Thu Jul 20 17:28:48 UTC 2017
Thank you for your review.
I will remove erroneous extra import.
Running jtreg test TestSystemGCWithG1.java (outside of aurora) on
Solaris Sparc takes about 2900 sec (before change) and is twice faster
With aurora it will be slower, but should be enough to eliminate timeout.
On 7/20/2017 12:37 PM, Daniel D. Daugherty wrote:
> 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
> 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.
More information about the hotspot-dev