RFR: 8348349: Refactor CDSConfig::is_dumping_heap() [v2]

Calvin Cheung ccheung at openjdk.org
Wed Jan 29 18:00:48 UTC 2025


On Mon, 27 Jan 2025 05:51:58 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Please review this small clean up:
>> 
>> `HeapShared::can_write()` and `CDSConfig::is_dumping_heap()` are both for deciding whether CDS should dump heap objects. I removed the former and consolidated all the logic to the latter.
>> 
>> I also updated the logging message in case heap objects cannot be dumped.
>> 
>> I also updated VMProps to clarify what `vmCDSCanWriteArchivedJavaHeap()` means.
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
> 
>   @matias9927 comment

LGTM

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

Marked as reviewed by ccheung (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23249#pullrequestreview-2581823665


More information about the hotspot-dev mailing list