RFR (S): 8190703: TestSystemGCWith* infrequently times out on SPARC
thomas.schatzl at oracle.com
Tue Nov 7 09:16:50 UTC 2017
thanks for your review, and sorry for the bad quality of the patch :(
On Mon, 2017-11-06 at 20:46 +0100, Aleksey Shipilev wrote:
> On 11/06/2017 12:27 PM, Thomas Schatzl wrote:
> > can I have reviews for this change the fixes intermittent
> > timeouts in
> > a stress test when run on slow machines and debug builds.
> > The fix is to simply let the test bail out early if time is up -
> > this
> > allows the test run through as intended in the many configurations
> > where it does not take excessive amount of time.
> Makes sense. Real timeouts would still surface.
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8190703
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8190703/webrev
> *) This change seems like a leftover:
> 93 System.err.println("SystemGCTask with delay " + delayMS);
> *) This change also does not look right -- should we instead increase
> delays in callers?
> 101 ThreadUtils.sleep(delayMS * 10);
Another debugging leftover when I tried to come up with alternative
> *) I think you can just say:
> for (int i = 0; (i < 4) && (System.currentTimeMillis() < endTime);
> ...instead of rewriting the whole loop.
Also added some very basic check that a timeout is passed to the main
http://cr.openjdk.java.net/~tschatzl/8190703/webrev.1/ (full patch)
I rechecked that these changes do not change the execution times of the
More information about the hotspot-gc-dev