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

Thomas Schatzl tschatzl at openjdk.org
Wed Oct 15 05:52:35 UTC 2025


On Wed, 15 Oct 2025 05:49:16 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
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update src/hotspot/share/gc/g1/g1CollectedHeap.cpp

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

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

Suggestion:

  st->print("region size %zuM, ", G1HeapRegion::GrainBytes / M);

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27798#discussion_r2431191197


More information about the hotspot-gc-dev mailing list