Webrev for 6972759

[David Holmes]

Hi Bill,

I believe this should have gone out to the Serviceability list (cc'ed) 
instead of, or perhaps as well as, the runtime list.

The change looks okay to me in that is does what you described it would. 
With JVMTI the proof-of-the-pudding is always in the testing and I 
assume the various JVMTI test suites have been thoroughly exercised ?

Thanks,
David

On 11/01/2012 8:40 AM, Bill Pittore wrote:
> Webrev for fix for 6972759 is at
> http://cr.openjdk.java.net/~bpittore/6972759/webrev.00/
>
> This bug has to do with single stepping after hitting an exception
> breakpoint. The short story is that JVMTI maintains some thread state
> information that was not updated correctly if a frame was popped off the
> stack. When you pop a frame the exception state should go back to
> _exception_detected=false. Otherwise when MethodExit event is sent at
> some time in the future the 'was_popped_by_exception' flag will be set
> and the agent receiving the event may not re-enable single stepping
> properly. This problem will also affect ForceEarlyReturn code. Hacked up
> a couple of existing nsk/jvmti tests to issue the necessary sequence of
> JVMTI calls/events in order to test original bug and this fix.
>
> bill
>
>