RFR: 8369818: G1: Add missing region types in gc+heap=debug heap output

Albert Mingkun Yang ayang at openjdk.org
Tue Oct 14 12:49:05 UTC 2025


On Tue, 14 Oct 2025 11:07:00 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this update to the `gc+heap=debug` output to contain all region types (I do not know why it only showed eden and survivor).
> 
> This is also printed in other places (like `hs_err` files) and it is sometimes very interesting to know that e.g. there are humongous regions around.
> 
> It also makes the unit MB because KB is too unwieldy, and the minimum region size is 1 MB anyway.
> 
> Testing: local compilation, checking the log output
> 
> Thanks,
>   Thomas

Marked as reviewed by ayang (Reviewer).

src/hotspot/share/gc/g1/g1CollectedHeap.cpp line 2287:

> 2285: 
> 2286:   StreamIndentor si(st, 1);
> 2287:   st->print("region size %zuK, ", G1HeapRegion::GrainBytes / K);

Maybe this line can also use `/ M`?

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

PR Review: https://git.openjdk.org/jdk/pull/27798#pullrequestreview-3335429977
PR Review Comment: https://git.openjdk.org/jdk/pull/27798#discussion_r2429034299


More information about the hotspot-gc-dev mailing list