[jdk19] RFR: 8289091: move oop safety check from SharedRuntime::get_java_tid() to JavaThread::threadObj() [v4]

Daniel D. Daugherty dcubed at openjdk.org
Fri Jul 1 01:01:04 UTC 2022


> A trivial move of the oop safety check from SharedRuntime::get_java_tid() to
> JavaThread::threadObj(). Also made adjustments to the threadObj() calls in
> JavaThread::print_on_error() and JavaThread::get_thread_name_string() so
> that we don't get secondary crashes when a JavaThread crashes after it has
> detached the GC barrier.
> 
> Tested with Mach5 Tier[1-7]. A Mach5 Tier8 will be started this weekend.

Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:

  dholmes CR - use Thread::current_or_null_safe() instead of Thread::current() since threadObj() can be called by a signal handler.

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

Changes:
  - all: https://git.openjdk.org/jdk19/pull/69/files
  - new: https://git.openjdk.org/jdk19/pull/69/files/e2f57577..088a3bdc

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

  Stats: 6 lines in 1 file changed: 3 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jdk19/pull/69.diff
  Fetch: git fetch https://git.openjdk.org/jdk19 pull/69/head:pull/69

PR: https://git.openjdk.org/jdk19/pull/69


More information about the hotspot-runtime-dev mailing list