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

[David Holmes]

I'll add a third Review :) My only query is whether just type2name is 
the right choice here - I'm not sure what the actual output will display 
in context?

Thanks,
David

On 15/07/2015 2:21 AM, 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.
>>
>