RFR: 8251158: Implementation of JEP 387: Elastic Metaspace [v8]
Thomas Stuefe
stuefe at openjdk.java.net
Mon Oct 5 10:28:44 UTC 2020
On Thu, 1 Oct 2020 19:58:31 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The pull request now
>> contains 13 commits:
>> - Merge branch 'master' into jep387
>> - Remove accidentally added file
>> - Merge branch 'jep387' of github.com:tstuefe/jdk into jep387
>> - Create submit.yml
>> - Review feedback Ioi
>> - cds MaxMetaspaceSize test does not need MaxMetaspaceSize increase
>> - Make MetaspaceGuardAllocations a diagnostic flag (2)
>> - Make MetaspaceGuardAllocations a diagnostic flag
>> - Style fixes
>> - Remove empty lines from include sections
>> - ... and 3 more: https://git.openjdk.java.net/jdk/compare/f80a6066...f5cf615b
>
> src/hotspot/share/services/virtualMemoryTracker.cpp line 679:
>
>> 677: }
>> 678:
>> 679: void MetaspaceSnapshot::snapshot(MetaspaceSnapshot& mss) {
>
> This is not part of your change, but aren't `ClassType` and `NonClassType` mixed up in the body of
> `MetaspaceSnapshot::snapshot` method?
Looks like it. I opened https://bugs.openjdk.java.net/browse/JDK-8254012.
-------------
PR: https://git.openjdk.java.net/jdk/pull/336
More information about the hotspot-gc-dev
mailing list