On Wed, 1 May 2024 20:42:16 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: Corrections in: 1) JVMTI/JDWP spec; 2) test vthread checks; 3) >> test comments > > src/java.se/share/data/jdwp/jdwp.spec line 1622: > >> 1620: (threadObject owner "The platform thread owning this >> monitor, or <code>nullptr</code> " >> 1621: "if owned` by a virtual thread or not >> owned.") >> 1622: (int entryCount "The number of times the owning platform >> thread has entered the monitor.") > > See the comment I left for the JVMTI spec. We should be more complete in the > explanation here, explaining how it is 0 for virtual threads. I don't think this has been resolved. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19030#discussion_r1588459462
