> A minor fix to add a new function: > > bool Thread::is_JavaThread_protected(const JavaThread* p) > > that returns true when the target JavaThread* is protected and false > otherwise. Update JavaThread::get_thread_name() to create a > ThreadsListHandle and use the new is_JavaThread_protected(). Also > update JvmtiTrace::safe_get_thread_name() to use the new > is_JavaThread_protected(). > > This fix is tested via a Mach5 Tier[1-8] run.
Daniel D. Daugherty 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 six additional commits since the last revision: - Merge branch 'master' into JDK-8241403 - Address dholmes-ora CR2 comments. - Address dholmes-ora CR1 comments. - Merge branch 'master' into JDK-8241403 - Address coleenp CR0 comments. - 8241403: JavaThread::get_thread_name() should be ThreadSMR-aware ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2535/files - new: https://git.openjdk.java.net/jdk/pull/2535/files/cbb71a20..19fe9538 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2535&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2535&range=03-04 Stats: 13293 lines in 368 files changed: 9145 ins; 2347 del; 1801 mod Patch: https://git.openjdk.java.net/jdk/pull/2535.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2535/head:pull/2535 PR: https://git.openjdk.java.net/jdk/pull/2535
