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