Integrated: 8351167: ZGC: Lazily initialize livemap

[Joel Sikström]

On Tue, 4 Mar 2025 19:55:38 GMT, Joel Sikström <jsikstro at openjdk.org> wrote:

> Memory for the bitmap inside the livemap of a ZPage is currently allocated upon calling its constructor, which adds a latency overhead when allocating pages. As preparation for the Mapped Cache ([JDK-8350441](https://bugs.openjdk.org/browse/JDK-8350441)), but also as a standalone improvement, we want to instead lazily initialize the livemap's bitmap.
> 
> This patch holds off with allocating memory for the bitmap that the livemap uses until the livemap is written to the first time (i.e. by calling ZLiveMap::set). The effect of this is that the latency impact of allocating the bitmap will only be taken by GC threads and not by mutator threads, since only GC threads mark objects before pushing them onto the mark stack. This improvement will reduce page allocation latencies somewhat.
> 
> In addition to lazily allocating the bitmap, I've converted the static C-style cast to a checked cast for `ZPage::object_max_count()`, which is passed as the size to the bitmaps. This is because a value not contained in 32 bits will overflow with the C-style cast and give a too small bitmap when passed to the livemap. This is not an observed issue, just more of a sanity check.
> 
> Testing:
> * Tiers 1-5
> * GHA

This pull request has now been integrated.

Changeset: 2672c40b
Author:    Joel Sikström <jsikstro at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2672c40bf10a6597ae861e2183e7558ffed43dba
Stats:     25 lines in 4 files changed: 17 ins; 1 del; 7 mod

8351167: ZGC: Lazily initialize livemap

Reviewed-by: sjohanss, eosterlund, tschatzl

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

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