On Tue, 23 Feb 2021 23:52:15 GMT, Daniel D. Daugherty <[email protected]> wrote:
>> 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 @dholmes-ora - Thanks for reviewing CR3. I've made more changes and once my testing looks good, I'll checkpoint and push to update the PR. ------------- PR: https://git.openjdk.java.net/jdk/pull/2535
