RFR: 8352251: Implement Cooperative JFR Sampling [v16]

[Markus Grönlund]

On Mon, 5 May 2025 19:48:35 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> Isn't this problem already solved by using the `_last_sender_Java_fp`? Why do we need both, the frame pop before the safepoint check and the _last_sender_Java_fp trick?

Because the sampler fetches the CPU context for threads running in state "_thread_in_Java". Without it, you can sample after the safepoint poll test is issued, but before the frame is popped. That sampled frame will represent something now being removed, and if the sender issues another call, also overwritten.

It may help to think about how the JIT methods works - they pop their frames before issuing the method return safepoint poll test.

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

PR Comment: https://git.openjdk.org/jdk/pull/24296#issuecomment-2852187118