RFR: Remove is_owning_locks() call from freeze slow path [v2]

Ron Pressler rpressler at openjdk.java.net
Wed Apr 20 20:41:44 UTC 2022


On Wed, 20 Apr 2022 20:39:02 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

>> The call to is_owning_locks() on each frame when we recurse on freeze can be removed. Instead we can check _held_monitor_count once at the beginning on freeze_internal().
>> 
>> Tested locally by running all tests in test/jdk/java/lang/Thread/virtual/ and jdk/jdk/internal/vm/Continuation/, and in mach5 tiers loom-tier1, loom-tier2 and loom-tier3.
>> 
>> Thanks,
>> Patricio
>
> Patricio Chilano Mateo has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove is_owning_locks calls from CONT_isPinned0

Marked as reviewed by rpressler (Lead).

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

PR: https://git.openjdk.java.net/loom/pull/155


More information about the loom-dev mailing list