On Mon, 22 Sep 2025 22:37:30 GMT, Patricio Chilano Mateo 
<[email protected]> wrote:

> Reconstructing a synthetic compiled frame when processing a sample request at 
> a poll on return doesn’t work for frames that need stack repair. The stack 
> walking code will read the frame size from the stack, but the memory has 
> already been overwritten by the SafepointBlob frame. See comments in JBS 
> issue for full details.
> 
> This patch skips the attempt to create this synthetic frame in case the frame 
> needs stack repair. This fixes the crash but doesn't handle the safepoint 
> bias issue. A more permanent solution will be required for that.
> 
> I tested the patch along the fix for 8368002 and verified that Jetty.java 
> doesn’t crash anymore.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: 52e1a19d
Author:    Patricio Chilano Mateo <[email protected]>
Committer: Chen Liang <[email protected]>
URL:       
https://git.openjdk.org/valhalla/commit/52e1a19d117110b0641d28540d8562af6c7134b1
Stats:     3 lines in 1 file changed: 2 ins; 0 del; 1 mod

8368099: [lworld] Crash in frame::sender_for_compiled_frame while processing 
JFR sampling request

Reviewed-by: mgronlun

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

PR: https://git.openjdk.org/valhalla/pull/1610

Reply via email to