RFR: 8306471: Add virtual threads support to JDWP ThreadReference.Stop and JDI ThreadReference.stop() [v4]

Chris Plummer cjplummer at openjdk.org
Tue May 2 20:17:25 UTC 2023


On Tue, 2 May 2023 19:57:32 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Some test logging improvements.
>
> test/hotspot/jtreg/vmTestbase/nsk/jdi/ThreadReference/stop/stop002.java line 63:
> 
>> 61:     // debuggee local var used to find needed non-throwable object
>> 62:     static final String DEBUGGEE_NON_THROWABLE_VAR= "stop002tNonThrowable";
>> 63:     // debuggee local var used to find needed non-throwable object
> 
> This comment is a little bit confusing.
> Should it be one line up or it is intentionally placed before `DEBUGGEE_THROWABLE_VAR`?

It should be "throwable", not "non-throwable"

> test/hotspot/jtreg/vmTestbase/nsk/jdi/ThreadReference/stop/stop002.java line 300:
> 
>> 298:         IntegerValue ival;
>> 299:         do {
>> 300:             ival = (IntegerValue)mainClass.getValue(mainClass.fieldByName("testNumReady"));
> 
> Do we need a sleep at each iteration?

I intentionally did not do one since `getValue()` is fairly slow and involves sending and receiving a JDWP packet. It didn't seem worth the extra noise of a sleep call and the needed exception handling.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13548#discussion_r1183002318
PR Review Comment: https://git.openjdk.org/jdk/pull/13548#discussion_r1183006005


More information about the serviceability-dev mailing list