On Fri, 18 Jun 2021 06:51:53 GMT, Denghui Dong <dd...@openjdk.org> wrote:

>> Hi,
>> 
>> Could I have a review of this change that merges three vm 
>> operations(VM_PrintThreads, VM_PrintJNI, VM_FindDeadlocks) in thread_dump 
>> and signal_thread_entry.
>> 
>> `jstack` is a very common command, even in the production environment.
>> 
>> In addition to reduce the cost of entering safepoint, I think this patch 
>> also could ensure the consistency of the results of VM_PrintThreads and 
>> VM_FindDeadlocks.
>
> Denghui Dong has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   update

Nothing further from me.

Thanks,
David

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4504

Reply via email to