review for 7092278: "jmap -finalizerinfo" throws "sun.jvm.hotspot.utilities.AssertionFailure: invalid cp index 0 137"

Tom Rodriguez tom.rodriguez at
Wed Sep 28 13:56:41 PDT 2011

It was pointed out to me that the vmSymbols changes were incomplete.  There aren't currently exercised, so I added a little logic to iterate them when the InstanceKlass is constructed.  This exposed another issue where getAllFieldsCount() wasn't being scaled properly.  It all works now.


On Sep 28, 2011, at 9:27 AM, Vladimir Kozlov wrote:

> Good.
> Vladimir
> Tom Rodriguez wrote:
>> 131 lines changed: 101 ins; 9 del; 21 mod; 5998 unchg
>> 7092278: "jmap -finalizerinfo" throws "sun.jvm.hotspot.utilities.AssertionFailure: invalid cp index 0 137"
>> Reviewed-by:
>> There's was a mismatch between using scaled and unscaled indexes to
>> refer to fields in the SA because I hadn't completly hidden access to
>> the fields.  I also added proper support for reading the internal
>> field names though that's currently not need for these tests.  Tested
>> with tmtools suite, including the failing test.

More information about the hotspot-compiler-dev mailing list