RFR [8055338]: (process) Add instrumentation to help diagnose JDK-6573254

Daniel D. Daugherty daniel.daugherty at oracle.com
Thu Aug 21 14:21:04 UTC 2014


The correct repository would be RT_Baseline. What testing has been
run on your fix? I know you've done a control build so that covers
JPRT testing.

I'll prep a repo for sponsoring the push, but I'll hold off pushing
until I know what testing has been done.

Dan


On 8/21/14 4:57 AM, Ivan Gerasimov wrote:
> Thanks for review!
>
> Could someone please sponsor it for me?
> I'm not sure what the correct repository is.
>
> Sincerely yours,
> Ivan
>
> On 21.08.2014 11:46, serguei.spitsyn at oracle.com wrote:
>> +1
>>
>> On 8/20/14 10:57 PM, Staffan Larsen wrote:
>>> Looks good to me. Let’s see what it uncovers.
>>>
>>> /Staffan
>>>
>>> On 20 aug 2014, at 21:36, Ivan Gerasimov <ivan.gerasimov at oracle.com> 
>>> wrote:
>>>
>>>> Hello everyone!
>>>>
>>>> Here's the third version of the webrev:
>>>> http://cr.openjdk.java.net/~igerasim/8055338/2/webrev/
>>>>
>>>> The control build of the previous one was causing a lot of test 
>>>> failures.
>>>> This one seems to be innocent enough: no new test failures so far.
>>>>
>>>> Additionally, this version keeps the timing around the thread exit 
>>>> close to original, which might be important if we deal with a race.
>>>>
>>>> Sincerely yours,
>>>> Ivan
>>>>
>>
>>
>>
>



More information about the hotspot-runtime-dev mailing list