RFR(S): 8217658 baseline_cleanups from Async Monitor Deflation project

[Robbin Ehn]

Hi Dan,

Looks great, thanks!

/Robbin

On 2019-01-26 04:46, Daniel D. Daugherty wrote:
> David,
> 
> Thanks for the re-review. Still hoping for one more reviewer...
> 
> Dan
> 
> 
> On 1/25/19 9:46 PM, David Holmes wrote:
>> Thanks Dan, incremental changes look good.
>>
>> David
>>
>> On 26/01/2019 3:38 am, Daniel D. Daugherty wrote:
>>> Greetings,
>>>
>>> I've updated the patch to include the two additional cleanups from David H.
>>> I've also rebased the patch to the jdk/jdk repo at the 'jdk-13+5' tag;
>>> there were no merge surprises.
>>>
>>> Here's an incremental webrev:
>>>
>>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/1-for-jdk13.inc/
>>>
>>> I've also generated a full webrev since I still need one more reviewer for this:
>>>
>>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/1-for-jdk13.full/
>>>
>>> I've also updated the bug to talk about the two additional cleanups.
>>>
>>> Thanks, in advance, for any questions, comments or suggestions.
>>>
>>> Dan
>>>
>>>
>>> On 1/23/19 3:07 PM, Daniel D. Daugherty wrote:
>>>> Greetings,
>>>>
>>>> I have a (S)mall patch extracted from the Async Monitor Deflation
>>>> project that is ready for code review.
>>>>
>>>> The short version of what this patch is about:
>>>>
>>>>     This sub-task captures updates to the baseline code that are not
>>>>     directly related to Async Monitor Deflation.
>>>>
>>>> The details are in the bug report:
>>>>
>>>>     JDK-8217658 baseline_cleanups from Async Monitor Deflation project
>>>>     https://bugs.openjdk.java.net/browse/JDK-8217658
>>>>
>>>> Here's the webrev:
>>>>
>>>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/0-for-jdk13/
>>>>
>>>> This patch along with the other two patches for Async Monitor Deflation
>>>> project have been through Mach5 builds-tier1,hs-tier1,jdk-tier1,hs-tier2,hs-tier3
>>>> testing and I'm currently running my stress testing kits on my Linux-X64
>>>> and Solaris-X64 servers.
>>>>
>>>> Thanks, in advance, for any questions, comments or suggestions.
>>>>
>>>> Dan
>>>>
>>>>
>>>
>