Skip to content

Commit 94e14da

Browse files
committedSep 28, 2022
8294057: Parallel: Tighten ParallelCompactData::initialize_region_data
Reviewed-by: tschatzl, sjohanss
1 parent 1ea0d6b commit 94e14da

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed
 

‎src/hotspot/share/gc/parallel/psParallelCompact.cpp

+4-3
Original file line numberDiff line numberDiff line change
@@ -432,8 +432,6 @@ bool ParallelCompactData::initialize(MemRegion covered_region)
432432

433433
assert(region_align_down(_region_start) == _region_start,
434434
"region start not aligned");
435-
assert((region_size & RegionSizeOffsetMask) == 0,
436-
"region size not a multiple of RegionSize");
437435

438436
bool result = initialize_region_data(region_size) && initialize_block_data();
439437
return result;
@@ -470,7 +468,10 @@ ParallelCompactData::create_vspace(size_t count, size_t element_size)
470468

471469
bool ParallelCompactData::initialize_region_data(size_t region_size)
472470
{
473-
const size_t count = (region_size + RegionSizeOffsetMask) >> Log2RegionSize;
471+
assert((region_size & RegionSizeOffsetMask) == 0,
472+
"region size not a multiple of RegionSize");
473+
474+
const size_t count = region_size >> Log2RegionSize;
474475
_region_vspace = create_vspace(count, sizeof(RegionData));
475476
if (_region_vspace != 0) {
476477
_region_data = (RegionData*)_region_vspace->reserved_low_addr();

1 commit comments

Comments
 (1)

openjdk-notifier[bot] commented on Sep 28, 2022

@openjdk-notifier[bot]
Please sign in to comment.