RFR: 8253064: monitor list simplifications and getting rid of TSM [v7]

Robbin Ehn rehn at openjdk.java.net
Wed Nov 11 16:24:06 UTC 2020


On Wed, 11 Nov 2020 16:01:15 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Changes from @fisk and @dcubed-ojdk to:
>> 
>> - simplify ObjectMonitor list management
>> - get rid of Type-Stable Memory (TSM)
>> 
>> This change has been tested with Mach5 Tier[1-3],4,5,6,7,8; no new regressions.
>> Aurora Perf runs have also been done (DaCapo-h2, Quick Startup/Footprint,
>> SPECjbb2015-Tuned-G1, SPECjbb2015-Tuned-ParGC, Volano)
>>   - a few minor regressions (<= -0.24%)
>>   - Volano is 6.8% better
>> 
>> Eric C. has also done promotion perf runs on these bits and says "the results look fine".
>
> Daniel D. Daugherty has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains ten commits:
> 
>  - @dholmes-ora, @robehn and @coleenp CR - a few more minor tweaks.
>  - Merge branch 'master' into JDK-8253064
>  - resolve more robehn and coleenp comments.
>  - coleenp CR - refactor common ThreadBlockInVM code block into ObjectSynchronizer::chk_for_block_req().
>  - dholmes-ora - convert inner while loop to do-while loop in unlink_deflated().
>  - Resolve more @dholmes-ora comments with help from @fisk.
>  - Resolve most of dholmes-ora comments.
>  - 8253064.v00.part2
>  - 8253064.v00.part1

Thanks!

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

Marked as reviewed by rehn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/642


More information about the hotspot-dev mailing list