RFR: 8320304: Refactor and simplify monitor deflation functions [v7]

[Stefan Karlsson]

On Thu, 30 Nov 2023 10:35:41 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> The recent rewrites to remove safepointed monitor deflation allows us to simplify the monitor deflation code a bit. The code is now guaranteed to be called from the MonitorDeflationThread, which is a JavaThread, and never from the VMThread.
>> 
>> I would like to propose two changes to the code:
>> 1) Remove all code that supports monitor deflation from other threads than the MonitorDeflationThread
>> 2) Extract out the logging code so that it is easier to see the structure of the monitor deflation code
>> 
>> I have run minimal manual of this patch. I'd like to get some feedback on the proposal before I proceed to run more in-depth testing.
>
> Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains ten commits:
> 
>  - Small indentation correction
>  - Merge remote-tracking branch 'upstream/master' into 8320304_cleanup_of_monitor_deflation
>  - Small whitespace cleanups
>  - Merge remote-tracking branch 'upstream/master' into 8320304_cleanup_of_monitor_deflation
>  - Merge remote-tracking branch 'upstream/master' into 8320304_cleanup_of_monitor_deflation
>  - David's review comments
>  - Dan's review comments
>  - Make classes StackObjs
>  - 8320304: Refactor and simplify monitor deflation functions

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/16706#issuecomment-1833699911