RFR: 8374469: Mixed jstack does not work on Windows [v7]
Yasumasa Suenaga
ysuenaga at openjdk.org
Fri Jan 23 03:06:28 UTC 2026
On Thu, 22 Jan 2026 02:22:23 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> Yasumasa Suenaga has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update TestJhsdbJstackMixed.java
>
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/debugger/windbg/WindbgCDebugger.java line 88:
>
>> 86: .filter(jt -> thread.equals(jt.getThreadProxy()))
>> 87: .findFirst()
>> 88: .orElse(null);
>
> I wonder if this code should be moved into its own API. Call it something like mapThreadProxyToJavaThread(), or maybe getJavaThreadForThreadProxy(). It would make the code more readable.
Thanks for your comment. I extracted this logic as `mapThreadProxyToJavaThread()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29019#discussion_r2719408169
More information about the serviceability-dev
mailing list