RFR(S): 8217658 baseline_cleanups from Async Monitor Deflation project
Daniel D. Daugherty
daniel.daugherty at oracle.com
Mon Jan 28 21:34:16 UTC 2019
Robbin, thanks for the review!
Dan
On 1/28/19 3:38 PM, Robbin Ehn wrote:
> 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
>>>>>
>>>>>
>>>>
>>
More information about the hotspot-runtime-dev
mailing list