8316929: Shenandoah: Shenandoah degenerated GC and full GC need to cleanup old OopMapCache entries

zhengyu123 · zhengyu123 · commit e25121d1d908 · 2023-10-02T20:21:58.000Z
Reviewed-by: shade
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahVMOperations.cpp b/src/hotspot/share/gc/shenandoah/shenandoahVMOperations.cpp
@@ -32,6 +32,7 @@
 #include "gc/shenandoah/shenandoahOopClosures.inline.hpp"
 #include "gc/shenandoah/shenandoahUtils.hpp"
 #include "gc/shenandoah/shenandoahVMOperations.hpp"
+#include "interpreter/oopMapCache.hpp"
 #include "memory/universe.hpp"
 
 bool VM_ShenandoahReferenceOperation::doit_prologue() {
@@ -40,6 +41,7 @@ bool VM_ShenandoahReferenceOperation::doit_prologue() {
 }
 
 void VM_ShenandoahReferenceOperation::doit_epilogue() {
+  OopMapCache::cleanup_old_entries();
   if (Universe::has_reference_pending_list()) {
     Heap_lock->notify_all();
   }

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@`
`32`	`32`	`#include "gc/shenandoah/shenandoahOopClosures.inline.hpp"`
`33`	`33`	`#include "gc/shenandoah/shenandoahUtils.hpp"`
`34`	`34`	`#include "gc/shenandoah/shenandoahVMOperations.hpp"`
	`35`	`+#include "interpreter/oopMapCache.hpp"`
`35`	`36`	`#include "memory/universe.hpp"`
`36`	`37`
`37`	`38`	`bool VM_ShenandoahReferenceOperation::doit_prologue() {`
`@@ -40,6 +41,7 @@ bool VM_ShenandoahReferenceOperation::doit_prologue() {`
`40`	`41`	`}`
`41`	`42`
`42`	`43`	`void VM_ShenandoahReferenceOperation::doit_epilogue() {`
	`44`	`+ OopMapCache::cleanup_old_entries();`
`43`	`45`	`if (Universe::has_reference_pending_list()) {`
`44`	`46`	`Heap_lock->notify_all();`
`45`	`47`	`}`