On Mon, 31 Aug 2026 13:29:02 GMT, Per Minborg <[email protected]> wrote:

>> The call in the `finally` clause needs to be `Continuation​.unpin()`.
>
> Adding pinning would have a relatively large performance impact:
> 
> No pinning:
> 
> Benchmark                                   (size)  Mode  Cnt   Score   Error 
>  Units
> AllocTest.OfVirtual.alloc_confined               5  avgt   30   1.401 ± 0.016 
>  ns/op
> AllocTest.OfVirtual.alloc_confined_no_pool       5  avgt   30  15.382 ± 0.106 
>  ns/op
> AllocTest.alloc_confined                         5  avgt   30   1.060 ± 0.026 
>  ns/op
> AllocTest.alloc_confined_no_pool                 5  avgt   30  15.335 ± 0.140 
>  ns/op
> 
> 
> Pinning
> 
> Benchmark                                   (size)  Mode  Cnt   Score   Error 
>  Units
> AllocTest.OfVirtual.alloc_confined               5  avgt   30   3.258 ± 0.312 
>  ns/op
> AllocTest.OfVirtual.alloc_confined_no_pool       5  avgt   30  15.607 ± 0.189 
>  ns/op
> AllocTest.alloc_confined                         5  avgt   30   1.076 ± 0.028 
>  ns/op
> AllocTest.alloc_confined_no_pool                 5  avgt   30  15.552 ± 0.097 
>  ns/op

I've added pinning

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31365#discussion_r3894910748

Reply via email to