RFR(S): 8222295 more baseline cleanups from Async Monitor Deflation project

[Daniel D. Daugherty]

On 4/23/19 2:39 PM, Karen Kinnear wrote:
> Dan,
>
> Looks good. Thank you for extracting these clean ups - that will make the changes for
> the Async work clearer.

Thanks for the review!

Dan


>
> thanks,
> Karen
>
>> On Apr 23, 2019, at 10:28 AM, Daniel D. Daugherty <daniel.daugherty at oracle.com> wrote:
>>
>> Greetings,
>>
>> I have a (S)mall patch extracted from the Async Monitor Deflation project
>> that is ready for code review.
>>
>> Karen, a number of the changes here are from your code review comments
>> to the parent bug:
>>
>>      JDK-8153224 Monitor deflation prolong safepoints
>>      https://bugs.openjdk.java.net/browse/JDK-8153224
>>
>> The short version of what this patch is about:
>>
>>      More baseline cleanups to the ObjectMonitor subsystem.
>>
>> The details are in the bug report:
>>
>>      JDK-8222295 more baseline cleanups from Async Monitor Deflation project
>>      https://bugs.openjdk.java.net/browse/JDK-8222295
>>
>> Here's the webrev:
>>
>> http://cr.openjdk.java.net/~dcubed/8153224-webrev/4-for-jdk13.8222295/
>>
>> This patch along with the current patch for Async Monitor Deflation
>> project have been through Mach5 tier[1-8] testing.
>>
>> I have been actively using the revised assert()'s and guarantee()'s with
>> additional diagnostic info while debugging my port of the Async Monitor
>> Deflation project code.
>>
>> Thanks, in advance, for any questions, comments or suggestions.
>>
>> Dan