On 11/1/2013 9:16 AM, Erik Helin wrote:
Hi Coleen,
this looks great, just one minor nit: could you please update the
comment starting at jvmtiImpl.cpp:242 (which explains why using
allow_unhandled_oop is correct) to use _class_holder instead of
_class_loader?
I fixed it and the : in JvmtiBreakpoint:s.
Thanks for the code review.
Coleen
Thanks,
Erik
On 2013-10-30, Coleen Phillimore wrote:
8026948: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint might
not work with anonymous classes
Summary: Walk methods in breakpoints for marking on stack so they
aren't deallocated by redefine classes. Use class_holder rather
than class_loader to keep GC from reclaiming class owning the
method.
This is two bug fixes to the same code.
open webrev at http://cr.openjdk.java.net/~coleenp/8026946_8026948/
bug link https://bugs.openjdk.java.net/browse/JDK-8026946
https://bugs.openjdk.java.net/browse/JDK-8026948
Tested with vm.quick.testlist - and verified the jdi tests eg
nsk/jdi/BScenarios/hotswap/tc01x001 test this path.
Thanks,
Coleen