RFR: 8283849: AsyncGetCallTrace may crash JVM on guarantee [v2]

[Jaroslav Bachorik]

On Fri, 6 May 2022 01:30:35 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Ok, moved it a bit around - now the bool field is after an int field which should make things slightly better, I guess.
>
> At the start of JavaThread we have:
> 
>  private:
>   bool           _on_thread_list;                // Is set when this JavaThread is added to the Threads list
>   OopHandle      _threadObj;                     // The Java level thread object
> 
> so adding it next to the existing bool seems good. The accessors don't have to be defined at the same place.

Ok. Done.
This requires an extra check and a static cast to `JavaThread` in `CodeCache::find_blob()` but it should not be that bad.

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

PR: https://git.openjdk.java.net/jdk/pull/8549