RFR: 8369238: Allow virtual thread preemption on some common class initialization paths [v6]

Patricio Chilano Mateo pchilanomate at openjdk.org
Mon Oct 27 19:29:07 UTC 2025


On Fri, 24 Oct 2025 14:00:33 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I added `patch_pd_unused` in 8359222 which should have been `patch_unused_pd`. :)
>
> I like AnchorMark.  There are other places that clear_anchor in this code.  Can they use AnchorMark?

Yes, I added it in `ThawBase::throw_interrupted_exception()` and in `thaw_fast()`.

>> A full `HandleMark` works too. It’s just that there is already a `HandleMark` in the callstack (from the original upcall to Java from the carrier thread), so we can use a `HandleMarkCleaner` here.
>
> The main place we have HandleMarkCleaners are in the JRT_ENTRY.  Can you change the comment to:
> // Returning to java so cleanup all handles including so._conth Handle
> 
> or something like that.

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27802#discussion_r2466823846
PR Review Comment: https://git.openjdk.org/jdk/pull/27802#discussion_r2466820533


More information about the core-libs-dev mailing list