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

Thomas Stuefe stuefe at openjdk.org
Mon Aug 12 10:41:33 UTC 2024


On Sun, 11 Aug 2024 21:17:08 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> We didn't support CDS archived heap object on Windows because
>> 
>> - The Windows implementation of `os::map_memory()` cannot map the contents of a file into a region that's already reserved by the garbage collector.
>> - We had a high failure rate for mapping the CDS archive on Windows due to ASLR, sometimes as high as 50%. So it didn't seem worth the effort (mainly testing) to support archived heap objects on Windows.
>> 
>> Both of the above issues were fixed in [JDK-8231610](https://bugs.openjdk.org/browse/JDK-8231610), so we should add the support to Windows now.
>> 
>> (Tested on Oracle CI tiers 1-7)
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
> 
>   @tstuefe review -- changed error message

This looks good. Thanks for changing the error message.

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

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20514#pullrequestreview-2232620901


More information about the hotspot-dev mailing list