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

Paul Hohensee phh at openjdk.org
Fri Jul 28 14:52:57 UTC 2023


On Fri, 28 Jul 2023 08:18:46 GMT, Xin Liu <xliu at openjdk.org> wrote:

> 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.

Marked as reviewed by phh (Reviewer).

windows-aarch64 build failure running adlc appears unrelated.

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

PR Review: https://git.openjdk.org/jdk17u-dev/pull/1630#pullrequestreview-1552281976
PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1630#issuecomment-1655820122


More information about the jdk-updates-dev mailing list