[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8241403: JavaThread::get_thread_name() should be ThreadSMR-aware [v8]
From:       Daniel D.Daugherty <dcubed () openjdk ! java ! net>
Date:       2021-02-26 15:43:59
Message-ID: -wPbQ6HSwVprz5LePFP4-mfXtR5IFikM4_oH992H7l8=.0b636c92-c8a9-42ed-8d94-abec11da1467 () github ! com
[Download RAW message or body]

> 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:

  Delete temporary guarantee included for testing.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2535/files
  - new: https://git.openjdk.java.net/jdk/pull/2535/files/4298bb4d..089727c7

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2535&range=07
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2535&range=06-07

  Stats: 2 lines in 1 file changed: 0 ins; 2 del; 0 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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic