Integrated: 8299375: [PPC64] GetStackTraceSuspendedStressTest tries to deoptimize frame with invalid fp

Richard Reingruber rrich at openjdk.org
Fri Mar 17 08:48:32 UTC 2023


On Thu, 9 Mar 2023 10:46:05 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

> Mark a frame as not fully initialized when copying it from a continuation StackChunk to the stack until the callers_sp (aka back link) is set.
> 
> This avoids the assertion given in the bug report when the copied frame is deoptimized before it is fully initialized.
> IMHO the deoptimization at that point is a little questionable but it actually only changes the pc of the frame which can be done.
> Note that the frame can get extended later (and metadata can get overridden) but [there is code that handles this](https://github.com/openjdk/jdk/blob/34a92466a615415b76c8cb6010ff7e6e1a1d63b4/src/hotspot/share/runtime/continuationFreezeThaw.cpp#L2108-L2110).
> 
> Testing: jdk_loom. The fix passed our CI testing. This includes most JCK and JTREG tiers 1-4, also in Xcomp mode, on the standard platforms and also on ppc64le.

This pull request has now been integrated.

Changeset: 9d518c52
Author:    Richard Reingruber <rrich at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9d518c528b11953b556aa7585fc69ff9c9a22435
Stats:     18 lines in 3 files changed: 14 ins; 0 del; 4 mod

8299375: [PPC64] GetStackTraceSuspendedStressTest tries to deoptimize frame with invalid fp

Reviewed-by: mdoerr

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

PR: https://git.openjdk.org/jdk/pull/12941


More information about the hotspot-compiler-dev mailing list