[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