RFR: 8369609: Continuations preempt_epilog is missing a call to invalidate_jvmti_stack [v3]
Serguei Spitsyn
sspitsyn at openjdk.org
Wed Oct 22 22:43:17 UTC 2025
On Wed, 22 Oct 2025 20:27:26 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> I see. What do you think about adding `invalidate_jvmti_stack` in `jvmti_yield_cleanup` if `!cont.entry()->is_virtual_thread()` is true to address that case?
>
> Good idea! It'd be nice to move the `invalidate_jvmti_stack()` calls out of the continuation code. I'll test it and let you know the results.
Strangely, the test `serviceability/jvmti/vthread/ContStackDepthTest` is failing with the `assert(_cur_stack_depth == num_frames)`. Obviously, some of the code paths is missed to call `invalidate_jvmti_stack()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27878#discussion_r2453509024
More information about the hotspot-runtime-dev
mailing list