[jdk17u-dev] RFR: 8292989: Avoid dynamic memory in AsyncLogWriter [v2]

Xin Liu xliu at openjdk.org
Fri Aug 4 19:40:10 UTC 2023


> This patch is *NOT* a clean backport.  The logic part is exactly the same, but I have to adjust code in the following parts:
> 1. change from KVHashtable to ResourceHashtable because we want to use resourceArea instead of C_HEAP. 
> 2. we added lambda-API iterator() to ResourceHashtable. Instead of backporting all patches from TIP, we just reuse the original iterator(). 
> 
> 
> template<typename Function>
> void iterate(Function function) const { // lambda enabled API
> 
> 
> Testing: 
> jdk-test1 tests including gtest. 
> Manually check there's no malloc/free in runtime.

Xin Liu has updated the pull request incrementally with one additional commit since the last revision:

  Revert resourceHash.hpp and use functor instead of C++ lambda.

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

Changes:
  - all: https://git.openjdk.org/jdk17u-dev/pull/1630/files
  - new: https://git.openjdk.org/jdk17u-dev/pull/1630/files/4511c52f..a0bf39dd

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=1630&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=1630&range=00-01

  Stats: 80 lines in 3 files changed: 44 ins; 26 del; 10 mod
  Patch: https://git.openjdk.org/jdk17u-dev/pull/1630.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/1630/head:pull/1630

PR: https://git.openjdk.org/jdk17u-dev/pull/1630


More information about the jdk-updates-dev mailing list