8257999: Parallel GC crash in gc/parallel/TestDynShrinkHeap.java: new…

… region is not in covered_region Backport-of: 61390d8
openjdk · Sep 15, 2021 · 56158e5 · 56158e5 · openjdk-notifier · Sep 15, 2021
1 parent e93b244
commit 56158e5
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/hotspot/share/gc/parallel/psOldGen.cpp b/src/hotspot/share/gc/parallel/psOldGen.cpp
@@ -37,6 +37,7 @@
 #include "logging/log.hpp"
 #include "oops/oop.inline.hpp"
 #include "runtime/java.hpp"
+#include "runtime/orderAccess.hpp"
 #include "utilities/align.hpp"
 
 inline const char* PSOldGen::select_name() {
@@ -402,7 +403,9 @@ void PSOldGen::post_resize() {
   start_array()->set_covered_region(new_memregion);
   ParallelScavengeHeap::heap()->card_table()->resize_covered_region(new_memregion);
 
-  // ALWAYS do this last!!
+  // Ensure the space bounds are updated and made visible to other
+  // threads after the other data structures have been resized.
+  OrderAccess::storestore();
   object_space()->initialize(new_memregion,
                              SpaceDecorator::DontClear,
                              SpaceDecorator::DontMangle);