Skip to content

Commit

Permalink
8257999: Parallel GC crash in gc/parallel/TestDynShrinkHeap.java: new…
Browse files Browse the repository at this point in the history
… region is not in covered_region

Reviewed-by: sjohanss, tschatzl
  • Loading branch information
Kim Barrett committed Dec 17, 2020
1 parent 952dc70 commit 61390d8
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/hotspot/share/gc/parallel/psOldGen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
#include "logging/log.hpp"
#include "oops/oop.inline.hpp"
#include "runtime/java.hpp"
#include "runtime/orderAccess.hpp"
#include "utilities/align.hpp"

PSOldGen::PSOldGen(ReservedSpace rs, size_t initial_size, size_t min_size,
Expand Down Expand Up @@ -380,7 +381,9 @@ void PSOldGen::post_resize() {
WorkGang* workers = Thread::current()->is_VM_thread() ?
&ParallelScavengeHeap::heap()->workers() : NULL;

// 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,
Expand Down

0 comments on commit 61390d8

Please sign in to comment.