From c8452615b1f3c4e03caf70e2c72928d49cb816a7 Mon Sep 17 00:00:00 2001 From: Albert Mingkun Yang Date: Wed, 8 May 2024 17:56:28 +0000 Subject: [PATCH] 8331924: Parallel: Remove unused MutableSpace::mangle_unused_area_complete Reviewed-by: tschatzl --- src/hotspot/share/gc/parallel/mutableNUMASpace.cpp | 5 +---- src/hotspot/share/gc/parallel/mutableNUMASpace.hpp | 2 +- src/hotspot/share/gc/parallel/mutableSpace.cpp | 4 ---- src/hotspot/share/gc/parallel/mutableSpace.hpp | 2 +- 4 files changed, 3 insertions(+), 10 deletions(-) diff --git a/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp b/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp index 2a39ce7b470..b4ea1955182 100644 --- a/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp +++ b/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp @@ -79,10 +79,7 @@ void MutableNUMASpace::mangle_unused_area() { // This method should do nothing. // It can be called on a numa space during a full compaction. } -void MutableNUMASpace::mangle_unused_area_complete() { - // This method should do nothing. - // It can be called on a numa space during a full compaction. -} + void MutableNUMASpace::mangle_region(MemRegion mr) { // This method should do nothing because numa spaces are not mangled. } diff --git a/src/hotspot/share/gc/parallel/mutableNUMASpace.hpp b/src/hotspot/share/gc/parallel/mutableNUMASpace.hpp index 18271fabd05..9a6cf420c0d 100644 --- a/src/hotspot/share/gc/parallel/mutableNUMASpace.hpp +++ b/src/hotspot/share/gc/parallel/mutableNUMASpace.hpp @@ -173,7 +173,7 @@ public: virtual void clear(bool mangle_space); virtual void mangle_unused_area() PRODUCT_RETURN; - virtual void mangle_unused_area_complete() PRODUCT_RETURN; + virtual void mangle_region(MemRegion mr) PRODUCT_RETURN; virtual void check_mangled_unused_area(HeapWord* limit) PRODUCT_RETURN; virtual void check_mangled_unused_area_complete() PRODUCT_RETURN; diff --git a/src/hotspot/share/gc/parallel/mutableSpace.cpp b/src/hotspot/share/gc/parallel/mutableSpace.cpp index 4641b4ce5c6..7d881f73dbf 100644 --- a/src/hotspot/share/gc/parallel/mutableSpace.cpp +++ b/src/hotspot/share/gc/parallel/mutableSpace.cpp @@ -167,10 +167,6 @@ void MutableSpace::mangle_unused_area() { mangler()->mangle_unused_area(); } -void MutableSpace::mangle_unused_area_complete() { - mangler()->mangle_unused_area_complete(); -} - void MutableSpace::mangle_region(MemRegion mr) { SpaceMangler::mangle_region(mr); } diff --git a/src/hotspot/share/gc/parallel/mutableSpace.hpp b/src/hotspot/share/gc/parallel/mutableSpace.hpp index b77c1d80332..1cf0d306dc0 100644 --- a/src/hotspot/share/gc/parallel/mutableSpace.hpp +++ b/src/hotspot/share/gc/parallel/mutableSpace.hpp @@ -109,7 +109,7 @@ class MutableSpace: public CHeapObj { // Methods used in mangling. See descriptions under SpaceMangler. virtual void mangle_unused_area() PRODUCT_RETURN; - virtual void mangle_unused_area_complete() PRODUCT_RETURN; + virtual void check_mangled_unused_area(HeapWord* limit) PRODUCT_RETURN; virtual void check_mangled_unused_area_complete() PRODUCT_RETURN; virtual void set_top_for_allocations(HeapWord* v) PRODUCT_RETURN;