Integrated: 8338745: Intrinsify Continuation.pin() and Continuation.unpin()
Markus Grönlund
mgronlun at openjdk.org
Fri Aug 23 09:29:08 UTC 2024
On Wed, 21 Aug 2024 20:11:05 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:
> Greetings,
>
> Please help review this change set that implements C2 intrinsics for jdk.internal.vm.Continuation.pin() and jdk.internal.vm.Continuation.unpin().
>
> This work is a consequence of [JDK-8338417](https://bugs.openjdk.org/browse/JDK-8338417), which required us to introduce explicit pin constructs for Virtual threads in a relatively performance-sensitive path.
>
> Testing: jdk_jfr, loom testing
>
> Comment: I changed the type of the ContinuationEntry::_pin_count field from uint to uin32_t to make the size explicit and to access it uniformly from the intrinsic code using T_INT.
>
> Thanks
> Markus
This pull request has now been integrated.
Changeset: fead3cf5
Author: Markus Grönlund <mgronlun at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/fead3cf54130e3ab10f94a94dfbd382e4cb1e597
Stats: 109 lines in 9 files changed: 105 ins; 0 del; 4 mod
8338745: Intrinsify Continuation.pin() and Continuation.unpin()
Reviewed-by: kvn
-------------
PR: https://git.openjdk.org/jdk/pull/20664
More information about the core-libs-dev
mailing list