diff --git a/src/hotspot/share/gc/g1/g1RemSetTrackingPolicy.cpp b/src/hotspot/share/gc/g1/g1RemSetTrackingPolicy.cpp index 31d1fb817f1..cec83231a26 100644 --- a/src/hotspot/share/gc/g1/g1RemSetTrackingPolicy.cpp +++ b/src/hotspot/share/gc/g1/g1RemSetTrackingPolicy.cpp @@ -47,10 +47,10 @@ void G1RemSetTrackingPolicy::update_at_allocate(HeapRegion* r) { r->rem_set()->set_state_complete(); } else if (r->is_archive()) { // Archive regions never move ever. So never build remembered sets for them. - r->rem_set()->set_state_empty(); + r->rem_set()->set_state_untracked(); } else if (r->is_old()) { // By default, do not create remembered set for new old regions. - r->rem_set()->set_state_empty(); + r->rem_set()->set_state_untracked(); } else { guarantee(false, "Unhandled region %u with heap region type %s", r->hrm_index(), r->get_type_str()); } diff --git a/src/hotspot/share/gc/g1/heapRegionRemSet.cpp b/src/hotspot/share/gc/g1/heapRegionRemSet.cpp index 375b1ead79e..be5535a00da 100644 --- a/src/hotspot/share/gc/g1/heapRegionRemSet.cpp +++ b/src/hotspot/share/gc/g1/heapRegionRemSet.cpp @@ -79,7 +79,7 @@ void HeapRegionRemSet::clear_locked(bool only_cardset) { } clear_fcc(); _card_set.clear(); - set_state_empty(); + set_state_untracked(); assert(occupied() == 0, "Should be clear."); } diff --git a/src/hotspot/share/gc/g1/heapRegionRemSet.hpp b/src/hotspot/share/gc/g1/heapRegionRemSet.hpp index 1d130f6b8b2..584275e9d8b 100644 --- a/src/hotspot/share/gc/g1/heapRegionRemSet.hpp +++ b/src/hotspot/share/gc/g1/heapRegionRemSet.hpp @@ -109,7 +109,7 @@ public: bool is_updating() { return _state == Updating; } bool is_complete() { return _state == Complete; } - inline void set_state_empty(); + inline void set_state_untracked(); inline void set_state_updating(); inline void set_state_complete(); diff --git a/src/hotspot/share/gc/g1/heapRegionRemSet.inline.hpp b/src/hotspot/share/gc/g1/heapRegionRemSet.inline.hpp index fb8849a0e6d..ca8f675e6df 100644 --- a/src/hotspot/share/gc/g1/heapRegionRemSet.inline.hpp +++ b/src/hotspot/share/gc/g1/heapRegionRemSet.inline.hpp @@ -33,7 +33,7 @@ #include "runtime/atomic.hpp" #include "utilities/bitMap.inline.hpp" -void HeapRegionRemSet::set_state_empty() { +void HeapRegionRemSet::set_state_untracked() { guarantee(SafepointSynchronize::is_at_safepoint() || !is_tracked(), "Should only set to Untracked during safepoint but is %s.", get_state_str()); if (_state == Untracked) {