8304657: G1: Rename set_state_empty to set_state_untracked
Reviewed-by: tschatzl
This commit is contained in:
parent
ca94287d2b
commit
c74507eeb3
@ -47,10 +47,10 @@ void G1RemSetTrackingPolicy::update_at_allocate(HeapRegion* r) {
|
|||||||
r->rem_set()->set_state_complete();
|
r->rem_set()->set_state_complete();
|
||||||
} else if (r->is_archive()) {
|
} else if (r->is_archive()) {
|
||||||
// Archive regions never move ever. So never build remembered sets for them.
|
// 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()) {
|
} else if (r->is_old()) {
|
||||||
// By default, do not create remembered set for new old regions.
|
// By default, do not create remembered set for new old regions.
|
||||||
r->rem_set()->set_state_empty();
|
r->rem_set()->set_state_untracked();
|
||||||
} else {
|
} else {
|
||||||
guarantee(false, "Unhandled region %u with heap region type %s", r->hrm_index(), r->get_type_str());
|
guarantee(false, "Unhandled region %u with heap region type %s", r->hrm_index(), r->get_type_str());
|
||||||
}
|
}
|
||||||
|
@ -79,7 +79,7 @@ void HeapRegionRemSet::clear_locked(bool only_cardset) {
|
|||||||
}
|
}
|
||||||
clear_fcc();
|
clear_fcc();
|
||||||
_card_set.clear();
|
_card_set.clear();
|
||||||
set_state_empty();
|
set_state_untracked();
|
||||||
assert(occupied() == 0, "Should be clear.");
|
assert(occupied() == 0, "Should be clear.");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -109,7 +109,7 @@ public:
|
|||||||
bool is_updating() { return _state == Updating; }
|
bool is_updating() { return _state == Updating; }
|
||||||
bool is_complete() { return _state == Complete; }
|
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_updating();
|
||||||
inline void set_state_complete();
|
inline void set_state_complete();
|
||||||
|
|
||||||
|
@ -33,7 +33,7 @@
|
|||||||
#include "runtime/atomic.hpp"
|
#include "runtime/atomic.hpp"
|
||||||
#include "utilities/bitMap.inline.hpp"
|
#include "utilities/bitMap.inline.hpp"
|
||||||
|
|
||||||
void HeapRegionRemSet::set_state_empty() {
|
void HeapRegionRemSet::set_state_untracked() {
|
||||||
guarantee(SafepointSynchronize::is_at_safepoint() || !is_tracked(),
|
guarantee(SafepointSynchronize::is_at_safepoint() || !is_tracked(),
|
||||||
"Should only set to Untracked during safepoint but is %s.", get_state_str());
|
"Should only set to Untracked during safepoint but is %s.", get_state_str());
|
||||||
if (_state == Untracked) {
|
if (_state == Untracked) {
|
||||||
|
Loading…
Reference in New Issue
Block a user