RFR: 8309146: extend JDI StackFrame.setValue() and JDWP StackFrame.setValues minimal support for virtual threads

Serguei Spitsyn sspitsyn at openjdk.org
Wed May 31 01:16:01 UTC 2023


On Tue, 30 May 2023 23:44:28 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> The JDWP spec for StackFrame.SetValue currently states:
> 
>         "If the thread is a virtual thread then this command can be used to set "
>         "the value of local variables in the top-most frame when the thread is "
>         "suspended at a breakpoint or single step event. The target VM may support "
>         "setting local variables in other cases."
> 
> The JDI spec for StackFrame.setValue() has similar wording. In [JDK-8308814](https://bugs.openjdk.org/browse/JDK-8308814) the JVMTI spec clarified support to be for a thread suspended at any event, not just a breakpoint or single step. That same clarification is needed in the JDWP and JDI specs. No implementation changes are needed.

The fix looks good.
I've reviewed the CSR as well.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14235#pullrequestreview-1452057650


More information about the serviceability-dev mailing list