RFR(S): 8215934: G1 Old Gen MemoryPool CollectionUsage.used values don't reflect mixed GC results
hohensee at amazon.com
Fri Jan 18 21:06:36 UTC 2019
Thank you for your review, Thomas.
I used TestLogging.java in the same directory as a template for TestOldGenCollectionUsage.java. I'd expect that running all the gc tests directly from jtreg would try to run these, but I'm not a jtreg expert so I'm not certain. The tests in the parent directory that run othervm include "@requires vm.gc.G1".
@requires vm.opt.MaxGCPauseMillis == "null" could go away, it's a copy-paste artifact. TestLogging.java looks to need to override MaxGCPauseMillis, but TestOldGenCollectionUsage.java doesn't. I'll file a separate (trivial) RFE to fix that in tip and backport it.
On 1/17/19, 2:13 AM, "Thomas Schatzl" <thomas.schatzl at oracle.com> wrote:
On Wed, 2019-01-09 at 20:41 +0000, Hohensee, Paul wrote:
> Thomas reviewed the CSR, so I’ve put it into Finalized state. Thanks
> Please review the backport.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8215934
> Webrev: http://cr.openjdk.java.net/~phh/8215934/webrev.00/
one nit: I do not completely understand the " 30 * @requires
vm.opt.MaxGCPauseMillis == "null"" in TestOldGenCollectionUsage.java as
the test itself runs in othervm mode anyway.
(The vm.gc=="G1" | vm.gc=="null" has traditionally been used to limit
the number of invocations of this test, i.e. to not get repeats when
running all tests with all collectors. But then the test is in the g1
The MaxGCPauseMillis @requires does not hurt, but seems to serve no
More information about the hotspot-gc-dev