RFR: 8274178: G1: Occupancy value in IHOP logging and JFR event is inaccurate [v3]

Tue Nov 11 14:30:32 UTC 2025

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

>> Hi all,
>> 
>>   please review this change that modifies heap occupancy reporting for IHOP to use the actual occupancy value the IHOP calculation actually uses.
>> 
>> I.e. previously IHOP logging used to report `G1CollectedHeap::used()` as occupancy (which is correct at a general level), but for IHOP/marking that value is not interesting, but the "occupancy" value used for comparing against the current heap occupancy threshold.
>> 
>> Testing: tier1-3, GHA
>> 
>> Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   * ayang, walulyai review
>   * also fix JDK-8371635

Marked as reviewed by iwalulya (Reviewer).

failing GHA

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

PR Review: https://git.openjdk.org/jdk/pull/28175#pullrequestreview-3448333532
Changes requested by iwalulya (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28175#pullrequestreview-3448340073