> This is a minor cleanup for the JVMTI `interp_only_mode` implementation which > includes the following changes: > - The `interp_only_mode` in `JavaThread` is represented with a counter which > is incremented and decremented. This is confusing because this value should > only take values `0` or `1`. Asserts are placed to make sure it is never > going out of bounds. The `interp_only_mode` in a `JavaThread` is checked by > the interpreter chunks which expect it to be an `integer`. This cleanup has > no intention to make it a boolean. > - The function `JvmtiThreadState::process_pending_interp_only()` does a sync > on the `JvmtiThreadState_lock` which is not really needed and is being > removed. It is called in a `VTMS` transition and so, can not clash with the > `SetEventNotificationMode` because it sets a `JvmtiVTMSTransitionDisabler`. > > Testing: > - TBD: Mach5 tiers 1-6
Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision: review: remove get_interp_only_mode(), set_interp_only_mode() and clear_interp_only_mode() ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25060/files - new: https://git.openjdk.org/jdk/pull/25060/files/d1ac6b5f..c4d167c4 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25060&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25060&range=00-01 Stats: 12 lines in 4 files changed: 1 ins; 4 del; 7 mod Patch: https://git.openjdk.org/jdk/pull/25060.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25060/head:pull/25060 PR: https://git.openjdk.org/jdk/pull/25060
