RFR: 8346143: add ClearAllFramePops function to speedup debugger single stepping in some cases [v4]
Serguei Spitsyn
sspitsyn at openjdk.org
Wed Dec 18 00:22:11 UTC 2024
> New JVMTI function `ClearAllFramePops` will help to speedup debugger single stepping in some cases.
> Additionally, the JVMTI `NotifyFramePop` implementation was fixed to return `JVMTI_ERROR_DUPLICATE` to make it consistent with the `SetBreakpoint` which also returns this error.
>
> The JDWP agent fix will be needed to make use of this new JVMTI function. The corresponding debugger bug is:
> [8229012](https://bugs.openjdk.org/browse/JDK-8229012): When single stepping, the debug agent can cause the thread to remain in interpreter mode after single stepping completes
>
> CSR: [8346144](https://bugs.openjdk.org/browse/JDK-8346144): add ClearAllFramePops function to speedup debugger single stepping in some cases
>
> Testing:
> - mach5 tiers 1-6 were run to make sure this fix caused no regressions
> - Chris tested the JVMTI patch with his JDWP fix of [8229012](https://bugs.openjdk.org/browse/JDK-8229012).
Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
review: add a unit test covering JVMTI ClearAllFramePops
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/22744/files
- new: https://git.openjdk.org/jdk/pull/22744/files/0e4ddcc1..156c1580
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=22744&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=22744&range=02-03
Stats: 261 lines in 2 files changed: 261 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/22744.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22744/head:pull/22744
PR: https://git.openjdk.org/jdk/pull/22744
More information about the serviceability-dev
mailing list