> 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 three additional commits since the last revision: - update - remove useless comment - remove VM_PrintJNI ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/4504/files - new: https://git.openjdk.java.net/jdk/pull/4504/files/82a35b85..56cd2a8b Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4504&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4504&range=00-01 Stats: 38 lines in 5 files changed: 4 ins; 28 del; 6 mod Patch: https://git.openjdk.java.net/jdk/pull/4504.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4504/head:pull/4504 PR: https://git.openjdk.java.net/jdk/pull/4504
