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

[Thomas Stuefe]

On Fri, 4 Aug 2023 19:40:10 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. rewrite lambda expression using Functor. 
>> 
>> 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.

Is it necessary to downport this patch? On its own, it does not sound that important, especially if we downport the glibc autotrimmer.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1630#issuecomment-1666408147