Assert that all allocated memory is used

openjdk · Oct 27, 2020 · 0cb3a80 · 0cb3a80
1 parent ced555a
commit 0cb3a80
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/src/hotspot/share/gc/z/zForwardingAllocator.hpp b/src/hotspot/share/gc/z/zForwardingAllocator.hpp
@@ -38,6 +38,7 @@ class ZForwardingAllocator {
 
   void reset(size_t size);
   size_t size() const;
+  bool is_full() const;
 
   void* alloc(size_t size);
 };

diff --git a/src/hotspot/share/gc/z/zForwardingAllocator.inline.hpp b/src/hotspot/share/gc/z/zForwardingAllocator.inline.hpp
@@ -25,11 +25,16 @@
 #define SHARE_GC_Z_ZFORWARDINGALLOCATOR_INLINE_HPP
 
 #include "gc/z/zForwardingAllocator.hpp"
+#include "utilities/debug.hpp"
 
 inline size_t ZForwardingAllocator::size() const {
   return _end - _start;
 }
 
+inline bool ZForwardingAllocator::is_full() const {
+  return _top == _end;
+}
+
 inline void* ZForwardingAllocator::alloc(size_t size) {
   char* const addr = _top;
   _top += size;

diff --git a/src/hotspot/share/gc/z/zRelocationSet.cpp b/src/hotspot/share/gc/z/zRelocationSet.cpp
@@ -27,6 +27,7 @@
 #include "gc/z/zRelocationSet.hpp"
 #include "gc/z/zStat.hpp"
 #include "memory/allocation.hpp"
+#include "utilities/debug.hpp"
 
 ZRelocationSet::ZRelocationSet() :
     _allocator(),
@@ -62,6 +63,8 @@ void ZRelocationSet::populate(ZPage* const* small, size_t nsmall,
     _forwardings[j++] = ZForwarding::alloc(&_allocator, small[i]);
   }
 
+  assert(_allocator.is_full(), "Should be full");
+
   // Update statistics
   ZStatRelocation::set_at_populate_relocation_set(_allocator.size());
 }