RFR: 8276229: Stop allowing implicit updates in G1BlockOffsetTable

Stefan Johansson sjohanss at openjdk.java.net
Tue Nov 16 08:30:42 UTC 2021


On Wed, 10 Nov 2021 13:42:05 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Please review this change that removes some code that is no longer needed.
>> 
>> **Summary**
>> In [JDK-8272083](https://bugs.openjdk.java.net/browse/JDK-8276229) G1 was changed to create a precise and complete BOT during evacuation (in the GC pause). This removes the need to update the BOT when looking up block starts. This previously happened in concurrent refinement and while scanning the heap in the pause, but with the precise BOT we should no longer require any such updates.
>> 
>> This change removes the slow path function that did the updates `forward_to_block_containing_addr_slow(...)`. It also renames some functions to no longer include a `_const` post-fix because there are only "const" versions of the functions remaining.
>> 
>> **Testing**
>> 
>> - [x] Mach5 1-3
>> - [x] Local stress-testing
>
> Marked as reviewed by tschatzl (Reviewer).

Thanks @tschatzl and @albertnetymk for the reviews.

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

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



More information about the hotspot-gc-dev mailing list