RFR: 8299915: Remove ArrayAllocatorMallocLimit and associated code [v9]

Justin King jcking at openjdk.org
Fri Jan 13 17:50:50 UTC 2023


> 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 two additional commits since the last revision:

 - Fix compilation error
   
   Signed-off-by: Justin King <jcking at google.com>
 - Preserve mmap usage in gc/g1/g1ConcurrentMark
   
   Signed-off-by: Justin King <jcking at google.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11931/files
  - new: https://git.openjdk.org/jdk/pull/11931/files/4aba784e..4b5ce84b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11931&range=08
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11931&range=07-08

  Stats: 33 lines in 2 files changed: 30 ins; 1 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/11931.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11931/head:pull/11931

PR: https://git.openjdk.org/jdk/pull/11931


More information about the serviceability-dev mailing list