8257793: Shenandoah: SATB barrier should only filter out already strongly marked oops

zhengyu123 · zhengyu123 · commit ecd7e476f838 · 2020-12-07T13:18:02.000Z
Reviewed-by: shade, rkennke
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahHeap.inline.hpp b/src/hotspot/share/gc/shenandoah/shenandoahHeap.inline.hpp
@@ -328,7 +328,7 @@ inline oop ShenandoahHeap::evacuate_object(oop p, Thread* thread) {
 
 inline bool ShenandoahHeap::requires_marking(const void* entry) const {
   oop obj = oop(entry);
-  return !_marking_context->is_marked(obj);
+  return !_marking_context->is_marked_strong(obj);
 }
 
 inline bool ShenandoahHeap::in_collection_set(oop p) const {

Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,7 @@ inline oop ShenandoahHeap::evacuate_object(oop p, Thread* thread) {`
`328`	`328`
`329`	`329`	`inline bool ShenandoahHeap::requires_marking(const void* entry) const {`
`330`	`330`	`oop obj = oop(entry);`
`331`		`- return !_marking_context->is_marked(obj);`
	`331`	`+ return !_marking_context->is_marked_strong(obj);`
`332`	`332`	`}`
`333`	`333`
`334`	`334`	`inline bool ShenandoahHeap::in_collection_set(oop p) const {`