stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8289575: G1: Remove unnecessary is-marking-active check in G1BarrierSetRuntime::write_ref_field_pre_entry

Browse Source 
Reviewed-by: tschatzl, iwalulya
This commit is contained in:
Albert Mingkun Yang 2022-07-04 08:04:01 +00:00
parent 8e7a3cb5ab
commit e31003a064
2 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/hotspot/share/gc/g1/g1BarrierSetRuntime.cpp
View File

@ -50,7 +50,7 @@ JRT_LEAF(void, G1BarrierSetRuntime::write_ref_field_pre_entry(oopDesc* orig, Jav
assert(oopDesc::is_oop(orig, true /* ignore mark word */), "Error");
// store the original value that was in the field reference
SATBMarkQueue& queue = G1ThreadLocalData::satb_mark_queue(thread);
G1BarrierSet::satb_mark_queue_set().enqueue(queue, orig);
G1BarrierSet::satb_mark_queue_set().enqueue_known_active(queue, orig);
JRT_END
// G1 post write barrier slowpath

4
src/hotspot/share/gc/shared/satbMarkQueue.hpp
View File

@ -140,10 +140,6 @@ public:
void flush_queue(SATBMarkQueue& queue);
// When active, add obj to queue by calling enqueue_known_active.
void enqueue(SATBMarkQueue& queue, oop obj) {
if (queue.is_active()) enqueue_known_active(queue, obj);
}
// Add obj to queue. This qset and the queue must be active.
void enqueue_known_active(SATBMarkQueue& queue, oop obj);
virtual void filter(SATBMarkQueue& queue) = 0;