RFR: 8319935: Ensure only one JvmtiThreadState is create for one JavaThread associated with attached native thread [v4]

Jiangli Zhou jiangli at openjdk.org
Wed Nov 22 22:18:23 UTC 2023


> Please review JvmtiThreadState::state_for_while_locked change to handle the state->get_thread_oop() null case. Please see https://bugs.openjdk.org/browse/JDK-8319935 for details.

Jiangli Zhou has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:

 - Merge branch 'master' into JDK-8319935
 - Add a check for a thread is_attaching_via_jni, based on David Holmes' comment.
 - Don't try to setup_jvmti_thread_state for obj allocation sampling if the current thread is attaching from native and is allocating the thread oop. That's to make sure we don't create a 'partial' JvmtiThreadState.
 - 8319935: Ensure only one JvmtiThreadState is create for one JavaThread associated with attached native thread

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16642/files
  - new: https://git.openjdk.org/jdk/pull/16642/files/7c0214e2..de7fac6d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16642&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16642&range=02-03

  Stats: 30285 lines in 831 files changed: 17825 ins; 7190 del; 5270 mod
  Patch: https://git.openjdk.org/jdk/pull/16642.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16642/head:pull/16642

PR: https://git.openjdk.org/jdk/pull/16642


More information about the serviceability-dev mailing list