RFR: 8272773: Investigate making card table size configurable [v3]

Vishal Chand duke at openjdk.java.net
Thu Oct 14 13:12:53 UTC 2021


On Thu, 14 Oct 2021 06:51:28 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> src/hotspot/share/gc/g1/g1CardTable.hpp line 79:
>> 
>>> 77:   static const size_t WordAllClean = SIZE_MAX;
>>> 78:   static const size_t WordAllDirty = 0;
>>> 79:   static const size_t LogOfMaxCardsPerRegion = BitsPerByte * sizeof(G1CardSetArray::EntryDataType);
>> 
>> @tschatzl Do we need this code once JDK-8275056 (Virtualize G1CardSet containers over heap region) gets integrated? From my understanding, this won't be a restriction from heap region point of view. Do I need to remove this and the changes in `g1CardTable.cpp `?
>
> Yes. The change you will need will be very similar to https://github.com/openjdk/jdk/commit/70dcca62c4fd6657729beb3c2b985eb342b5788b (slightly modified because of review comments); but please wait until it is in. No hurry though, we'll need to complete the CSR before we can integrate anyway.
> 
> Thomas

okay, got it.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5838



More information about the hotspot-gc-dev mailing list