On Wed, 14 Jun 2023 01:02:09 GMT, Alex Menkov <amen...@openjdk.org> wrote:

> If virtual thread has frames in stackChunks, need to apply load barriers 
> before processing nmethods (javaVFrame::locals() and 
> javaVFrame::expressions() do it internally)
> 
> Testing: tier1-tier5;
> 400 runs of VThreadStackRefTest.java test on linux-x64 and linux-x64-debug 
> with "-XX:+UseZGC -Xcomp -XX:-TieredCompilation"

Looks good. It isn't obvious that we need to follow nmethod oops that are not 
reachable through the current JVM state, but if we do, that's a good fix.

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

Marked as reviewed by eosterlund (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14460#pullrequestreview-1562370959

Reply via email to