8210884: ZGC: Remove insertion of filler objects
authorpliden
Wed, 19 Sep 2018 19:12:20 +0200
changeset 51804 eef954e29714
parent 51803 a16777c0a6c5
child 51805 eb2adb0a9b09
8210884: ZGC: Remove insertion of filler objects Reviewed-by: eosterlund
src/hotspot/share/gc/z/zPage.inline.hpp
src/hotspot/share/gc/z/zUtils.cpp
src/hotspot/share/gc/z/zUtils.hpp
--- a/src/hotspot/share/gc/z/zPage.inline.hpp	Wed Sep 19 19:12:18 2018 +0200
+++ b/src/hotspot/share/gc/z/zPage.inline.hpp	Wed Sep 19 19:12:20 2018 +0200
@@ -285,11 +285,6 @@
 
   _top = new_top;
 
-  // Fill alignment padding if needed
-  if (aligned_size != size) {
-    ZUtils::insert_filler_object(addr + size, aligned_size - size);
-  }
-
   return ZAddress::good(addr);
 }
 
@@ -308,11 +303,6 @@
 
     const uintptr_t prev_top = Atomic::cmpxchg(new_top, &_top, addr);
     if (prev_top == addr) {
-      // Fill alignment padding if needed
-      if (aligned_size != size) {
-        ZUtils::insert_filler_object(addr + size, aligned_size - size);
-      }
-
       // Success
       return ZAddress::good(addr);
     }
--- a/src/hotspot/share/gc/z/zUtils.cpp	Wed Sep 19 19:12:18 2018 +0200
+++ b/src/hotspot/share/gc/z/zUtils.cpp	Wed Sep 19 19:12:20 2018 +0200
@@ -22,8 +22,6 @@
  */
 
 #include "precompiled.hpp"
-#include "gc/shared/collectedHeap.hpp"
-#include "gc/z/zAddress.inline.hpp"
 #include "gc/z/zUtils.inline.hpp"
 #include "utilities/debug.hpp"
 
@@ -40,10 +38,3 @@
 
   return (uintptr_t)res;
 }
-
-void ZUtils::insert_filler_object(uintptr_t addr, size_t size) {
-  const size_t fill_size_in_words = bytes_to_words(size);
-  if (fill_size_in_words >= CollectedHeap::min_fill_size()) {
-    CollectedHeap::fill_with_objects((HeapWord*)ZAddress::good(addr), fill_size_in_words);
-  }
-}
--- a/src/hotspot/share/gc/z/zUtils.hpp	Wed Sep 19 19:12:18 2018 +0200
+++ b/src/hotspot/share/gc/z/zUtils.hpp	Wed Sep 19 19:12:20 2018 +0200
@@ -42,9 +42,6 @@
   // Object
   static size_t object_size(uintptr_t addr);
   static void object_copy(uintptr_t from, uintptr_t to, size_t size);
-
-  // Filler
-  static void insert_filler_object(uintptr_t addr, size_t size);
 };
 
 #endif // SHARE_GC_Z_ZUTILS_HPP