RFR: 8229012: When single stepping, the debug agent can cause the thread to remain in interpreter mode after single stepping completes [v2]

Fri Jan 24 20:09:31 UTC 2025

On Fri, 24 Jan 2025 11:52:04 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 minor cleanup.
>
> src/jdk.jdwp.agent/share/native/libjdwp/stepControl.c line 913:
> 
>> 911: 
>> 912:         if (needsSuspending) {
>> 913:             tty_message("clearStep: suspending thread");
> 
> Nit: This `tty_message()` is still here. Should it be at least under `#ifdef DEBUG`?

It's gone. Are you sure you are looking at the latest sources?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23182#discussion_r1929176082