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

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Sep 11 16:57:39 PDT 2012


This looks good.

Vladimir

Coleen Phillimore 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/
> 
> 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