[jdk17u-dev] RFR: 8319876: Reduce memory consumption of VM_ThreadDump::doit

[Goetz Lindenmaier]

On Wed, 19 Nov 2025 23:42:58 GMT, Elif Aslan <duke at openjdk.org> wrote:

>> This pull request contains a backport of commit [8ec6b8de3bb3d7aeebdcb45d761b18cce3bab75e](https://github.com/openjdk/jdk/commit/8ec6b8de3bb3d7aeebdcb45d761b18cce3bab75e) from the openjdk/jdk repository to JDK 17.
>> 
>> This backport significantly reduces RSS memory usage during ThreadMXBean.dumpAllThreads(boolean, boolean) execution.
>> 
>> This is not a clean backport due to differences between JDK 17 and upstream:
>> 
>> Threading Context: Changed ResourceMark rm(VMThread::vm_thread()) to ResourceMark rm;
>>    • Similar to [JDK 21 backport](https://github.com/openjdk/jdk21u-dev/pull/307)
>> 
>> Minor conflict resolved: _threads = nullptr; → _threads = NULL; to match JDK 17 conventions
>> 
>> GHA tested additionally ran tier1 tests on my local mac and passed.
>
> /Approval request. This pull request contains a backport of commit [8ec6b8de3bb3d7aeebdcb45d761b18cce3bab75e](https://github.com/openjdk/jdk/commit/8ec6b8de3bb3d7aeebdcb45d761b18cce3bab75e) from the openjdk/jdk repository to JDK 17.
> 
> This backport significantly reduces RSS memory usage during ThreadMXBean.dumpAllThreads(boolean, boolean) execution.
> 
> This is not a clean backport due to differences between JDK 17 and upstream:
> 
> Threading Context: Changed ResourceMark rm(VMThread::vm_thread()) to ResourceMark rm;
> • Similar to https://github.com/openjdk/jdk21u-dev/pull/307
> 
> Minor conflict resolved: _threads = nullptr; → _threads = NULL; to match JDK 17 conventions
> 
> Ran tier 2 tests as recommended in addition to GHA tested additionally ran tier1 tests on my local mac and passed.

Hi @elifaslan1 
I removed the label. No fix request comment supplied.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/4006#issuecomment-3581673841