RFR(S): 8215934: G1 Old Gen MemoryPool CollectionUsage.used values don't reflect mixed GC results

[Hohensee, Paul]

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.

Paul

On 1/17/19, 2:13 AM, "Thomas Schatzl" <thomas.schatzl at oracle.com> wrote:

    Hi,
    
    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
    > Thomas.
    >  
    > 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
    subdirectory anyway).
    
    The MaxGCPauseMillis @requires does not hurt, but seems to serve no
    real purpose.
    
    Looks good.
    
    Thomas