RFR: 8357445: G1: Time-Based Heap Uncommit During Idle Periods [v4]

[Monica Beckwith]

On Wed, 27 Aug 2025 11:22:10 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> src/hotspot/share/gc/g1/g1HeapEvaluationTask.hpp line 36:
>> 
>>> 34: // Time-based heap evaluation task that runs during idle periods.
>>> 35: // Uses PeriodicTask rather than G1ServiceTask due to build compatibility issues
>>> 36: // in JDK 25+. PeriodicTask's 10ms granularity is adequate for heap evaluation
>> 
>> Can you please elaborate on these "compatibility issues"
>
> +1. I would absolutely try to avoid subscribing to `PeriodicTask`. Is there something missing with `G1ServiceTask`?

changed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26240#discussion_r2402353147