[jdk22u] RFR: 8310513: [s390x] Intrinsify recursive ObjectMonitor locking [v2]

Lutz Schmidt lucy at openjdk.org
Mon Apr 29 07:46:09 UTC 2024


On Wed, 10 Apr 2024 04:16:33 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> Hi all,
>> 
>> This pull request contains a backport of commit [47df1459](https://github.com/openjdk/jdk/commit/47df14590c003ccb1607ec0edfe999fcf2aebd86) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository.
>> 
>> The commit being backported was authored by Amit Kumar on 10 Apr 2024 and was reviewed by Lutz Schmidt and Martin Doerr.
>> 
>> Thanks!
>> 
>> 
>> Tested `{tier1} X {fastdebug}` with `-XX:-TieredCompilation` and do not see any new failure appearing. 
>> 
>> Benchmarking Result: 
>> 
>> 
>> ./build/linux-s390x-server-release/jdk/bin/java -Xms4g -Xmx4g -jar dacapo-9.12-MR1-bach.jar h2 -s huge -t 1 -n 1
>> 
>> With Patch: 
>> ===== DaCapo 9.12-MR1 h2 PASSED in 148008 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 152460 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 151682 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 146845 msec =====
>> 
>> 
>> Without Patch: 
>> ===== DaCapo 9.12-MR1 h2 PASSED in 192967 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 206936 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 191804 msec =====
>> ===== DaCapo 9.12-MR1 h2 PASSED in 196735 msec =====
>
> Amit Kumar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   updates copyright header

Looks good.
Nice performance improvement.

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

Marked as reviewed by lucy (Reviewer).

PR Review: https://git.openjdk.org/jdk22u/pull/134#pullrequestreview-2027762873


More information about the jdk-updates-dev mailing list