RFR: 8268857: Merge three vm operations in thread_dump [v2]
Denghui Dong
ddong at openjdk.java.net
Thu Jun 17 05:15:36 UTC 2021
> 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
More information about the serviceability-dev
mailing list