Code Review for hotspot

Prepared by:sjohanss on Wed Nov 11 15:31:20 CET 2015
Workspace:/home/sjohanss/repos/jdk9/hs-rt/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk9/hs-rt/hotspot
Compare against version:qparent
Summary of changes: 100 lines changed: 99 ins; 1 del; 0 mod; 6947 unchg
Patch of changes: hotspot.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc/g1/g1CollectedHeap.cpp

rev 9312 : 8139424: SIGSEGV, Problematic frame: # V [libjvm.so+0xd0c0cc] void InstanceKlass::oop_oop_iterate_oop_maps_specialized<true,oopDesc*,MarkAndPushClosure>
Summary: The crash was caused by a faulty eager humongous reclaim. The reason for reclaiming a live obejct was that the call to cleanupHRRS was done after dirtying cards and clearing the remembered sets for the humongous object. This could lead to one or many cards being missed.
Reviewed-by:
8 lines changed: 8 ins; 0 del; 0 mod; 6342 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc/g1/g1RemSet.cpp

rev 9312 : 8139424: SIGSEGV, Problematic frame: # V [libjvm.so+0xd0c0cc] void InstanceKlass::oop_oop_iterate_oop_maps_specialized<true,oopDesc*,MarkAndPushClosure>
Summary: The crash was caused by a faulty eager humongous reclaim. The reason for reclaiming a live obejct was that the call to cleanupHRRS was done after dirtying cards and clearing the remembered sets for the humongous object. This could lead to one or many cards being missed.
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 605 unchg

------ ------ ------ ------ --- New Patch Raw test/gc/g1/TestNoEagerReclaimOfHumongousRegions.java

rev 9312 : 8139424: SIGSEGV, Problematic frame: # V [libjvm.so+0xd0c0cc] void InstanceKlass::oop_oop_iterate_oop_maps_specialized<true,oopDesc*,MarkAndPushClosure>
Summary: The crash was caused by a faulty eager humongous reclaim. The reason for reclaiming a live obejct was that the call to cleanupHRRS was done after dirtying cards and clearing the remembered sets for the humongous object. This could lead to one or many cards being missed.
Reviewed-by:
91 lines changed: 91 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/sjohanss/repos/webrev/webrev.ksh (vers 25.12-hg+openjdk.java.net).