RFR: 8296155: Simplify CDS heap region address calculation [v2]

Ioi Lam iklam at openjdk.org
Wed Nov 2 03:56:10 UTC 2022


> Rewrite and simplify the convoluted logic used to calculate the runtime location of a CDS heap region. Now it's as simple as:
> 
> 
> FileMapRegion::dumptime_base() + ArchiveHeapLoader::runtime_delta();
> 
> 
> See `FileMapInfo::heap_region_runtime_start_address` for an example.
> 
> Also removed dead code:
> 
> - `FileMapInfo::is_in_shared_region`
> - `MetaspaceShared::is_in_shared_region`
> - `FileMapInfo::stop_sharing_and_unmap`

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  @coleenp comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10923/files
  - new: https://git.openjdk.org/jdk/pull/10923/files/5cbbe3a5..06f85c9e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10923&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10923&range=00-01

  Stats: 5 lines in 2 files changed: 3 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/10923.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10923/head:pull/10923

PR: https://git.openjdk.org/jdk/pull/10923


More information about the hotspot-runtime-dev mailing list