8329962: Remove CardTable::invalidate

Reviewed-by: tschatzl, gli
This commit is contained in:
Albert Mingkun Yang 2024-04-12 07:37:48 +00:00
parent c7fcd62302
commit 006a516aa0
3 changed files with 2 additions and 12 deletions

@ -202,6 +202,7 @@ void CardTable::resize_covered_region(MemRegion new_region) {
void CardTable::dirty_MemRegion(MemRegion mr) {
assert(align_down(mr.start(), HeapWordSize) == mr.start(), "Unaligned start");
assert(align_up (mr.end(), HeapWordSize) == mr.end(), "Unaligned end" );
assert(_covered[0].contains(mr) || _covered[1].contains(mr), "precondition");
CardValue* cur = byte_for(mr.start());
CardValue* last = byte_after(mr.last());
memset(cur, dirty_card, pointer_delta(last, cur, sizeof(CardValue)));
@ -226,15 +227,6 @@ uintx CardTable::ct_max_alignment_constraint() {
return GCCardSizeInBytes * os::vm_page_size();
}
void CardTable::invalidate(MemRegion mr) {
assert(align_down(mr.start(), HeapWordSize) == mr.start(), "Unaligned start");
assert(align_up (mr.end(), HeapWordSize) == mr.end(), "Unaligned end" );
for (int i = 0; i < max_covered_regions; i++) {
MemRegion mri = mr.intersection(_covered[i]);
if (!mri.is_empty()) dirty_MemRegion(mri);
}
}
#ifndef PRODUCT
void CardTable::verify_region(MemRegion mr, CardValue val, bool val_equals) {
CardValue* start = byte_for(mr.start());

@ -131,8 +131,6 @@ public:
return byte_for(p) + 1;
}
void invalidate(MemRegion mr);
// Provide read-only access to the card table array.
const CardValue* byte_for_const(const void* p) const {
return byte_for(p);

@ -85,7 +85,7 @@ void CardTableBarrierSet::write_ref_array_work(MemRegion mr) {
}
void CardTableBarrierSet::invalidate(MemRegion mr) {
_card_table->invalidate(mr);
_card_table->dirty_MemRegion(mr);
}
void CardTableBarrierSet::print_on(outputStream* st) const {