src/share/vm/gc/g1/heapRegion.cpp

Print this page

        

*** 597,608 **** st->print("|AC%3u", allocation_context()); st->print_cr("|TAMS " PTR_FORMAT ", " PTR_FORMAT "|", p2i(prev_top_at_mark_start()), p2i(next_top_at_mark_start())); } ! class VerifyLiveClosure: public OopClosure { ! private: G1CollectedHeap* _g1h; CardTableModRefBS* _bs; oop _containing_obj; bool _failures; int _n_failures; --- 597,608 ---- st->print("|AC%3u", allocation_context()); st->print_cr("|TAMS " PTR_FORMAT ", " PTR_FORMAT "|", p2i(prev_top_at_mark_start()), p2i(next_top_at_mark_start())); } ! class G1VerificationClosure : public OopClosure { ! protected: G1CollectedHeap* _g1h; CardTableModRefBS* _bs; oop _containing_obj; bool _failures; int _n_failures;
*** 609,648 **** VerifyOption _vo; public: // _vo == UsePrevMarking -> use "prev" marking information, // _vo == UseNextMarking -> use "next" marking information, // _vo == UseMarkWord -> use mark word from object header. ! VerifyLiveClosure(G1CollectedHeap* g1h, VerifyOption vo) : _g1h(g1h), _bs(barrier_set_cast<CardTableModRefBS>(g1h->barrier_set())), ! _containing_obj(NULL), _failures(false), _n_failures(0), _vo(vo) ! { } void set_containing_obj(oop obj) { _containing_obj = obj; } bool failures() { return _failures; } int n_failures() { return _n_failures; } - virtual void do_oop(narrowOop* p) { do_oop_work(p); } - virtual void do_oop( oop* p) { do_oop_work(p); } - void print_object(outputStream* out, oop obj) { #ifdef PRODUCT Klass* k = obj->klass(); const char* class_name = k->external_name(); out->print_cr("class name %s", class_name); #else // PRODUCT obj->print_on(out); #endif // PRODUCT } template <class T> void do_oop_work(T* p) { assert(_containing_obj != NULL, "Precondition"); assert(!_g1h->is_obj_dead_cond(_containing_obj, _vo), "Precondition"); T heap_oop = oopDesc::load_heap_oop(p); LogHandle(gc, verify) log; if (!oopDesc::is_null(heap_oop)) { oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); bool failed = false; --- 609,657 ---- VerifyOption _vo; public: // _vo == UsePrevMarking -> use "prev" marking information, // _vo == UseNextMarking -> use "next" marking information, // _vo == UseMarkWord -> use mark word from object header. ! G1VerificationClosure(G1CollectedHeap* g1h, VerifyOption vo) : _g1h(g1h), _bs(barrier_set_cast<CardTableModRefBS>(g1h->barrier_set())), ! _containing_obj(NULL), _failures(false), _n_failures(0), _vo(vo) { ! } void set_containing_obj(oop obj) { _containing_obj = obj; } bool failures() { return _failures; } int n_failures() { return _n_failures; } void print_object(outputStream* out, oop obj) { #ifdef PRODUCT Klass* k = obj->klass(); const char* class_name = k->external_name(); out->print_cr("class name %s", class_name); #else // PRODUCT obj->print_on(out); #endif // PRODUCT } + }; + class VerifyLiveClosure : public G1VerificationClosure { + public: + VerifyLiveClosure(G1CollectedHeap* g1h, VerifyOption vo) : G1VerificationClosure(g1h, vo) {} + virtual void do_oop(narrowOop* p) { do_oop_work(p); } + virtual void do_oop(oop* p) { do_oop_work(p); } + template <class T> void do_oop_work(T* p) { assert(_containing_obj != NULL, "Precondition"); assert(!_g1h->is_obj_dead_cond(_containing_obj, _vo), "Precondition"); + verify_liveness(p); + } + + template <class T> + void verify_liveness(T* p) { T heap_oop = oopDesc::load_heap_oop(p); LogHandle(gc, verify) log; if (!oopDesc::is_null(heap_oop)) { oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); bool failed = false;
*** 673,684 **** log.info("----------"); _failures = true; failed = true; _n_failures++; } ! if (!_g1h->collector_state()->full_collection() || G1VerifyRSetsDuringFullGC) { HeapRegion* from = _g1h->heap_region_containing((HeapWord*)p); HeapRegion* to = _g1h->heap_region_containing(obj); if (from != NULL && to != NULL && from != to && !to->is_pinned()) { --- 682,716 ---- log.info("----------"); _failures = true; failed = true; _n_failures++; } + } + } + }; ! class VerifyRemSetClosure : public G1VerificationClosure { ! public: ! VerifyRemSetClosure(G1CollectedHeap* g1h, VerifyOption vo) : G1VerificationClosure(g1h, vo) {} ! virtual void do_oop(narrowOop* p) { do_oop_work(p); } ! virtual void do_oop(oop* p) { do_oop_work(p); } ! ! template <class T> ! void do_oop_work(T* p) { ! assert(_containing_obj != NULL, "Precondition"); ! assert(!_g1h->is_obj_dead_cond(_containing_obj, _vo), ! "Precondition"); ! verify_remembered_set(p); ! } ! ! template <class T> ! void verify_remembered_set(T* p) { ! T heap_oop = oopDesc::load_heap_oop(p); ! LogHandle(gc, verify) log; ! if (!oopDesc::is_null(heap_oop)) { ! oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); ! bool failed = false; HeapRegion* from = _g1h->heap_region_containing((HeapWord*)p); HeapRegion* to = _g1h->heap_region_containing(obj); if (from != NULL && to != NULL && from != to && !to->is_pinned()) {
*** 712,722 **** if (!failed) _n_failures++; } } } } - } }; // This really ought to be commoned up into OffsetTableContigSpace somehow. // We would need a mechanism to make that code skip dead objects. --- 744,753 ----
*** 725,734 **** --- 756,766 ---- G1CollectedHeap* g1 = G1CollectedHeap::heap(); *failures = false; HeapWord* p = bottom(); HeapWord* prev_p = NULL; VerifyLiveClosure vl_cl(g1, vo); + VerifyRemSetClosure vr_cl(g1, vo); bool is_region_humongous = is_humongous(); size_t object_num = 0; while (p < top()) { oop obj = oop(p); size_t obj_size = block_size(p);
*** 750,770 **** "not a klass", p2i(klass), p2i(obj)); *failures = true; return; } else { vl_cl.set_containing_obj(obj); obj->oop_iterate_no_header(&vl_cl); if (vl_cl.failures()) { *failures = true; } if (G1MaxVerifyFailures >= 0 && vl_cl.n_failures() >= G1MaxVerifyFailures) { return; } } } else { ! log_info(gc, verify)(PTR_FORMAT " no an oop", p2i(obj)); *failures = true; return; } } prev_p = p; --- 782,818 ---- "not a klass", p2i(klass), p2i(obj)); *failures = true; return; } else { vl_cl.set_containing_obj(obj); + if (!g1->collector_state()->full_collection() || G1VerifyRSetsDuringFullGC) { + // verify liveness and rem_set + vr_cl.set_containing_obj(obj); + G1Mux2Closure mux(&vl_cl, &vr_cl); + obj->oop_iterate_no_header(&mux); + + if (vr_cl.failures()) { + *failures = true; + } + if (G1MaxVerifyFailures >= 0 && + vr_cl.n_failures() >= G1MaxVerifyFailures) { + return; + } + } else { + // verify only liveness obj->oop_iterate_no_header(&vl_cl); + } if (vl_cl.failures()) { *failures = true; } if (G1MaxVerifyFailures >= 0 && vl_cl.n_failures() >= G1MaxVerifyFailures) { return; } } } else { ! log_info(gc, verify)(PTR_FORMAT " not an oop", p2i(obj)); *failures = true; return; } } prev_p = p;
*** 850,859 **** --- 898,947 ---- void HeapRegion::verify() const { bool dummy = false; verify(VerifyOption_G1UsePrevMarking, /* failures */ &dummy); } + void HeapRegion::verify_rem_set(VerifyOption vo, bool* failures) const { + G1CollectedHeap* g1 = G1CollectedHeap::heap(); + *failures = false; + HeapWord* p = bottom(); + HeapWord* prev_p = NULL; + VerifyRemSetClosure vr_cl(g1, vo); + while (p < top()) { + oop obj = oop(p); + size_t obj_size = block_size(p); + + if (!g1->is_obj_dead_cond(obj, this, vo)) { + if (obj->is_oop()) { + vr_cl.set_containing_obj(obj); + obj->oop_iterate_no_header(&vr_cl); + + if (vr_cl.failures()) { + *failures = true; + } + if (G1MaxVerifyFailures >= 0 && + vr_cl.n_failures() >= G1MaxVerifyFailures) { + return; + } + } else { + log_info(gc, verify)(PTR_FORMAT " not an oop", p2i(obj)); + *failures = true; + return; + } + } + + prev_p = p; + p += obj_size; + } + } + + void HeapRegion::verify_rem_set() const { + bool failures = false; + verify_rem_set(VerifyOption_G1UsePrevMarking, &failures); + guarantee(!failures, "HeapRegion RemSet verification failed"); + } + void HeapRegion::prepare_for_compaction(CompactPoint* cp) { scan_and_forward(this, cp); } // G1OffsetTableContigSpace code; copied from space.cpp. Hope this can go