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 > _Mailing list message from [David Holmes](mailto:david.hol...@oracle.com) on > [serviceability-dev](mailto:serviceability-...@mail.openjdk.java.net):_ > > Hi, > > Please note that hotspot changes require two reviews before integrating > (unless a trivial change). > > David > > On 22/06/2021 6:31 pm, Denghui Dong wrote: Sorry about this, I will be more aware next time. Denghui. ------------- PR: https://git.openjdk.java.net/jdk/pull/4504
