review for 7051798: SA-JDI: NPE in Frame.addressOfStackSlot(Frame.java:244)

Tom Rodriguez tom.rodriguez at oracle.com
Wed Aug 31 12:56:38 PDT 2011


http://cr.openjdk.java.net/~never/7051798
1346 lines changed: 585 ins; 637 del; 124 mod; 26143 unchg

7051798: SA-JDI: NPE in Frame.addressOfStackSlot(Frame.java:244)
Reviewed-by:

The SA was never updated to handle ricochet frames so stack walking
was broken when they were encountered.  The X86 stack walking code
hadn't been updated in a while so I sync'ed it the current version of
frame_x86.cpp and eliminated the AMD64 variants of many of these
classes since they should be exactly that same.  All SA related
exceptions in the mlvm test have been fixed.  I had to convert the
PcDesc flags into masks since the SA can't deal with bitfields.

Because of some JDI features being used by the test I had to fix other
unreported SAJDI issues when asking for locals for optimized and
native frames.  I also hit an unreported assertion failure in C1 with
large frames.

Tested with failing mlvm sajdi tests from report plus the regular
tmtools and sajdi test to stress the stack walking.



More information about the hotspot-compiler-dev mailing list