RFR: 8338011: CDS archived heap object support for 64-bit Windows [v2]

Ioi Lam iklam at openjdk.org
Sun Aug 11 21:17:08 UTC 2024


On Sat, 10 Aug 2024 16:43:51 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   @tstuefe review -- changed error message
>
> src/hotspot/share/cds/filemap.cpp line 2188:
> 
>> 2186:                      /* do_commit = */ true)) {
>> 2187:       dealloc_heap_region();
>> 2188:       log_error(cds)("Failed to read archived heap region at " INTPTR_FORMAT, p2i(addr));
> 
> Very minor bikeshedding, we don't try to read the heap region at this address but load it into memory at that address. Up to you if you change anything.

I changed the message to "Failed to read archived heap region into 0x12345678...". Does that sound better?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20514#discussion_r1713059616


More information about the hotspot-dev mailing list