RFR (XS): 8060116: After JDK-8047976 gc/g1/TestSummarizeRSetStatsThreads fails

Discussion:

Marcus Larsson

10 years ago

Hi,

I would like reviews for the following patch modifying the
TestSummarizeRSetStatsThreads. The patch modifies the test so that it
does not set ParallelGCThreads=0, since this is no longer allowed [1].
Additionally the test is modified to not use the 0 value for
G1ConcRefinementThreads as this is part of a different test [2], and
should not really be tested here.

Webrev:
http://cr.openjdk.java.net/~mlarsson/8060116/webrev.00/

Bug:
https://bugs.openjdk.java.net/browse/JDK-8060116

Tested locally with jtreg.

Thanks,
Marcus

[1]: https://bugs.openjdk.java.net/browse/JDK-8059527
[2]: https://bugs.openjdk.java.net/browse/JDK-8047976

Bengt Rutisson

10 years ago

Permalink

Looks good! Thanks for fixing this, Marcus. I'll sponsor the change.

Bengt

...

Stefan Johansson

10 years ago

Permalink

Change looks good.

Stefan

...