RFR: dcubed - misc cleanups of jvm.cpp and JVM/TI files. [v2]

Daniel D.Daugherty dcubed at openjdk.java.net
Fri Apr 8 19:18:10 UTC 2022


On Fri, 8 Apr 2022 08:21:49 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   sspitsyn, AlanBateman CR - resolve CR0 feedback.
>
> src/hotspot/share/prims/jvmti.xml line 1868:
> 
>> 1866:         <functionlink id="ResumeThreadList"></functionlink> or
>> 1867:         <functionlink id="ResumeThread"></functionlink>.
>> 1868:         Virtual threads that are currently resumed do not change state.
> 
> Just to add to Serguei's comment. I think it would be clearer if we move the sentence "Virtual threads that are currently resumed do not change state" to before the "Virtual threads may be suspended ..."

I moved the sentence as @AlanBateman suggested.
I also moved the equivalent sentence in SuspendAllVirtualThreads.

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

PR: https://git.openjdk.java.net/loom/pull/140


More information about the loom-dev mailing list