RFR: 8254012: NMT: MetaspaceSnapshot::snapshot uses wrong enum

Yumin Qi minqi at openjdk.java.net
Tue Oct 13 23:52:18 UTC 2020


Please review this simple change.
Snapshot should be done first on NonClassType which will include ClassType allocation when
Metaspace::using_class_space() is false.  _class_vsm (a VirtualSpaceManager) is set only for class type allocation when
using_class_space is true. The correct order is do snapshot for NonClassType first, then for ClassType if
using_class_space.

Tests: mach5 tier1-4 in progress.

-------------

Commit messages:
 - 8254012: NMT: MetaspaceSnapshot::snapshot uses wrong enum

Changes: https://git.openjdk.java.net/jdk/pull/645/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=645&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8254012
  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/645.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/645/head:pull/645

PR: https://git.openjdk.java.net/jdk/pull/645


More information about the hotspot-runtime-dev mailing list