RFR: 8048353: jstack -l crashes VM when a Java mirror for a primitive type is locked

Daniel D. Daugherty daniel.daugherty at oracle.com
Mon Jul 20 14:39:46 UTC 2015


Vladimir,

I see reviews from me, Coleen, and David H. That's three (R)eviewers.

Are you waiting for a review from someone in Serviceability?

Dan


On 7/20/15 8:20 AM, Vladimir Kempik wrote:
> hello
>
> Can I have one more review for this please?
>
> jstack/tmtools tested passed fine.
>
> Thanks, Vladimir.
>
> On 15.07.2015 17:31, Vladimir Kempik wrote:
>> Hello
>>
>> Thanks for looking into this.
>>
>> Where can I find jstack/tmtools tests to run them ?
>>
>> Thanks, Vladimir.
>>
>> On 14.07.2015 19:21, Daniel D. Daugherty wrote:
>>> > Webrev: http://cr.openjdk.java.net/~vkempik/8048353/webrev.00/
>>>
>>> src/share/vm/classfile/javaClasses.cpp
>>>     No comments. (Thanks for including the "<null>" case.)
>>>
>>> src/share/vm/classfile/javaClasses.hpp
>>>     No comments.
>>>
>>> src/share/vm/runtime/vframe.cpp
>>>     No comments.
>>>
>>> Thumbs up (modulo running the proper tests).
>>>
>>> Dan
>>>
>>>
>>> On 7/14/15 9:56 AM, Daniel D. Daugherty wrote:
>>>> Adding serviceability-dev at ... since jstack belongs to the
>>>> Serviceability team...
>>>>
>>>> The jstack and/or tmtools tests should also be run...
>>>>
>>>> Dan
>>>>
>>>>
>>>> On 7/14/15 9:51 AM, Vladimir Kempik wrote:
>>>>> Hello,
>>>>>
>>>>> Please review the patch to fix 8048353.
>>>>> Customer has an issue with this bug when running jstack on jdk7.
>>>>> He tested FVB and cofirmed it has fixed the issue.
>>>>> Before pushing jdk7 backport I need to get the fix to 9 and 8.
>>>>>
>>>>> Webrev: http://cr.openjdk.java.net/~vkempik/8048353/webrev.00/
>>>>> JBS bug: https://bugs.openjdk.java.net/browse/JDK-8048353
>>>>>
>>>>> testing: JPRT.
>>>>>
>>>>> Thanks, Vladimir.
>>>>
>>>
>>
>
>
>



More information about the hotspot-runtime-dev mailing list