On Thu, 25 Feb 2021 23:14:01 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 incrementally with one > additional commit since the last revision: > > Address dholmes-ora CR4 comments. Hi Dan, The update looks okay, but I have one question. So, the fix in jvmtiTrace.cpp is not needed anymore? Thanks, Serguei ------------- PR: https://git.openjdk.java.net/jdk/pull/2535
