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