RFR: 8319896: Remove monitor deflation from final audit [v3]

Stefan Karlsson stefank at openjdk.org
Fri Nov 17 10:49:45 UTC 2023


On Thu, 16 Nov 2023 23:45:51 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update comments
>
> src/hotspot/share/runtime/synchronizer.cpp line 1624:
> 
>> 1622:   size_t unlinked_count = 0;
>> 1623:   size_t deleted_count = 0;
>> 1624:   if (deflated_count > 0) {
> 
> There's a conditional at line 1631 is_monitor_deflation_thread, and an else at 1657 that deflating idle monitors is at a safepoint.  Should there not be this conditional, ie am I reading the description right?

@coleenp I created a PR to clean this up:
https://github.com/openjdk/jdk/pull/16706

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16605#discussion_r1397079342


More information about the hotspot-runtime-dev mailing list