< prev index next >

src/hotspot/share/gc/shenandoah/shenandoahAsserts.cpp

Print this page
*** 35,11 ***
  void print_raw_memory(ShenandoahMessageBuffer &msg, void* loc) {
    // Be extra safe. Only access data that is guaranteed to be safe:
    // should be in heap, in known committed region, within that region.
  
    ShenandoahHeap* heap = ShenandoahHeap::heap();
!   if (!heap->is_in(loc)) return;
  
    ShenandoahHeapRegion* r = heap->heap_region_containing(loc);
    if (r != nullptr && r->is_committed()) {
      address start = MAX2((address) r->bottom(), (address) loc - 32);
      address end   = MIN2((address) r->end(),    (address) loc + 128);
--- 35,11 ---
  void print_raw_memory(ShenandoahMessageBuffer &msg, void* loc) {
    // Be extra safe. Only access data that is guaranteed to be safe:
    // should be in heap, in known committed region, within that region.
  
    ShenandoahHeap* heap = ShenandoahHeap::heap();
!   if (!heap->is_in_reserved(loc)) return;
  
    ShenandoahHeapRegion* r = heap->heap_region_containing(loc);
    if (r != nullptr && r->is_committed()) {
      address start = MAX2((address) r->bottom(), (address) loc - 32);
      address end   = MIN2((address) r->end(),    (address) loc + 128);

*** 69,17 ***
    msg.append("    %3s allocated after mark start\n", ctx->allocated_after_mark_start(obj) ? "" : "not");
    msg.append("    %3s after update watermark\n",     cast_from_oop<HeapWord*>(obj) >= r->get_update_watermark() ? "" : "not");
    msg.append("    %3s marked strong\n",              ctx->is_marked_strong(obj) ? "" : "not");
    msg.append("    %3s marked weak\n",                ctx->is_marked_weak(obj) ? "" : "not");
    msg.append("    %3s in collection set\n",          heap->in_collection_set(obj) ? "" : "not");
    msg.append("  mark:%s\n", mw_ss.freeze());
    msg.append("  region: %s", ss.freeze());
  }
  
  void ShenandoahAsserts::print_non_obj(ShenandoahMessageBuffer& msg, void* loc) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
!   if (heap->is_in(loc)) {
      msg.append("  inside Java heap\n");
      ShenandoahHeapRegion *r = heap->heap_region_containing(loc);
      stringStream ss;
      r->print_on(&ss);
  
--- 69,20 ---
    msg.append("    %3s allocated after mark start\n", ctx->allocated_after_mark_start(obj) ? "" : "not");
    msg.append("    %3s after update watermark\n",     cast_from_oop<HeapWord*>(obj) >= r->get_update_watermark() ? "" : "not");
    msg.append("    %3s marked strong\n",              ctx->is_marked_strong(obj) ? "" : "not");
    msg.append("    %3s marked weak\n",                ctx->is_marked_weak(obj) ? "" : "not");
    msg.append("    %3s in collection set\n",          heap->in_collection_set(obj) ? "" : "not");
+   if (heap->mode()->is_generational() && !obj->is_forwarded()) {
+     msg.append("  age: %d\n", obj->age());
+   }
    msg.append("  mark:%s\n", mw_ss.freeze());
    msg.append("  region: %s", ss.freeze());
  }
  
  void ShenandoahAsserts::print_non_obj(ShenandoahMessageBuffer& msg, void* loc) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
!   if (heap->is_in_reserved(loc)) {
      msg.append("  inside Java heap\n");
      ShenandoahHeapRegion *r = heap->heap_region_containing(loc);
      stringStream ss;
      r->print_on(&ss);
  

*** 94,11 ***
  }
  
  void ShenandoahAsserts::print_obj_safe(ShenandoahMessageBuffer& msg, void* loc) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
    msg.append("  " PTR_FORMAT " - safe print, no details\n", p2i(loc));
!   if (heap->is_in(loc)) {
      ShenandoahHeapRegion* r = heap->heap_region_containing(loc);
      if (r != nullptr) {
        stringStream ss;
        r->print_on(&ss);
        msg.append("  region: %s", ss.freeze());
--- 97,11 ---
  }
  
  void ShenandoahAsserts::print_obj_safe(ShenandoahMessageBuffer& msg, void* loc) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
    msg.append("  " PTR_FORMAT " - safe print, no details\n", p2i(loc));
!   if (heap->is_in_reserved(loc)) {
      ShenandoahHeapRegion* r = heap->heap_region_containing(loc);
      if (r != nullptr) {
        stringStream ss;
        r->print_on(&ss);
        msg.append("  region: %s", ss.freeze());

*** 111,11 ***
                                         const char* phase, const char* label,
                                         const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
    ResourceMark rm;
  
!   bool loc_in_heap = (loc != nullptr && heap->is_in(loc));
  
    ShenandoahMessageBuffer msg("%s; %s\n\n", phase, label);
  
    msg.append("Referenced from:\n");
    if (interior_loc != nullptr) {
--- 114,11 ---
                                         const char* phase, const char* label,
                                         const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
    ResourceMark rm;
  
!   bool loc_in_heap = (loc != nullptr && heap->is_in_reserved(loc));
  
    ShenandoahMessageBuffer msg("%s; %s\n\n", phase, label);
  
    msg.append("Referenced from:\n");
    if (interior_loc != nullptr) {

*** 164,38 ***
    }
  
    report_vm_error(file, line, msg.buffer());
  }
  
! void ShenandoahAsserts::assert_in_heap(void* interior_loc, oop obj, const char *file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
!   if (!heap->is_in(obj)) {
!     print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_in_heap failed",
!                   "oop must point to a heap address",
                    file, line);
    }
  }
  
! void ShenandoahAsserts::assert_in_heap_or_null(void* interior_loc, oop obj, const char *file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
!   if (obj != nullptr && !heap->is_in(obj)) {
!     print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_in_heap_or_null failed",
!                   "oop must point to a heap address",
                    file, line);
    }
  }
  
  void ShenandoahAsserts::assert_correct(void* interior_loc, oop obj, const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
    // Step 1. Check that obj is correct.
    // After this step, it is safe to call heap_region_containing().
!   if (!heap->is_in(obj)) {
      print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                   "oop must point to a heap address",
                    file, line);
    }
  
    Klass* obj_klass = obj->klass_or_null();
    if (obj_klass == nullptr) {
--- 167,38 ---
    }
  
    report_vm_error(file, line, msg.buffer());
  }
  
! void ShenandoahAsserts::assert_in_heap_bounds(void* interior_loc, oop obj, const char *file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
!   if (!heap->is_in_reserved(obj)) {
!     print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_in_heap_bounds failed",
!                   "oop must be in heap bounds",
                    file, line);
    }
  }
  
! void ShenandoahAsserts::assert_in_heap_bounds_or_null(void* interior_loc, oop obj, const char *file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
!   if (obj != nullptr && !heap->is_in_reserved(obj)) {
!     print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_in_heap_bounds_or_null failed",
!                   "oop must be in heap bounds",
                    file, line);
    }
  }
  
  void ShenandoahAsserts::assert_correct(void* interior_loc, oop obj, const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
  
    // Step 1. Check that obj is correct.
    // After this step, it is safe to call heap_region_containing().
!   if (!heap->is_in_reserved(obj)) {
      print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                   "oop must be in heap bounds",
                    file, line);
    }
  
    Klass* obj_klass = obj->klass_or_null();
    if (obj_klass == nullptr) {

*** 208,10 ***
--- 211,16 ---
      print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
                    "Object klass pointer must go to metaspace",
                    file,line);
    }
  
+   if (!heap->is_in(obj)) {
+     print_failure(_safe_unknown, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
+                   "Object should be in active region area",
+                   file, line);
+   }
+ 
    oop fwd = ShenandoahForwarding::get_forwardee_raw_unchecked(obj);
  
    if (obj != fwd) {
      // When Full GC moves the objects, we cannot trust fwdptrs. If we got here, it means something
      // tries fwdptr manipulation when Full GC is running. The only exception is using the fwdptr

*** 221,26 ***
                      "Non-trivial forwarding pointer during Full GC moves, probable bug.",
                      file, line);
      }
  
      // Step 2. Check that forwardee is correct
!     if (!heap->is_in(fwd)) {
        print_failure(_safe_oop, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                     "Forwardee must point to a heap address",
                      file, line);
      }
  
      if (obj_klass != fwd->klass()) {
        print_failure(_safe_oop, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
                      "Forwardee klass disagrees with object class",
                      file, line);
      }
  
      // Step 3. Check that forwardee points to correct region
      if (heap->heap_region_index_containing(fwd) == heap->heap_region_index_containing(obj)) {
        print_failure(_safe_all, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                     "Non-trivial forwardee should in another region",
                      file, line);
      }
  
      // Step 4. Check for multiple forwardings
      oop fwd2 = ShenandoahForwarding::get_forwardee_raw_unchecked(fwd);
--- 230,32 ---
                      "Non-trivial forwarding pointer during Full GC moves, probable bug.",
                      file, line);
      }
  
      // Step 2. Check that forwardee is correct
!     if (!heap->is_in_reserved(fwd)) {
        print_failure(_safe_oop, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                     "Forwardee must be in heap bounds",
                      file, line);
      }
  
      if (obj_klass != fwd->klass()) {
        print_failure(_safe_oop, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
                      "Forwardee klass disagrees with object class",
                      file, line);
      }
  
      // Step 3. Check that forwardee points to correct region
+     if (!heap->is_in(fwd)) {
+       print_failure(_safe_oop, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
+                     "Forwardee should be in active region area",
+                     file, line);
+     }
+ 
      if (heap->heap_region_index_containing(fwd) == heap->heap_region_index_containing(obj)) {
        print_failure(_safe_all, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
!                     "Non-trivial forwardee should be in another region",
                      file, line);
      }
  
      // Step 4. Check for multiple forwardings
      oop fwd2 = ShenandoahForwarding::get_forwardee_raw_unchecked(fwd);

*** 248,10 ***
--- 263,29 ---
        print_failure(_safe_all, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
                      "Multiple forwardings",
                      file, line);
      }
    }
+ 
+   // Do additional checks for special objects: their fields can hold metadata as well.
+   // We want to check class loading/unloading did not corrupt them.
+ 
+   if (obj_klass == vmClasses::Class_klass()) {
+     Metadata* klass = obj->metadata_field(java_lang_Class::klass_offset());
+     if (klass != nullptr && !Metaspace::contains(klass)) {
+       print_failure(_safe_all, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
+                     "Instance class mirror should point to Metaspace",
+                     file, line);
+     }
+ 
+     Metadata* array_klass = obj->metadata_field(java_lang_Class::array_klass_offset());
+     if (array_klass != nullptr && !Metaspace::contains(array_klass)) {
+       print_failure(_safe_all, obj, interior_loc, nullptr, "Shenandoah assert_correct failed",
+                     "Array class mirror should point to Metaspace",
+                     file, line);
+     }
+   }
  }
  
  void ShenandoahAsserts::assert_in_correct_region(void* interior_loc, oop obj, const char* file, int line) {
    assert_correct(interior_loc, obj, file, line);
  

*** 262,11 ***
                    "Object must reside in active region",
                    file, line);
    }
  
    size_t alloc_size = obj->size();
!   if (alloc_size > ShenandoahHeapRegion::humongous_threshold_words()) {
      size_t idx = r->index();
      size_t num_regions = ShenandoahHeapRegion::required_regions(alloc_size * HeapWordSize);
      for (size_t i = idx; i < idx + num_regions; i++) {
        ShenandoahHeapRegion* chain_reg = heap->get_region(i);
        if (i == idx && !chain_reg->is_humongous_start()) {
--- 296,11 ---
                    "Object must reside in active region",
                    file, line);
    }
  
    size_t alloc_size = obj->size();
!   if (ShenandoahHeapRegion::requires_humongous(alloc_size)) {
      size_t idx = r->index();
      size_t num_regions = ShenandoahHeapRegion::required_regions(alloc_size * HeapWordSize);
      for (size_t i = idx; i < idx + num_regions; i++) {
        ShenandoahHeapRegion* chain_reg = heap->get_region(i);
        if (i == idx && !chain_reg->is_humongous_start()) {

*** 383,11 ***
  
    if (lock->owned_by_self()) {
      return;
    }
  
!   ShenandoahMessageBuffer msg("Must ba at a Shenandoah safepoint or held %s lock", lock->name());
    report_vm_error(file, line, msg.buffer());
  }
  
  void ShenandoahAsserts::assert_heaplocked(const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();
--- 417,11 ---
  
    if (lock->owned_by_self()) {
      return;
    }
  
!   ShenandoahMessageBuffer msg("Must be at a Shenandoah safepoint or held %s lock", lock->name());
    report_vm_error(file, line, msg.buffer());
  }
  
  void ShenandoahAsserts::assert_heaplocked(const char* file, int line) {
    ShenandoahHeap* heap = ShenandoahHeap::heap();

*** 416,12 ***
  
    if (heap->lock()->owned_by_self()) {
      return;
    }
  
!   if (ShenandoahSafepoint::is_at_shenandoah_safepoint() && Thread::current()->is_VM_thread()) {
      return;
    }
  
    ShenandoahMessageBuffer msg("Heap lock must be owned by current thread, or be at safepoint");
    report_vm_error(file, line, msg.buffer());
  }
--- 450,57 ---
  
    if (heap->lock()->owned_by_self()) {
      return;
    }
  
!   if (ShenandoahSafepoint::is_at_shenandoah_safepoint()) {
      return;
    }
  
    ShenandoahMessageBuffer msg("Heap lock must be owned by current thread, or be at safepoint");
    report_vm_error(file, line, msg.buffer());
  }
+ 
+ void ShenandoahAsserts::assert_generational(const char* file, int line) {
+   if (ShenandoahHeap::heap()->mode()->is_generational()) {
+     return;
+   }
+ 
+   ShenandoahMessageBuffer msg("Must be in generational mode");
+   report_vm_error(file, line, msg.buffer());
+ }
+ 
+ void ShenandoahAsserts::assert_control_or_vm_thread_at_safepoint(bool at_safepoint, const char* file, int line) {
+   Thread* thr = Thread::current();
+   if (thr == ShenandoahHeap::heap()->control_thread()) {
+     return;
+   }
+   if (thr->is_VM_thread()) {
+     if (!at_safepoint) {
+       return;
+     } else if (SafepointSynchronize::is_at_safepoint()) {
+       return;
+     }
+   }
+ 
+   ShenandoahMessageBuffer msg("Must be either control thread, or vm thread");
+   if (at_safepoint) {
+     msg.append(" at a safepoint");
+   }
+   report_vm_error(file, line, msg.buffer());
+ }
+ 
+ void ShenandoahAsserts::assert_generations_reconciled(const char* file, int line) {
+   if (!SafepointSynchronize::is_at_safepoint()) {
+     return;
+   }
+ 
+   ShenandoahHeap* heap = ShenandoahHeap::heap();
+   ShenandoahGeneration* ggen = heap->gc_generation();
+   ShenandoahGeneration* agen = heap->active_generation();
+   if (agen == ggen) {
+     return;
+   }
+ 
+   ShenandoahMessageBuffer msg("Active(%d) & GC(%d) Generations aren't reconciled", agen->type(), ggen->type());
+   report_vm_error(file, line, msg.buffer());
+ }
< prev index next >