8275440: Remove VirtualSpaceList::is_full()

Backport-of: c9e65f8ef926c3796867558afa536770eed71cd6

Author: GoeLin

src/hotspot/share/memory/metaspace/virtualSpaceList.cpp

@@ -200,15 +200,6 @@ bool VirtualSpaceList::contains(const MetaWord* p) const {
   return false;
 }
 
-// Returns true if the vslist is not expandable and no more root chunks
-// can be allocated.
-bool VirtualSpaceList::is_full() const {
-  if (!_can_expand && _first_node != NULL && _first_node->free_words() == 0) {
-    return true;
-  }
-  return false;
-}
-
 // Convenience methods to return the global class-space chunkmanager
 //  and non-class chunkmanager, respectively.
 VirtualSpaceList* VirtualSpaceList::vslist_class() {

src/hotspot/share/memory/metaspace/virtualSpaceList.hpp

@@ -128,10 +128,6 @@ class VirtualSpaceList : public CHeapObj<mtClass> {
   // Returns true if this pointer is contained in one of our nodes.
   bool contains(const MetaWord* p) const;
 
-  // Returns true if the list is not expandable and no more root chunks
-  // can be allocated.
-  bool is_full() const;
-
   // Convenience methods to return the global class-space vslist
   //  and non-class vslist, respectively.
   static VirtualSpaceList* vslist_class();

test/hotspot/gtest/metaspace/test_chunkManager_stress.cpp

@@ -61,9 +61,9 @@ class ChunkManagerRandomChunkAllocTest {
     return max_chunks;
   }
 
-  // Return true if, after an allocation error happened, a reserve error seems likely.
+  // Return true if, after an allocation error happened, a reserve error seems possible.
   bool could_be_reserve_error() {
-    return _context.vslist().is_full();
+    return _context.reserve_limit() < max_uintx;
   }
 
   // Return true if, after an allocation error happened, a commit error seems likely.