RFR: 8234930: Use MAP_JIT when allocating pages for code cache on macOS [v10]

Thomas Stuefe stuefe at openjdk.java.net
Tue Dec 15 17:49:59 UTC 2020


On Tue, 15 Dec 2020 17:40:39 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> src/hotspot/os/windows/os_windows.cpp line 3271:
>> 
>>> 3269: 
>>> 3270: char* os::reserve_memory_aligned(size_t size, size_t alignment, bool exec) {
>>> 3271:   // exec can be ignored
>> 
>> Change to "Support for exec not implemented" ? (Maybe even with an assert) (Or - leave that up to you - leave this parameter out of os::reserve_memory_aligned completely.)
>
> Actually, since exec and non-exec reservations are equal on windows, ignoring exec is a correct implementation. Assert would not fit here. Please let me know if the comment fails to deliver this message. Also, I have a prototype implementation of CDS support for macOS/AArch64 and it needs executable aligned mapping, I think we'll need this parameter anyway.

Okay, leave it as it is.

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

PR: https://git.openjdk.java.net/jdk/pull/294


More information about the hotspot-dev mailing list