RFR (trivial): 8229808: javaVFrame::print_lock_info_on fails to disable extra printing

Daniel D. Daugherty daniel.daugherty at oracle.com
Fri Aug 16 12:55:47 UTC 2019


On 8/16/19 8:49 AM, David Holmes wrote:
> Thanks Harold!
>
> I had to rebase now that Stefan pushed the markOop->markWord changes.
>
> webrev updated in place.
>
> David
>
> On 16/08/2019 9:51 pm, Harold Seigel wrote:
>> Looks good and trivial.
>>
>> Thanks, Harold
>>
>> On 8/16/2019 2:53 AM, David Holmes wrote:
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8229808
>>> webrev: http://cr.openjdk.java.net/~dholmes/8229808/webrev/

src/hotspot/share/runtime/vframe.cpp
     No comments.

Thumbs up.

Dan


>>>
>>> Trivial code cleanup that Stefan noticed. The original code with 
>>> disabled printing was added in JDK-8069412 using the Verbose flag 
>>> combined with a non-NULL mark. The use of the verbose flag was 
>>> replaced by ObjectMonitor::Knob_verbose in JDK-8130448. Then the 
>>> printing code was removed by JDK-8210848 "Obsolete SyncKnobs" - at 
>>> which point the comment about "disable extra printing below" should 
>>> also have been removed, and the mark variable returned to being 
>>> local to the if-clause. There's also an earlier comment relating to 
>>> lockbits that refers to the printing code that was deleted, so its 
>>> gone too.
>>>
>>> Thanks,
>>> David
>>>
>>>
>>>



More information about the hotspot-runtime-dev mailing list