Hi all, could I have a review for the following change? The recursive lock count for an object is not correct, in cases, where a monitor is inflated after recursive lightweight locks. In this case, the recursion count is taken from the heavyweight monitor, represented by the class ObjectMonitor. ObjectMonitor::_recursions is the number of times ObjectMonitor::enter() was called to acquire the lock minus 1. This counter does not include the recursions of lightweight locks, that happen before inflating the monitor and is not equal to the recursion count from a Java source level point of view.
I added a test to the webrev to reproduce the problem. The suggested fix is to call count_locked_objects, even if there's a heavyweight monitor and get the recursion count by iterating the vframes. Bug: https://bugs.openjdk.java.net/browse/JDK-8036666 Webrev: http://cr.openjdk.java.net/~asiebenborn/8036666/webrev/ Thanks, Axel
