Please review fix for 7196681: NPG: Some JSR 292 tests crash in Windows exception handler

Christian Thalinger christian.thalinger at oracle.com
Tue Sep 11 17:21:13 PDT 2012


On Sep 11, 2012, at 4:38 PM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:

> 
> How is this?  I added some TODO's and will file an RFE to implement this.   I tested -XX:+VerifyOops with sparc and x86.
> 
> http://oklahoma.us.oracle.com/~cphillim/webrev/7196681_2/

src/cpu/x86/vm/cppInterpreter_x86.cpp:

In this file you use verify_method instead of verify_method_ptr.

-- Chris

> 
> There were some verify_oop's removed in the permgen project that would be good to add back.  I will add that to the RFE.
> 
> Thanks,
> Coleen
> 
> On 9/11/2012 7:27 PM, Christian Thalinger wrote:
>> On Sep 11, 2012, at 1:51 PM, Coleen Phillimore<coleen.phillimore at oracle.com>  wrote:
>> 
>>> Summary: There was a rogue os::breakpoint() call in log_dependency left over from the jsr292 merge.  Also removed verify_oop() calls for metadata.
>>> 
>>> Tested CallSiteTest on windows 64 where this os::breakpoint() actually sends a breakpoint signal.
>> Oops.  It must be a left-over from the merge.  Otherwise this looks good and I agree that empty verify methods would be nice.
>> 
>> -- Chris
>> 
>>> open webrev at http://cr.openjdk.java.net/~coleenp/7196681/
>>> bug link at http://bugs.sun.com/view_bug.do?bug_id=7196681
>>> 
>>> Thanks,
>>> Coleen
>>> 
>>> 



More information about the hotspot-dev mailing list