RFR: 8329603: G1: Merge G1BlockOffsetTablePart into G1BlockOffsetTable

Albert Mingkun Yang ayang at openjdk.org
Thu Apr 4 17:37:09 UTC 2024


On Thu, 4 Apr 2024 16:55:49 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> This patch merges `G1BlockOffsetTablePart` into `G1BlockOffsetTable`. The previous fields `_reserved` and `_offset_base` of `G1BlockOffsetTable` are marked as `static` so that they can be shared by BOTs of all the heap regions.
> 
> The tests `make test-tier1_gc` passed locally. Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxiong

src/hotspot/share/gc/g1/g1BlockOffsetTable.hpp line 56:

> 54: 
> 55:   // The region that owns this BOT.
> 56:   HeapRegion* _hr;

Is it possible to have a single bot in `G1CollectedHeap` and every region has a pointer to that?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18634#discussion_r1552135704


More information about the hotspot-gc-dev mailing list