RFR: 8357445: G1: Time-Based Heap Uncommit During Idle Periods [v3]
Monica Beckwith
mbeckwit at openjdk.org
Thu Jul 17 01:13:42 UTC 2025
> **Implements:** https://bugs.openjdk.org/browse/JDK-8357445
>
> Implement time-based heap uncommit for G1 during idle periods.
>
> Key changes:
> - Added G1HeapEvaluationTask for periodic heap evaluation
> - Switch from G1ServiceTask to PeriodicTask for improved scheduling
> - Implemented time-based heap sizing policy with configurable uncommit delay
> - Added region activity tracking with last access timestamps
> - Integrated VM_G1ShrinkHeap operation for safe heap shrinking
> - Added new G1 flags: G1UseTimeBasedHeapSizing, G1TimeBasedEvaluationIntervalMillis, G1UncommitDelayMillis, G1MinRegionsToUncommit
> - Added 'sizing' log tag for heap sizing operations
>
> Comprehensive Test Coverage:
> - Enhanced TestG1RegionUncommit: minimum heap boundaries, concurrent allocation/uncommit scenarios
> - Enhanced TestTimeBasedHeapSizing: humongous object handling, rapid allocation cycles, edge cases
> - Enhanced TestTimeBasedRegionTracking: concurrent region access, lifecycle transition validation
> - Enhanced TestTimeBasedHeapConfig: parameter boundary values, small heap configurations
>
> This ensures time-based heap uncommit works correctly while maintaining all safety guarantees and test expectations.
Monica Beckwith has updated the pull request incrementally with one additional commit since the last revision:
8357445: Address code review feedback
Extra line removed
Include cycle comment removed
completed changed to flagged
nullptr assignment changed to assert
Redundant nullptr check removed
VM_G1ShrinkHeap moved
Forward declaration removed
Constructor initialization for static _uncommit_delay_ms
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26240/files
- new: https://git.openjdk.org/jdk/pull/26240/files/70a872fc..8897a2b0
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26240&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26240&range=01-02
Stats: 69 lines in 12 files changed: 26 ins; 32 del; 11 mod
Patch: https://git.openjdk.org/jdk/pull/26240.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26240/head:pull/26240
PR: https://git.openjdk.org/jdk/pull/26240
More information about the hotspot-gc-dev
mailing list