On Fri, 13 Jan 2023 17:39:41 GMT, Justin King <jck...@openjdk.org> wrote:
>> Remove abstraction that is a holdover from Solaris. Direct usages of >> `MmapArrayAllocator` have been switched to normal `malloc`. The >> justification is that none of the code paths are called from signal >> handlers, so using `mmap` directly does not make sense and is potentially >> slower than going through `malloc` which can potentially re-use memory >> without making any system calls. The remaining usages of `ArrayAllocator` >> and `MallocArrayAllocator` are equivalent. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Apply patch from stefank@ > > Signed-off-by: Justin King <jck...@google.com> Restored `mmap` for G1, though in the future it might make sense to switch to just using malloc. ------------- PR: https://git.openjdk.org/jdk/pull/11931