> 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: added comment with clarification ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25060/files - new: https://git.openjdk.org/jdk/pull/25060/files/c4d167c4..b61ff511 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25060&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25060&range=01-02 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 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
