8293213: G1: Remove redundant assertion in G1RemSet::clean_card_befor…

…e_refine Reviewed-by: tschatzl, kbarrett
openjdk · Sep 6, 2022 · 6a1e98c · 6a1e98c · openjdk-notifier · Sep 6, 2022
1 parent a92c1ff
commit 6a1e98c
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/src/hotspot/share/gc/g1/g1RemSet.cpp b/src/hotspot/share/gc/g1/g1RemSet.cpp
@@ -1602,7 +1602,7 @@ inline void check_card_ptr(CardTable::CardValue* card_ptr, G1CardTable* ct) {
 }
 
 bool G1RemSet::clean_card_before_refine(CardValue** const card_ptr_addr) {
-  assert(!_g1h->is_gc_active(), "Only call concurrently");
+  assert(!SafepointSynchronize::is_at_safepoint(), "Only call concurrently");
 
   CardValue* card_ptr = *card_ptr_addr;
   // Find the start address represented by the card.
@@ -1657,8 +1657,6 @@ bool G1RemSet::clean_card_before_refine(CardValue** const card_ptr_addr) {
   //
 
   if (G1HotCardCache::use_cache()) {
-    assert(!SafepointSynchronize::is_at_safepoint(), "sanity");
-
     const CardValue* orig_card_ptr = card_ptr;
     card_ptr = _hot_card_cache->insert(card_ptr);
     if (card_ptr == NULL) {