> HeapDumper dumps virtual threads in 2 places:
> - dumping platform threads (mounted virtual threads are dumped as separate 
> thread object);
> - dumping heap objects when the object is `java.lang.VirtualThread`.
> 
> In the 2nd case mounted virtual threads should be skipped (as they are 
> already dumped with correct stack traces/stack references)
> Check that a virtual thread is mounted is non-trivial, method from 
> JvmtiEnvBase was used for this.
> 
> Testing: tier1..3, heapdump-related tests: 
> open/test/hotspot/jtreg/serviceability,open/test/hotspot/jtreg/runtime/ErrorHandling,open/test/hotspot/jtreg/gc/epsilon,open/test/jdk/sun/tools/jhsdb

Alex Menkov has updated the pull request incrementally with one additional 
commit since the last revision:

  feedback: extended comment

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17134/files
  - new: https://git.openjdk.org/jdk/pull/17134/files/c6ad90ca..011ed399

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17134&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17134&range=02-03

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/17134.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17134/head:pull/17134

PR: https://git.openjdk.org/jdk/pull/17134

Reply via email to