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

Coleen Phillimore coleenp at openjdk.java.net
Wed Apr 20 21:46:48 UTC 2022


On Wed, 20 Apr 2022 21:29:55 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:
> 
>   add parenthesis to silence clang warning

Marked as reviewed by coleenp (Committer).

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

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


More information about the loom-dev mailing list