[riscv-port] Integrated: 8277431: riscv: Intrinsify recursive ObjectMonitor locking for C2
Yadong Wang
yadongwang at openjdk.java.net
Tue Nov 23 12:25:39 UTC 2021
On Tue, 23 Nov 2021 07:02:33 GMT, Yadong Wang <yadongwang at openjdk.org> wrote:
> According to https://bugs.openjdk.java.net/browse/JDK-8277180, the C2 fast_lock and fast_unlock intrinsics don't support recursive ObjectMonitor locking. Some workload can significantly benefit from this. This fixes the problem on riscv.
>
> Tier1 tests of hotspot and jdk are passed on unmatched and all JTreg tests are tested on qemu without new failures.
>
> [The original performance regression after JDK-8253064](https://bugs.openjdk.java.net/browse/JDK-8263864) was resolved by this intrinsic.
> The benchmark ran on unmatched:
>
> dacapo-9.12-MR1-bach.jar h2 -s huge -t 1 -n 1
>
> Non-intrisified recursive fast_lock/fast_unlock:
> ===== DaCapo 9.12-MR1 h2 PASSED in 2614652 msec =====
> Intrisified recursive fast_lock/fast_unlock:
> ===== DaCapo 9.12-MR1 h2 PASSED in 2027247 msec =====
This pull request has now been integrated.
Changeset: a319e9ab
Author: Yadong Wang <yadongwang at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL: https://git.openjdk.java.net/riscv-port/commit/a319e9aba1eddb558fd4fa0c6c85ea36fe0303d2
Stats: 21 lines in 1 file changed: 20 ins; 1 del; 0 mod
8277431: riscv: Intrinsify recursive ObjectMonitor locking for C2
Reviewed-by: fyang
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/13
More information about the riscv-port-dev
mailing list