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