On Thu, 29 May 2025 18:21:11 GMT, Alex Menkov <amen...@openjdk.org> wrote:

>> This is first (hotspot) part of the update for 
>> `HotSpotDiagnosticMXBean.dumpThreads` and `jcmd Thread.dump_to_file` to 
>> include lock information in thread dumps (JDK-8356870).
>> The update has been split into parts to simplify reviewing.
>> The fix contains an implementation of `jdk.internal.vm.ThreadSnapshot` class 
>> to gather required information about a thread.
>> Second (dependent) part includes changes in 
>> `HotSpotDiagnosticMXBean.dumpThreads`/`jcmd Thread.dump_to_file`, spec 
>> updates and tests for the functionality.
>> 
>> Testing: new `HotSpotDiagnosticMXBean.dumpThreads`/`jcmd 
>> Thread.dump_to_file` functionality was tested in loom repo;
>>   sanity tier1 (this fix only)
>
> Alex Menkov has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   typo

Alex, thank you for the updates!
Looks good now.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25425#pullrequestreview-2879223457

Reply via email to