RFR [8055338]: (process) Add instrumentation to help diagnose JDK-6573254
Daniel D. Daugherty
daniel.daugherty at oracle.com
Thu Aug 21 15:05:22 UTC 2014
Here's the commit message:
$ cat commit.txt
8055338: (process) Add instrumentation to help diagnose JDK-6573254
Reviewed-by: dcubed, ohair, iklam, dholmes, sspitsyn, sla
I will commit the changeset as 'igerasim'.
Didn't think a summary line was needed.
Dan
On 8/21/14 8:21 AM, Daniel D. Daugherty wrote:
> 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