Skip to content

Commit 5981697

Browse files
committed
8337828: CDS: Trim down minimum GC region alignment
Reviewed-by: iklam, phh
1 parent cafb3dc commit 5981697

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/hotspot/share/cds/archiveHeapWriter.hpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,10 @@ class ArchiveHeapWriter : AllStatic {
112112
public:
113113
static const intptr_t NOCOOPS_REQUESTED_BASE = 0x10000000;
114114

115-
// The minimum region size of all collectors that are supported by CDS in
116-
// ArchiveHeapLoader::can_map() mode. Currently only G1 is supported. G1's region size
117-
// depends on -Xmx, but can never be smaller than 1 * M.
118-
// (TODO: Perhaps change to 256K to be compatible with Shenandoah)
119-
static constexpr int MIN_GC_REGION_ALIGNMENT = 1 * M;
115+
// The minimum region size of all collectors that are supported by CDS.
116+
// G1 heap region size can never be smaller than 1M.
117+
// Shenandoah heap region size can never be smaller than 256K.
118+
static constexpr int MIN_GC_REGION_ALIGNMENT = 256 * K;
120119

121120
private:
122121
class EmbeddedOopRelocator;

0 commit comments

Comments
 (0)