Re: RFR: 8214455: Relocate CDS archived regions to the top of the G1 heap [v2]

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8214455: Relocate CDS archived regions to the top of the G1 heap [v2]

Ioi Lam-2
> When the runtime heap and dump time heap have different sizes, it's possible that the archived heap regions are mapped to the middle of the runtime heap. Because the archived heap regions are pinned, this could reduce the size of the largest "humongous" array allocation. In the worst case, the maximum allocatable array length may be half of the optimal value.
>
> The fix is to relocate the archived regions to the top of the  G1 heap (currently archived regions are supported only by G1 on 64-bit JVM).
>
> Note that usually the top of the G1 heap is placed just below the 32GB boundary. As a result, the archived heap regions are at the same location between run time and dump time, so no relocation is necessary.
>
> In mach5 testing, we occasionally run into this problem (see https://bugs.openjdk.java.net/browse/JDK-8239089), probably because we fail to reserve the heap below 32GB due to address space layout randomization (ASLR).

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

  @tschatzl review -- fix typos and whitespace

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3349/files
  - new: https://git.openjdk.java.net/jdk/pull/3349/files/def7db7b..bedd676d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3349&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3349&range=00-01

  Stats: 6 lines in 3 files changed: 0 ins; 0 del; 6 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3349.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3349/head:pull/3349

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